PEL: Change method of asserting a fault LED
There was a recent change in direction on how PELs should request that
fault LEDs be turned on. Previously, the code would talk to the LED
group objects directly. The new direction is to set the Functional
property on the OperationalStatus interface on the inventory objects in
question to false, and the LED manager code will watch that to know when
to turn on the LEDs.
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: Ieebb09ba002843cf863359a09aba26540356aa91
diff --git a/extensions/openpower-pels/service_indicators.hpp b/extensions/openpower-pels/service_indicators.hpp
index 522f3db..85adf2b 100644
--- a/extensions/openpower-pels/service_indicators.hpp
+++ b/extensions/openpower-pels/service_indicators.hpp
@@ -123,22 +123,26 @@
private:
/**
- * @brief Returns the LED group D-Bus paths to use to turn on the
- * LEDs for the passed in location codes.
+ * @brief Returns the inventory D-Bus paths for the passed
+ * in location codes.
*
* @param[in] locationCodes - The location codes
*
- * @return std::vector<std::string> - The LED group D-Bus paths
+ * @return std::vector<std::string> - The inventory D-Bus paths
*/
std::vector<std::string>
- getLEDGroupPaths(const std::vector<std::string>& locationCodes) const;
+ getInventoryPaths(const std::vector<std::string>& locationCodes) const;
/**
- * @brief Sets the Assert property on the LED group D-Bus objects
+ * @brief Sets the Functional property on the passed in
+ * inventory paths to false.
*
- * @param[in] std::vector<std::string> - The LED group D-Bus paths
+ * There is code watching for this that will then turn on
+ * any LEDs for that FRU.
+ *
+ * @param[in] inventoryPaths - The inventory D-Bus paths
*/
- void assertLEDs(const std::vector<std::string>& ledGroups) const;
+ void setNotFunctional(const std::vector<std::string>& inventoryPaths) const;
/**
* @brief Checks if the callout priority is one that the policy