commit 63508a73e0eb47760ca2520517b16f6a9d177db2
author Brad Bishop <bradleyb@fuzziesquirrel.com> Tue Oct 27 18:55:01 2020 -0400
committer Jayanth Othayoth <ojayanth@in.ibm.com> Fri Oct 30 02:37:57 2020 +0000
tree ac24c3fa62efece97b39defc3421267563d2c913
parent 56d14d049bcb1966fac953d1bf07a9e5bedf5ebc

fix compiler warnings

Fix a couple pedantic, format=, and unused-parameter warnings so we can
turn the warnings all the way up.

Change-Id: I7c6309e30147f54fc64ace99dc784fbd96c77c70
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>

procedures/common/cfam_overrides.cpp

diff --git a/procedures/common/cfam_overrides.cpp b/procedures/common/cfam_overrides.cpp
index 13f7659..ea311f5 100644
--- a/procedures/common/cfam_overrides.cpp
+++ b/procedures/common/cfam_overrides.cpp
@@ -27,7 +27,7 @@
 
 void CFAMOverride()
 {
-    int pos = 0;
+    size_t pos = 0;
     cfam_address_t address = 0;
     cfam_data_t data = 0;
     cfam_mask_t mask = 0;
@@ -48,7 +48,7 @@
                 if (!line.empty() && line.at(0) != '#')
                 {
                     mask = 0xFFFFFFFF;
-                    if (sscanf(line.c_str(), "%x %hx %x %x", &pos, &address,
+                    if (sscanf(line.c_str(), "%zu %hx %x %x", &pos, &address,
                                &data, &mask) >= 3)
                     {
                         const auto& target = targets.getTarget(pos);
@@ -74,7 +74,7 @@
     return;
 }
 
-REGISTER_PROCEDURE("CFAMOverride", CFAMOverride);
+REGISTER_PROCEDURE("CFAMOverride", CFAMOverride)
 
 } // namespace p9
 } // namespace openpower