diff options
author | Alvin Wang <wangat@tw.ibm.com> | 2018-10-31 19:48:46 -0500 |
---|---|---|
committer | Sachin Gupta <sgupta2m@in.ibm.com> | 2019-01-16 03:12:58 -0600 |
commit | dbdcd7a7c9dc7630aba13401aa0981c8eb5991e8 (patch) | |
tree | dbed0921a90f03d97189908525d9c04ad1aa1ea7 | |
parent | 37a2ec2b135464f51a61164ef8d5466831917209 (diff) | |
download | talos-sbe-dbdcd7a7c9dc7630aba13401aa0981c8eb5991e8.tar.gz talos-sbe-dbdcd7a7c9dc7630aba13401aa0981c8eb5991e8.zip |
Update setup_fw_boot_config() to read out actual values from attributes
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>
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/69961
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Sachin Gupta <sgupta2m@in.ibm.com>
-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 0a6ad3e3..839b792e 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 @@ -126,16 +126,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 } /// |