psu-ng: Change logic to clear brownout
The chassis component only recognizes brownout clearing and performs
auto power restart when the chassis state is off. Update the power
supply application to conform to this expectation.
Tested with a power level disturbance (PLD) hardware tester through a
matrix of varying durations and voltage percentages.
Signed-off-by: Jim Wright <jlwright@us.ibm.com>
Change-Id: I9938e9b3e2edc3954cc14b330a517889317b4477
diff --git a/phosphor-power-supply/psu_manager.hpp b/phosphor-power-supply/psu_manager.hpp
index 5d2dce3..2ee61a6 100644
--- a/phosphor-power-supply/psu_manager.hpp
+++ b/phosphor-power-supply/psu_manager.hpp
@@ -178,6 +178,12 @@
*/
void analyze();
+ /**
+ * @brief Analyze the set of the power supplies for a brownout failure. Log
+ * error when necessary, clear brownout condition when window has passed.
+ */
+ void analyzeBrownout();
+
/** @brief True if the power is on. */
bool powerOn = false;
@@ -317,28 +323,6 @@
void setPowerConfigGPIO();
/**
- * @brief Determine if system is in brownout failure
- * @param additionalData AdditionalData property of the error log entry
- * @return true if system is in brownout failure, false otherwise.
- */
- bool isBrownout(std::map<std::string, std::string>& additionalData);
-
- /**
- * @brief Indicate that the system is in a brownout condition by creating an
- * error log and setting the PowerSystemInputs status property to Fault.
- *
- * @param[in] additionalData - Contains debug information on the number of
- * PSUs in fault state or not present.
- */
- void setBrownout(std::map<std::string, std::string>& additionalData);
-
- /**
- * @brief Indicate that the system is no longer in a brownout condition by
- * setting the PowerSystemInputs status property to Good.
- */
- void clearBrownout();
-
- /**
* @brief Map of supported PSU configurations that include the model name
* and their properties.
*/