commit 9a13daaefbf2da4bcecca625b2f4b09fc9fb77f0
author AppaRao Puli <apparao.puli@linux.intel.com> Mon Jul 13 17:53:00 2020 +0530
committer Vernon Mauery <vernon.mauery@linux.intel.com> Tue Jul 21 23:52:52 2020 +0000
tree fa7ef54247ca7692f16c5dd2d1b054ed22210460
parent 65b66ad8db83a9b1b1ec2e2ef2372d059a9033cb

minor fix: clear CMOS command CC correction

The clearCMOS command is supported only in manufacturing
mode. Outside of MTM mode, It should return Invalid
command(0xC1) but its returning 0xD4. Corrected the
CC to 0xC1.

Tested:
 - Outside MTM mode cleaCMOS returns 0xC1.

Change-Id: Icd747f8e904dbafc4a35fef0fb55a65ee4721a9b
Signed-off-by: AppaRao Puli <apparao.puli@linux.intel.com>

src/manufacturingcommands.cpp

diff --git a/src/manufacturingcommands.cpp b/src/manufacturingcommands.cpp
index 3b48bb2..6423289 100644
--- a/src/manufacturingcommands.cpp
+++ b/src/manufacturingcommands.cpp
@@ -789,6 +789,7 @@
         case makeCmdKey(ipmi::netFnOemOne,
                         ipmi::intel::general::cmdGetManufacturingData):
         case makeCmdKey(ipmi::netFnStorage, ipmi::storage::cmdWriteFruData):
+        case makeCmdKey(ipmi::netFnOemTwo, ipmi::intel::platform::cmdClearCMOS):
 
             // Check for Special mode
             if (mtm.getMfgMode() == SpecialMode::none)
@@ -970,11 +971,6 @@
     std::vector<uint8_t> writeData = {0x61, 0x1};
     std::vector<uint8_t> readBuf(0);
 
-    if (mtm.getMfgMode() == SpecialMode::none)
-    {
-        return ipmi::responseInsufficientPrivilege();
-    }
-
     ipmi::Cc retI2C = ipmi::i2cWriteRead(i2cBus, slaveAddr, writeData, readBuf);
     return ipmi::response(retI2C);
 }