From bab51274cbfb901e2c6d1f2e76513eb4c728f9f5 Mon Sep 17 00:00:00 2001 From: "Richard J. Knight" Date: Wed, 3 Feb 2016 10:36:52 -0600 Subject: 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 --- src/include/usr/ipmi/ipmisensor.H | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) (limited to 'src/include/usr/ipmi') 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. -- cgit v1.2.3