diff options
author | Claus Michael Olsen <cmolsen@us.ibm.com> | 2017-04-05 05:16:51 -0500 |
---|---|---|
committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2017-12-10 16:02:46 -0500 |
commit | b78b33c5a32ac5c12ca0fd980f054a86258eec00 (patch) | |
tree | 10cb7d12bd76545fa9245973ffee12f9c4b61903 /src/import/chips/p9/utils/imageProcs/p9_infrastruct_help.H | |
parent | fabc7bd251908585f097d37fcc0cadd29a2ee635 (diff) | |
download | talos-hostboot-b78b33c5a32ac5c12ca0fd980f054a86258eec00.tar.gz talos-hostboot-b78b33c5a32ac5c12ca0fd980f054a86258eec00.zip |
Code restruct: TOR API
Key_Cronus_Test=XIP_REGRESS
Code restructuring aiming at:
- utilizing TOR magic header info
- enforce a common approach for
- extracting metadata for all image,chipType combinations
- traversing images for all image,chipType combinations
- shrinking code size by reusing common code segments
- improve readability by
- separating more clearly metadata extraction and image traversal
- slight rearrange of certain code segments
- remove leftover hardcoded assumptions about ring/TOR data and structs
- variables appropriately renamed and now all using camel style
Change-Id: I50ace8b2fdb340a97ce6d74ce545c5e1acd21c40
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/38863
Tested-by: HWSV CI <hwsv-ci+hostboot@us.ibm.com>
Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Cronus HW CI <cronushw-ci+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: GIRISANKAR PAULRAJ <gpaulraj@in.ibm.com>
Reviewed-by: Thi N. Tran <thi@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/43251
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/import/chips/p9/utils/imageProcs/p9_infrastruct_help.H')
-rw-r--r-- | src/import/chips/p9/utils/imageProcs/p9_infrastruct_help.H | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/src/import/chips/p9/utils/imageProcs/p9_infrastruct_help.H b/src/import/chips/p9/utils/imageProcs/p9_infrastruct_help.H index bb8ab5c1f..736f6025b 100644 --- a/src/import/chips/p9/utils/imageProcs/p9_infrastruct_help.H +++ b/src/import/chips/p9/utils/imageProcs/p9_infrastruct_help.H @@ -53,14 +53,13 @@ const uint32_t MAX_HBBL_SIZE = 20 * 1024; // Max hbbl section siz const uint32_t MAX_NOOF_DD_LEVELS_IN_IMAGE = 20; //@FIXME: CMO: Aren't these defined somewhere else? -#define NUM_OF_CORES 24 -#define NUM_OF_CMES 12 -#define NUM_OF_QUADS 6 -#define CORES_PER_QUAD (NUM_OF_CORES/NUM_OF_QUADS) +#define NUM_OF_CORES (uint8_t)24 +#define NUM_OF_CMES (uint8_t)12 +#define NUM_OF_QUADS (uint8_t) 6 +#define CORES_PER_QUAD (NUM_OF_CORES/NUM_OF_QUADS) -#define INFRASTRUCT_RC_SUCCESS 0 -#define INFRASTRUCT_RC_CODE_BUG 1 -#define INFRASTRUCT_RC_INVALID_IMAGE_SECTION_TYPE 2 +#define INSTANCE_ID_MIN (uint8_t)0x01 +#define INSTANCE_ID_MAX (uint8_t)0x37 enum SYSPHASE { |