diff options
author | Claus Michael Olsen <cmolsen@us.ibm.com> | 2017-05-16 05:21:33 -0500 |
---|---|---|
committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2017-06-15 12:24:59 -0400 |
commit | 87cf91da75417c4596a6f18b4ca4586ebcfe9e71 (patch) | |
tree | 3410f7c6bc0eb386fbb466a04e4d0f5a57561756 /src/import/chips/p9/xip | |
parent | 64a09f22b6dc58965212b2713da3be30ed15ee3b (diff) | |
download | talos-hostboot-87cf91da75417c4596a6f18b4ca4586ebcfe9e71.tar.gz talos-hostboot-87cf91da75417c4596a6f18b4ca4586ebcfe9e71.zip |
xip_image: GPTR/overlays stage 1 support
Updated p9_xip_dd_section_support to return "overlays not
supported" if sectionId==.overlays is passed as arg.
Further add a small update to xip_image.C to clear the
ddSupport field in XipSection upon deleting a section.
Change-Id: If64a8a6d7046aa857bd2c3f8ca8c1092565df5ad
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/40546
Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Richard J. Knight <rjknight@us.ibm.com>
Reviewed-by: Sumit Kumar <sumit_kumar@in.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/40648
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/import/chips/p9/xip')
-rw-r--r-- | src/import/chips/p9/xip/p9_xip_image.C | 6 | ||||
-rw-r--r-- | src/import/chips/p9/xip/p9_xip_image.h | 3 |
2 files changed, 8 insertions, 1 deletions
diff --git a/src/import/chips/p9/xip/p9_xip_image.C b/src/import/chips/p9/xip/p9_xip_image.C index ef57004d8..0dbf5cf6d 100644 --- a/src/import/chips/p9/xip/p9_xip_image.C +++ b/src/import/chips/p9/xip/p9_xip_image.C @@ -559,7 +559,6 @@ xipSetSectionSize(void* io_image, const int i_section, const uint32_t i_size) /// Set the properties of a section -// XIP_STATIC int xipSetSectionProps(void* io_image, const int i_section, const uint8_t i_props) { @@ -3265,6 +3264,11 @@ int p9_xip_dd_section_support(const void* i_image, int rc; P9XipSection section; + if (i_sectionId == P9_XIP_SECTION_HW_OVERLAYS) + { + return P9_XIP_OVERLAYS_NOT_SUPPORTED; + } + rc = p9_xip_get_section(i_image, i_sectionId, §ion); if (!rc) diff --git a/src/import/chips/p9/xip/p9_xip_image.h b/src/import/chips/p9/xip/p9_xip_image.h index 859cdd89e..06d339981 100644 --- a/src/import/chips/p9/xip/p9_xip_image.h +++ b/src/import/chips/p9/xip/p9_xip_image.h @@ -1506,6 +1506,8 @@ p9_xip_decode_toc_dump(void* i_image, void* i_dump, /// Code bug in the ddLevel handling codes #define P9_XIP_DDLEVEL_CODE_BUG 22 +/// Temporary error code to help prevent image co-req for stage 2 release of RTC174302 +#define P9_XIP_OVERLAYS_NOT_SUPPORTED 23 /// Applications can expand this macro to declare an array of string forms of /// the error codes if desired. @@ -1534,6 +1536,7 @@ p9_xip_decode_toc_dump(void* i_image, void* i_dump, "P9_XIP_NO_DDLEVEL_SUPPORT", \ "P9_XIP_DDLEVEL_NOT_FOUND", \ "P9_XIP_DDLEVEL_CODE_BUG", \ + "P9_XIP_OVERLAYS_NOT_SUPPORTED", \ } /// Applications can use this macro to safely index the array of error |