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>

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
 }
 
 ///