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/power_supply.cpp b/phosphor-power-supply/power_supply.cpp
index c68e9be..c965892 100644
--- a/phosphor-power-supply/power_supply.cpp
+++ b/phosphor-power-supply/power_supply.cpp
@@ -509,19 +509,20 @@
// Remember that this PSU has seen an AC fault
acFault = AC_FAULT_LIMIT;
}
-
- if (vinUVFault &&
- !(statusWord & phosphor::pmbus::status_word::VIN_UV_FAULT))
+ else
{
- log<level::INFO>(
- fmt::format("{} VIN_UV fault cleared: STATUS_WORD = {:#06x}, "
- "STATUS_MFR_SPECIFIC = {:#04x}, "
- "STATUS_INPUT = {:#04x}",
- shortName, statusWord, statusMFR, statusInput)
- .c_str());
- vinUVFault = 0;
+ if (vinUVFault != 0)
+ {
+ log<level::INFO>(
+ fmt::format("{} VIN_UV fault cleared: STATUS_WORD = {:#06x}, "
+ "STATUS_MFR_SPECIFIC = {:#04x}, "
+ "STATUS_INPUT = {:#04x}",
+ shortName, statusWord, statusMFR, statusInput)
+ .c_str());
+ vinUVFault = 0;
+ }
// No AC fail, decrement counter
- if (acFault)
+ if (acFault != 0)
{
--acFault;
}
@@ -615,7 +616,7 @@
clearFaultFlags();
// No AC fail, decrement counter
- if (acFault)
+ if (acFault != 0)
{
--acFault;
}