DIMM Present sensor not active after dimm is deconfigured

    -Modified set sensor reading command to not use the
     bit by default in the operation field which triggers the BMC
     to overwrite the existing assertion mask.

    -Updated the reboot count sensor to use the operation
     specified by AMI to overwrite existing data in assertion
     mask field.

Fixes open-power/hostboot#47

Change-Id: I81958225480f495f8b0d4f523d0c8109832475f6
CQ:SW328073
Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/23947
Tested-by: Jenkins Server
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>

1 files changed, 10 insertions, 1 deletions

diff --git a/src/include/usr/ipmi/ipmisensor.H b/src/include/usr/ipmi/ipmisensor.H
index 23dc594da..9f57d4ded 100644
--- a/src/include/usr/ipmi/ipmisensor.H
+++ b/src/include/usr/ipmi/ipmisensor.H
@@ -196,8 +196,17 @@ namespace SENSOR
     //                  the event deassertion bits, zeros are ignored.
     //      [0:1] 01b - write the given value to the sensor reading byte.
     //
+    //      Note: Per AMI implementation bits 0:1 will force the assertion
+    //            mask value as sent to be the new sensor value, all other bits
+    //            are ignored.
+    //
     //      default operation = 1010 1001
-    static const uint8_t DEFAULT_OPERATION = 0xA9;  // see IPMI Spec. "Set
+    static const uint8_t DEFAULT_OPERATION = 0xA8;  // see IPMI Spec. "Set
+
+    // use this value for sensors which use assertion fields for sensor values
+    // eg. Reboot count
+    static const uint8_t SET_SENSOR_VALUE_OPERATION = 0x01;
+
     // Sensor Reading and Event
     // Status command" for
     // details on this byte.