diff options
author | Alvin Wang <wangat@tw.ibm.com> | 2018-10-31 19:48:46 -0500 |
---|---|---|
committer | Raja Das <rajadas2@in.ibm.com> | 2019-07-26 00:53:16 -0500 |
commit | 2932b6fa080ef2e104b7c4bafc9c45d16f465ff7 (patch) | |
tree | 6e72d4b695516e6a9e5d1f651b2e3717520c419d /src/import/chips/ocmb/explorer/procedures/hwp/memory/lib | |
parent | 226899a2098c170c9183170724c99f2de33bac06 (diff) | |
download | talos-sbe-2932b6fa080ef2e104b7c4bafc9c45d16f465ff7.tar.gz talos-sbe-2932b6fa080ef2e104b7c4bafc9c45d16f465ff7.zip |
Update setup_fw_boot_config() to read out actual values from attributes
Change-Id: I58005ae1a612f664a6463300395b49d4d5df8b1c
Original-Change-Id: I2574f01fceb79fb0d3234010581aab92d48ce409
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/68248
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Louis Stermole <stermole@us.ibm.com>
Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com>
Reviewed-by: JEREMY R. NEATON <jrneaton@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Diffstat (limited to 'src/import/chips/ocmb/explorer/procedures/hwp/memory/lib')
-rw-r--r-- | src/import/chips/ocmb/explorer/procedures/hwp/memory/lib/i2c/exp_i2c.H | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/src/import/chips/ocmb/explorer/procedures/hwp/memory/lib/i2c/exp_i2c.H b/src/import/chips/ocmb/explorer/procedures/hwp/memory/lib/i2c/exp_i2c.H index 07b6ca3e..986b5b75 100644 --- a/src/import/chips/ocmb/explorer/procedures/hwp/memory/lib/i2c/exp_i2c.H +++ b/src/import/chips/ocmb/explorer/procedures/hwp/memory/lib/i2c/exp_i2c.H @@ -128,16 +128,20 @@ fapi_try_exit: inline void boot_config_setup(std::vector<uint8_t>& io_data) { + // Need data length as well - boot config can only ever be written - io_data.push_back(FW_BOOT_CONFIG_BYTE_LEN); + io_data.insert(io_data.begin(), FW_BOOT_CONFIG_BYTE_LEN); // Then add the command - io_data.push_back(FW_BOOT_CONFIG); + io_data.insert(io_data.begin(), FW_BOOT_CONFIG); - // Written commands need to be in the form of (MSB first) + // Written commands need to be in the form of (MSB first). Confirmed by hardware characterization team. // CMD // DATA LEN - // DATA + // DATA3 + // DATA2 + // DATA1 + // DATA0 } /// |