PEL: Set platform SAI LED group
There is now an LED group to set when the platform SAI LED needs to be
asserted when handling the service indicators for PEL callouts.
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: I2af66c8c1820c131e1d4d7dede802b5f128b79a4
diff --git a/test/openpower-pels/service_indicators_test.cpp b/test/openpower-pels/service_indicators_test.cpp
index 8f1d9ab..f83edeb 100644
--- a/test/openpower-pels/service_indicators_test.cpp
+++ b/test/openpower-pels/service_indicators_test.cpp
@@ -282,7 +282,24 @@
lightPath.activate(pel);
}
- // Make getInventoryFromLocCode fail
+ // A non-info BMC PEL with no callouts will set the platform SAI LED.
+ {
+ MockDataInterface dataIface;
+ service_indicators::LightPath lightPath{dataIface};
+
+ EXPECT_CALL(dataIface,
+ assertLEDGroup("/xyz/openbmc_project/led/groups/"
+ "platform_system_attention_indicator",
+ true))
+ .Times(1);
+
+ auto data = pelDataFactory(TestPELType::pelSimple);
+ PEL pel{data};
+
+ lightPath.activate(pel);
+ }
+
+ // Make getInventoryFromLocCode fail - will set the platform SAI LED
{
MockDataInterface dataIface;
service_indicators::LightPath lightPath{dataIface};
@@ -292,7 +309,11 @@
EXPECT_CALL(dataIface, getFaultLEDGroup(_)).Times(0);
- EXPECT_CALL(dataIface, assertLEDGroup(_, true)).Times(0);
+ EXPECT_CALL(dataIface,
+ assertLEDGroup("/xyz/openbmc_project/led/groups/"
+ "platform_system_attention_indicator",
+ true))
+ .Times(1);
auto data = pelFactory(1, 'O', 0x20, 0xA400, 500);
PEL pel{data};
@@ -300,7 +321,7 @@
lightPath.activate(pel);
}
- // Make getFaultLEDGroup fail
+ // Make getFaultLEDGroup fail - will set the platform SAI LED
{
MockDataInterface dataIface;
service_indicators::LightPath lightPath{dataIface};
@@ -311,7 +332,11 @@
EXPECT_CALL(dataIface, getFaultLEDGroup("/system/chassis/processor"))
.WillOnce(Throw(std::runtime_error("Fail")));
- EXPECT_CALL(dataIface, assertLEDGroup(_, true)).Times(0);
+ EXPECT_CALL(dataIface,
+ assertLEDGroup("/xyz/openbmc_project/led/groups/"
+ "platform_system_attention_indicator",
+ true))
+ .Times(1);
auto data = pelFactory(1, 'O', 0x20, 0xA400, 500);
PEL pel{data};