commit 5287c02c5f96b40f0941c9c72ab29d2c7ac44a96
author Jason M. Bills <jason.m.bills@intel.com> Tue May 19 11:16:09 2020 -0700
committer Jason M. Bills <jason.m.bills@intel.com> Wed Jun 10 14:26:19 2020 -0700
tree 23f1bb84fed6510b316bfa515c81c6c06b8b3f04
parent 1429ca8db1e59637dc00f880ebc5013c7f8540cd

Fix memory thermtrip detection

The memory thermtrip SGPIOs are not latched from the CPLD which
is causing spurious events to get logged on DC-cycles.

To fix this, the CPLD is adding latching for the memory
thermtrip SGPIOs, so BMC no longer needs to check host state on
memory thermtrip errors.

Also make both the memory thermtrip events falling-edge.

Change-Id: I72bbd12c0a33657eed312f87dbee8e4c31fb30eb
Signed-off-by: Jason M. Bills <jason.m.bills@intel.com>

src/host_error_monitor.cpp

diff --git a/src/host_error_monitor.cpp b/src/host_error_monitor.cpp
index 9047677..f8de2d3 100644
--- a/src/host_error_monitor.cpp
+++ b/src/host_error_monitor.cpp
@@ -812,17 +812,15 @@
 
 static void cpu1MemtripHandler()
 {
-    if (!hostOff)
-    {
-        gpiod::line_event gpioLineEvent = cpu1MemtripLine.event_read();
+    gpiod::line_event gpioLineEvent = cpu1MemtripLine.event_read();
 
-        bool cpu1Memtrip =
-            gpioLineEvent.event_type == gpiod::line_event::FALLING_EDGE;
-        if (cpu1Memtrip)
-        {
-            memThermTripLog(1);
-        }
+    bool cpu1Memtrip =
+        gpioLineEvent.event_type == gpiod::line_event::FALLING_EDGE;
+    if (cpu1Memtrip)
+    {
+        memThermTripLog(1);
     }
+
     cpu1MemtripEvent.async_wait(
         boost::asio::posix::stream_descriptor::wait_read,
         [](const boost::system::error_code ec) {
@@ -874,17 +872,15 @@
 
 static void cpu2MemtripHandler()
 {
-    if (!hostOff)
-    {
-        gpiod::line_event gpioLineEvent = cpu2MemtripLine.event_read();
+    gpiod::line_event gpioLineEvent = cpu2MemtripLine.event_read();
 
-        bool cpu2Memtrip =
-            gpioLineEvent.event_type == gpiod::line_event::RISING_EDGE;
-        if (cpu2Memtrip)
-        {
-            memThermTripLog(2);
-        }
+    bool cpu2Memtrip =
+        gpioLineEvent.event_type == gpiod::line_event::FALLING_EDGE;
+    if (cpu2Memtrip)
+    {
+        memThermTripLog(2);
     }
+
     cpu2MemtripEvent.async_wait(
         boost::asio::posix::stream_descriptor::wait_read,
         [](const boost::system::error_code ec) {