commit 43d3263ca782c5894b7bd7fc48bdd1e956e574a6
author Brandon Wyman <bjwyman@gmail.com> Tue Oct 26 23:44:11 2021 +0000
committer Brandon Wyman <bjwyman@gmail.com> Fri Oct 29 21:15:45 2021 +0000
tree 3b96ebb59911ebae9975d6996167b4a15661d4e4
parent 6710ba2cc776dc9e4306807f4f586c22822100b4

psu-ng: Change fault traces from INFO to ERR

Having fault bits on in the various PMBus STATUS_* commands responses is
basically bad, an error. Change the journal tracing from
log<level::INFO> to log<level::ERR>.

Tested:
    Verified that tracing for power supply fault bits is red.
    Tested CML fault, INPUT fault, and VOUT faults.

Change-Id: If3efda267799630b1bb560347392b03ca73a79f1
Signed-off-by: Brandon Wyman <bjwyman@gmail.com>

phosphor-power-supply/power_supply.cpp

diff --git a/phosphor-power-supply/power_supply.cpp b/phosphor-power-supply/power_supply.cpp
index 2102e53..3f3a595 100644
--- a/phosphor-power-supply/power_supply.cpp
+++ b/phosphor-power-supply/power_supply.cpp
@@ -217,7 +217,7 @@
                 {
                     if (!cmlFault)
                     {
-                        log<level::INFO>(
+                        log<level::ERR>(
                             fmt::format("CML fault: STATUS_WORD = {:#04x}, "
                                         "STATUS_CML = {:#02x}",
                                         statusWord, statusCML)
@@ -231,7 +231,7 @@
                 {
                     if (!inputFault)
                     {
-                        log<level::INFO>(
+                        log<level::ERR>(
                             fmt::format("INPUT fault: STATUS_WORD = {:#04x}, "
                                         "STATUS_MFR_SPECIFIC = {:#02x}, "
                                         "STATUS_INPUT = {:#02x}",
@@ -247,7 +247,7 @@
                 {
                     if (!voutOVFault)
                     {
-                        log<level::INFO>(
+                        log<level::ERR>(
                             fmt::format("INPUT fault: STATUS_WORD = {:#04x}, "
                                         "STATUS_MFR_SPECIFIC = {:#02x}, "
                                         "STATUS_VOUT = {:#02x}",
@@ -277,7 +277,7 @@
                 {
                     if (!vinUVFault)
                     {
-                        log<level::INFO>(
+                        log<level::ERR>(
                             fmt::format("VIN_UV fault: STATUS_WORD = {:#04x}, "
                                         "STATUS_MFR_SPECIFIC = {:#02x}, "
                                         "STATUS_INPUT = {:#02x}",