Change order of powered on fault checks
It seems to make more sense to see the checks logging errors in the
order that they lead to the power supply shutting off.
A fan fault could lead to a thermal fault, which could lead to the power
supply shutting down when it should be on (if it did not have the fan or
thermal fault).
I am re-ordering the powered on checks to be in an order that seems to
make more sense.
Resolves: openbmc/openbmc#1732
Change-Id: I5a36678d03722a2de70d0e8ec5d9d4c04e07f943
Signed-off-by: Brandon Wyman <bjwyman@gmail.com>
diff --git a/power-supply/power_supply.cpp b/power-supply/power_supply.cpp
index 670065b..379aa46 100644
--- a/power-supply/power_supply.cpp
+++ b/power-supply/power_supply.cpp
@@ -123,11 +123,11 @@
if (powerOn)
{
- checkPGOrUnitOffFault(statusWord);
- checkCurrentOutOverCurrentFault(statusWord);
- checkOutputOvervoltageFault(statusWord);
checkFanFault(statusWord);
checkTemperatureFault(statusWord);
+ checkOutputOvervoltageFault(statusWord);
+ checkCurrentOutOverCurrentFault(statusWord);
+ checkPGOrUnitOffFault(statusWord);
}
}
}