commit d065702bc3533fe7334b6fc08eddcb30b7295353
author Jayaprakash Mutyala <mutyalax.jayaprakash@intel.com> Thu Aug 26 21:18:08 2021 +0000
committer Vernon Mauery <vernon.mauery@linux.intel.com> Wed Sep 01 15:43:46 2021 +0000
tree 79c3294fa7baccabe1b44366d11ae80dd0b4c068
parent 5cb2c045856ebaef15fe7da714deb1266c13b901

Fix for ME version in get OEM Device info command

Issue: With current implementation, get OEM device info command is
giving incorrect ME minor version in response.

Fix: Populate ME minor version same as ME get device ID command minor
version.

Tested:
Verified using IPMI commands
Command: ipmitool raw 0x30 0x27 1 //get OEM device info
Response: 0a 00 72 ff ff ff ff 06 01 ff ff
//ME get device ID
Command:  ipmitool -I lanplus -H <bmc-ip> -U <username> -P <password>
          -C 17 -t 0x2c -b 6 raw 6 1
Response: 50 01 06 01 02 21 57 01 00 18 0b 06 02 70 01

Signed-off-by: Jayaprakash Mutyala <mutyalax.jayaprakash@intel.com>
Change-Id: Ie0c6533eda5cb59f41a92be0381a21768a2d1082

src/oemcommands.cpp

diff --git a/src/oemcommands.cpp b/src/oemcommands.cpp
index 0400ed2..3cd26bc 100644
--- a/src/oemcommands.cpp
+++ b/src/oemcommands.cpp
@@ -350,7 +350,8 @@
         if (results.size() == matchedPhosphor)
         {
             meMajor = static_cast<uint8_t>(std::stoi(results[1]));
-            meMinor = static_cast<uint8_t>(std::stoi(results[2]));
+            meMinor = static_cast<uint8_t>(std::stoi(results[2]) << 4 |
+                                           std::stoi(results[3]));
         }
     }
     return true;