Add parameter to set the MCERR assert polarity
Some systems have MCERR as high-assert, so this change removes the
hardcoded low-assert, and adds a parameter to select the polarity.
Tested:
Confirmed that MCERR polarity can be configured with the parameter.
Change-Id: I732354068495aefba0508c80e9042956e2fa35a5
Signed-off-by: Jason M. Bills <jason.m.bills@intel.com>
diff --git a/include/error_monitors/mcerr_monitor.hpp b/include/error_monitors/mcerr_monitor.hpp
index a0d2dce..cbe954d 100644
--- a/include/error_monitors/mcerr_monitor.hpp
+++ b/include/error_monitors/mcerr_monitor.hpp
@@ -23,9 +23,6 @@
class MCERRMonitor :
public host_error_monitor::base_gpio_monitor::BaseGPIOMonitor
{
- const static host_error_monitor::base_gpio_monitor::AssertValue
- assertValue =
- host_error_monitor::base_gpio_monitor::AssertValue::lowAssert;
const size_t cpuNum;
void logEvent() override
@@ -39,9 +36,12 @@
}
public:
- MCERRMonitor(boost::asio::io_service& io,
- std::shared_ptr<sdbusplus::asio::connection> conn,
- const std::string& signalName, const size_t cpuNum) :
+ MCERRMonitor(
+ boost::asio::io_service& io,
+ std::shared_ptr<sdbusplus::asio::connection> conn,
+ const std::string& signalName,
+ const host_error_monitor::base_gpio_monitor::AssertValue assertValue,
+ const size_t cpuNum) :
BaseGPIOMonitor(io, conn, signalName, assertValue),
cpuNum(cpuNum)
{