Modify logging output for easy debugging

Must use capital string for “entry”

Change-Id: I29bc7fd9ae591a7293e630c1f169dfdfcd5942c7
Signed-off-by: Yong Li <yong.b.li@linux.intel.com>

3 files changed, 14 insertions, 11 deletions

diff --git a/services/chassis/Buttons/inc/PowerButton.hpp b/services/chassis/Buttons/inc/PowerButton.hpp
index 6ec947e..4feb5f4 100644
--- a/services/chassis/Buttons/inc/PowerButton.hpp
+++ b/services/chassis/Buttons/inc/PowerButton.hpp
@@ -75,6 +75,9 @@ struct PowerButton
             return n;
         }
 
+        phosphor::logging::log<phosphor::logging::level::DEBUG>(
+            "PowerButton event", phosphor::logging::entry("BUF=%c", buf));
+
         if (buf == '0')
         {
             // emit pressed signal
diff --git a/services/chassis/I2c/src/I2c.cpp b/services/chassis/I2c/src/I2c.cpp
index 315e35d..0eaa503 100644
--- a/services/chassis/I2c/src/I2c.cpp
+++ b/services/chassis/I2c/src/I2c.cpp
@@ -16,7 +16,7 @@ int i2cSet(uint8_t bus, uint8_t slaveAddr, uint8_t regAddr, uint8_t value)
         phosphor::logging::log<phosphor::logging::level::ERR>(
             "i2c bus des not exist!",
             phosphor::logging::entry("PATH=%s", devPath.c_str()),
-            phosphor::logging::entry("slaveAddr=%d", slaveAddr));
+            phosphor::logging::entry("SLAVEADDR=%d", slaveAddr));
         return -1;
     }
 
@@ -26,7 +26,7 @@ int i2cSet(uint8_t bus, uint8_t slaveAddr, uint8_t regAddr, uint8_t value)
         phosphor::logging::log<phosphor::logging::level::ERR>(
             "Error in open!",
             phosphor::logging::entry("PATH=%s", devPath.c_str()),
-            phosphor::logging::entry("slaveAddr=%d", slaveAddr));
+            phosphor::logging::entry("SLAVEADDR=%d", slaveAddr));
         return -1;
     }
 
@@ -35,7 +35,7 @@ int i2cSet(uint8_t bus, uint8_t slaveAddr, uint8_t regAddr, uint8_t value)
         phosphor::logging::log<phosphor::logging::level::ERR>(
             "Error in I2C_FUNCS!",
             phosphor::logging::entry("PATH=%s", devPath.c_str()),
-            phosphor::logging::entry("slaveAddr=%d", slaveAddr));
+            phosphor::logging::entry("SLAVEADDR=%d", slaveAddr));
 
         ::close(fd);
         return -1;
@@ -47,7 +47,7 @@ int i2cSet(uint8_t bus, uint8_t slaveAddr, uint8_t regAddr, uint8_t value)
         phosphor::logging::log<phosphor::logging::level::ERR>(
             "i2c bus des not support write!",
             phosphor::logging::entry("PATH=%s", devPath.c_str()),
-            phosphor::logging::entry("slaveAddr=%d", slaveAddr));
+            phosphor::logging::entry("SLAVEADDR=%d", slaveAddr));
         ::close(fd);
         return -1;
     }
@@ -57,7 +57,7 @@ int i2cSet(uint8_t bus, uint8_t slaveAddr, uint8_t regAddr, uint8_t value)
         phosphor::logging::log<phosphor::logging::level::ERR>(
             "Error in I2C_SLAVE_FORCE!",
             phosphor::logging::entry("PATH=%s", devPath.c_str()),
-            phosphor::logging::entry("slaveAddr=%d", slaveAddr));
+            phosphor::logging::entry("SLAVEADDR=%d", slaveAddr));
         ::close(fd);
         return -1;
     }
@@ -67,7 +67,7 @@ int i2cSet(uint8_t bus, uint8_t slaveAddr, uint8_t regAddr, uint8_t value)
         phosphor::logging::log<phosphor::logging::level::ERR>(
             "Error in i2c write!",
             phosphor::logging::entry("PATH=%s", devPath.c_str()),
-            phosphor::logging::entry("slaveAddr=%d", slaveAddr));
+            phosphor::logging::entry("SLAVEADDR=%d", slaveAddr));
         ::close(fd);
         return -1;
     }
@@ -75,9 +75,9 @@ int i2cSet(uint8_t bus, uint8_t slaveAddr, uint8_t regAddr, uint8_t value)
     phosphor::logging::log<phosphor::logging::level::DEBUG>(
         "i2cset successfully",
         phosphor::logging::entry("PATH=%s", devPath.c_str()),
-        phosphor::logging::entry("slaveAddr=0x%x", slaveAddr),
-        phosphor::logging::entry("regAddr=0x%x", regAddr),
-        phosphor::logging::entry("value=0x%x", value));
+        phosphor::logging::entry("SLAVEADDR=0x%x", slaveAddr),
+        phosphor::logging::entry("REGADDR=0x%x", regAddr),
+        phosphor::logging::entry("VALUE=0x%x", value));
     ::close(fd);
     return 0;
 }
\ No newline at end of file
diff --git a/services/chassis/PowerControl/src/PowerControl.cpp b/services/chassis/PowerControl/src/PowerControl.cpp
index 3784c28..caa69e5 100644
--- a/services/chassis/PowerControl/src/PowerControl.cpp
+++ b/services/chassis/PowerControl/src/PowerControl.cpp
@@ -33,11 +33,11 @@ int32_t PowerControl::setPowerState(int32_t newState) {
   char buf;
 
   phosphor::logging::log<phosphor::logging::level::DEBUG>(
-      "setPowerState", phosphor::logging::entry("newState=%d", newState));
+      "setPowerState", phosphor::logging::entry("NEWSTATE=%d", newState));
 
   if (state() == newState) {
     phosphor::logging::log<phosphor::logging::level::DEBUG>(
-        "Same powerstate", phosphor::logging::entry("newState=%d", newState));
+        "Same powerstate", phosphor::logging::entry("NEWSTATE=%d", newState));
   }
 
   if (newState == 1) {