[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