aboutsummaryrefslogtreecommitdiff
path: root/include/fsp.h
diff options
context:
space:
mode:
authorOliver O'Halloran <oohall@gmail.com>2020-03-05 17:52:00 +1100
committerOliver O'Halloran <oohall@gmail.com>2020-03-11 17:24:16 +1100
commitb776bb0fa86b31873007c7e4f677dfb7a70e54d2 (patch)
treefafe6b2a357047bc10a99e0217d4048edaa582d2 /include/fsp.h
parent2e122fbadf7f1175202151f72ee4b81bcb811cb3 (diff)
downloadskiboot-b776bb0fa86b31873007c7e4f677dfb7a70e54d2.zip
skiboot-b776bb0fa86b31873007c7e4f677dfb7a70e54d2.tar.gz
skiboot-b776bb0fa86b31873007c7e4f677dfb7a70e54d2.tar.bz2
hw/fsp: Fix GENERIC_FAILURE mailbox status code
The 0xEF return code is used to tell the hypervisor that the FSP was not able to replicate an NVRAM write to the secondary FSP. The GENERIC_FAILURE is using this code instead of the correct 0xFE code which indicates a generic error condition. We already have a FSP_STATUS_GENERIC_ERROR for 0xFE so convert the existing users of FSP_STATUS_GENERIC_FAILURE to use GENERIC_ERROR and remove the duplicate. Reviewed-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com> Signed-off-by: Oliver O'Halloran <oohall@gmail.com>
Diffstat (limited to 'include/fsp.h')
-rw-r--r--include/fsp.h3
1 files changed, 1 insertions, 2 deletions
diff --git a/include/fsp.h b/include/fsp.h
index 92f5459..ec079a2 100644
--- a/include/fsp.h
+++ b/include/fsp.h
@@ -46,7 +46,6 @@
#define FSP_STATUS_FLASH_INPROGRESS 0x61
#define FSP_STATUS_FLASH_NOPROGRESS 0x62
#define FSP_STATUS_FLASH_INVALID_SIDE 0x63
-#define FSP_STATUS_GENERIC_ERROR 0xfe
#define FSP_STATUS_EOF_ERROR 0x02
#define FSP_STATUS_DMA_ERROR 0x24
#define FSP_STATUS_BUSY 0x3e
@@ -61,7 +60,7 @@
#define FSP_STATUS_TOD_PERMANENT_ERROR 0xAF /* Permanent error in TOD */
#define FSP_STATUS_I2C_TRANS_ERROR 0xE4 /* I2C device / transmission error */
#define FSP_STATUS_IND_PARTIAL_SUCCESS 0xE5 /* Indicator partial success */
-#define FSP_STATUS_GENERIC_FAILURE 0xEF /* Generic Failure in execution */
+#define FSP_STATUS_GENERIC_ERROR 0xFE /* Generic Failure in execution */
/*
* FSP registers