commit 8729e44a35e0b78c6119c769156cb46bd0987415
author Matthew Barth <msbarth@us.ibm.com> Mon Apr 02 09:36:33 2018 -0500
committer Matt Spinler <spinler@us.ibm.com> Fri Apr 13 14:33:41 2018 +0000
tree fbbb2fdbf8ba1e374e07947909cce3af58415a7c
parent 38c74e70d98d5dc9f106868252ea8afd0787430b

Remove OCC EAGAIN workaround

The ability to handle an EAGAIN for a sensor from the OCC device can be
done by defining the EAGAIN return code(11) as a sensor removal return
code within each OCC config file. This return code can be defined at the
device level within the config file to be applied across all sensors the
OCC provides.

Tested:
    OCC sensors are removed from Dbus when EAGAIN is returned

Resolves openbmc/openbmc#2327

Change-Id: If16ec37da6d960d54de1d47af3a39944249df772
Signed-off-by: Matthew Barth <msbarth@us.ibm.com>

sysfs.cpp

diff --git a/sysfs.cpp b/sysfs.cpp
index 5c7dbee..f689314 100644
--- a/sysfs.cpp
+++ b/sysfs.cpp
@@ -338,14 +338,7 @@
 
             if (isOCC)
             {
-                if (rc == EAGAIN)
-                {
-                    // For the OCCs, when an EAGAIN is return, just set the
-                    // value to 0 (0x00 = sensor is unavailable)
-                    val = 0;
-                    break;
-                }
-                else if (rc == EREMOTEIO)
+                if (rc == EREMOTEIO)
                 {
                     // For the OCCs, when an EREMOTEIO is return, set the
                     // value to 255*1000