summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTom Joseph <tomjoseph@in.ibm.com>2018-03-09 07:54:02 -0600
committerTom Joseph <tomjoseph@in.ibm.com>2018-03-16 16:13:43 +0000
commitaf8a09805df29e486cf5a195bc81ff0fadbe6f15 (patch)
tree3cd2d84e24c803ad1dd5fcd9f55297a8c020fd4d
parent744398dc49661363f32af3a34b71ca4b865c20e9 (diff)
downloadphosphor-host-ipmid-af8a09805df29e486cf5a195bc81ff0fadbe6f15.tar.gz
phosphor-host-ipmid-af8a09805df29e486cf5a195bc81ff0fadbe6f15.zip
Change the byte order for aux firmware revision
According to the specification, any generic utilities should display each byte as 2-digit hexadecimal numbers, with byte 13 displayed first as the most-significant byte. This patch corrects the byte order so that byte 13 represents the most-significant byte. Change-Id: Ie71d2af6a9e9f14c8d1d12cea7e45a57aa15a3dd Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>
-rw-r--r--apphandler.cpp8
1 files changed, 4 insertions, 4 deletions
diff --git a/apphandler.cpp b/apphandler.cpp
index 6564181..a53f491 100644
--- a/apphandler.cpp
+++ b/apphandler.cpp
@@ -302,10 +302,10 @@ ipmi_ret_t ipmi_app_get_device_id(ipmi_netfn_t netfn, ipmi_cmd_t cmd,
dev_id.manuf_id[0] = data.value("manuf_id", 0);
dev_id.prod_id[1] = data.value("prod_id", 0) >> 8;
dev_id.prod_id[0] = data.value("prod_id", 0);
- dev_id.aux[3] = data.value("aux", 0) >> 24;
- dev_id.aux[2] = data.value("aux", 0) >> 16;
- dev_id.aux[1] = data.value("aux", 0) >> 8;
- dev_id.aux[0] = data.value("aux", 0);
+ dev_id.aux[3] = data.value("aux", 0);
+ dev_id.aux[2] = data.value("aux", 0) >> 8;
+ dev_id.aux[1] = data.value("aux", 0) >> 16;
+ dev_id.aux[0] = data.value("aux", 0) >> 24;
//Don't read the file every time if successful
dev_id_initialized = true;
OpenPOWER on IntegriCloud