[Skiboot] [RESEND][PATCH] phb4/capp: Update the expected Eye-catcher for CAPP ucode lid
Vaibhav Jain
vaibhav at linux.ibm.com
Mon Sep 3 19:12:37 AEST 2018
Currently on a FSP based P9 system load_capp_code() expects CAPP ucode
lid header to have eye-catcher magic of 'CAPPPSLL'. However skiboot
currently supports CAPP ucode only lids that have a eye-catcher magic
of 'CAPPLIDH'. This prevents skiboot from loading the ucode with this
error message:
CAPP: ucode header invalid
We fix this issue by updating load_capp_ucode() to use the eye-catcher
value of 'CAPPLIDH' instead of 'CAPPPSLL'.
Cc: stable
Fixes: e50764d4f2b1("capi: Load capp microcode")
Signed-off-by: Vaibhav Jain <vaibhav at linux.ibm.com>
---
Change-log:
Resend -> Updated patch description & marked the patch to
stable. [Andrew]
---
hw/phb4.c | 4 ++--
include/capp.h | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/hw/phb4.c b/hw/phb4.c
index d1245dce..f9053f76 100644
--- a/hw/phb4.c
+++ b/hw/phb4.c
@@ -3001,9 +3001,9 @@ static int64_t load_capp_ucode(struct phb4 *p)
if (p->index != CAPP0_PHB_INDEX && p->index != CAPP1_PHB_INDEX)
return OPAL_HARDWARE;
- /* 0x4341505050534C4C = 'CAPPPSLL' in ASCII */
+ /* 0x434150504c494448 = 'CAPPLIDH' in ASCII */
rc = capp_load_ucode(p->chip_id, p->phb.opal_id, p->index,
- 0x4341505050534C4C, PHB4_CAPP_REG_OFFSET(p),
+ 0x434150504c494448, PHB4_CAPP_REG_OFFSET(p),
CAPP_APC_MASTER_ARRAY_ADDR_REG,
CAPP_APC_MASTER_ARRAY_WRITE_REG,
CAPP_SNP_ARRAY_ADDR_REG,
diff --git a/include/capp.h b/include/capp.h
index 597401d5..6ec3f7fe 100644
--- a/include/capp.h
+++ b/include/capp.h
@@ -19,7 +19,7 @@
/*
* eyecatcher PHB3: 'CAPPLIDH' in ASCII
- * eyecatcher PHB4: 'CAPPPSLL' in ASCII
+ * eyecatcher PHB4: 'CAPPLIDH' in ASCII
*/
struct capp_lid_hdr {
be64 eyecatcher;
--
2.17.1
More information about the Skiboot
mailing list