From 2932b6fa080ef2e104b7c4bafc9c45d16f465ff7 Mon Sep 17 00:00:00 2001 From: Alvin Wang Date: Wed, 31 Oct 2018 19:48:46 -0500 Subject: 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 Reviewed-by: Louis Stermole Reviewed-by: STEPHEN GLANCY Reviewed-by: JEREMY R. NEATON Tested-by: Hostboot CI Reviewed-by: Jennifer A. Stofer --- .../ocmb/explorer/procedures/hwp/memory/lib/i2c/exp_i2c.H | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) (limited to 'src') 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& 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 } /// -- cgit v1.2.3