commit 5f1d7f7adeef902d5776d1af528129892a53d1c3
author Jason M. Bills <jason.m.bills@intel.com> Fri May 12 13:58:35 2023 -0700
committer Jason M. Bills <jason.m.bills@intel.com> Fri May 12 15:13:21 2023 -0700
tree e02eecd424b20f3a9d8c314a829fe0db5e7e3080
parent 7169faac9eb8a428b4a2ecb680a071f732842004

Reset the cc when looping

It's possible if one loop passes and the next fails early that the cc
will be left at the previous value. This resets it for each loop so we
don't get stale ccs reported.

Tested:
Confirmed that the cc changes to 0 on failures while looping.

Change-Id: I5c1dc62cbd9309175d7add2475366329d295bd6c
Signed-off-by: Jason M. Bills <jason.m.bills@intel.com>

peci_cmds.c

diff --git a/peci_cmds.c b/peci_cmds.c
index a900cf8..10e89d8 100644
--- a/peci_cmds.c
+++ b/peci_cmds.c
@@ -387,6 +387,7 @@
         }
         while (loops--)
         {
+            cc = 0; // reset the cc for each loop
             clock_gettime(CLOCK_REALTIME, &begin);
             ret = peci_RdPkgConfig_dom(address, domainId, u8PkgIndex,
                                        u16PkgParam, u8Size,
@@ -442,6 +443,7 @@
         }
         while (loops--)
         {
+            cc = 0; // reset the cc for each loop
             clock_gettime(CLOCK_REALTIME, &begin);
             ret = peci_WrPkgConfig_dom(address, domainId, u8PkgIndex,
                                        u16PkgParam, u32PkgValue, u8Size, &cc);
@@ -490,6 +492,7 @@
         }
         while (loops--)
         {
+            cc = 0; // reset the cc for each loop
             clock_gettime(CLOCK_REALTIME, &begin);
             ret = peci_RdIAMSR_dom(address, domainId, u8MsrThread, u16MsrAddr,
                                    &u64MsrVal, &cc);
@@ -551,6 +554,7 @@
         }
         while (loops--)
         {
+            cc = 0; // reset the cc for each loop
             clock_gettime(CLOCK_REALTIME, &begin);
             ret = peci_RdPCIConfig_dom(address, domainId, u8PciBus, u8PciDev,
                                        u8PciFunc, u16PciReg,
@@ -613,6 +617,7 @@
         }
         while (loops--)
         {
+            cc = 0; // reset the cc for each loop
             clock_gettime(CLOCK_REALTIME, &begin);
             ret = peci_RdPCIConfigLocal_dom(
                 address, domainId, u8PciBus, u8PciDev, u8PciFunc, u16PciReg,
@@ -676,6 +681,7 @@
         }
         while (loops--)
         {
+            cc = 0; // reset the cc for each loop
             clock_gettime(CLOCK_REALTIME, &begin);
             ret = peci_WrPCIConfigLocal_dom(address, domainId, u8PciBus,
                                             u8PciDev, u8PciFunc, u16PciReg,
@@ -730,6 +736,7 @@
         }
         while (loops--)
         {
+            cc = 0; // reset the cc for each loop
             clock_gettime(CLOCK_REALTIME, &begin);
             ret = peci_RdEndPointConfigPciLocal_dom(
                 address, domainId, u8Seg, u8PciBus, u8PciDev, u8PciFunc,
@@ -791,6 +798,7 @@
         }
         while (loops--)
         {
+            cc = 0; // reset the cc for each loop
             clock_gettime(CLOCK_REALTIME, &begin);
             ret = peci_WrEndPointPCIConfigLocal_dom(
                 address, domainId, u8Seg, u8PciBus, u8PciDev, u8PciFunc,
@@ -843,6 +851,7 @@
         }
         while (loops--)
         {
+            cc = 0; // reset the cc for each loop
             clock_gettime(CLOCK_REALTIME, &begin);
             ret = peci_RdEndPointConfigPci_dom(
                 address, domainId, u8Seg, u8PciBus, u8PciDev, u8PciFunc,
@@ -903,6 +912,7 @@
         }
         while (loops--)
         {
+            cc = 0; // reset the cc for each loop
             clock_gettime(CLOCK_REALTIME, &begin);
             ret = peci_WrEndPointPCIConfig_dom(
                 address, domainId, u8Seg, u8PciBus, u8PciDev, u8PciFunc,
@@ -959,6 +969,7 @@
         }
         while (loops--)
         {
+            cc = 0; // reset the cc for each loop
             clock_gettime(CLOCK_REALTIME, &begin);
             ret = peci_RdEndPointConfigMmio_dom(
                 address, domainId, u8Seg, u8PciBus, u8PciDev, u8PciFunc, u8Bar,
@@ -1022,6 +1033,7 @@
         }
         while (loops--)
         {
+            cc = 0; // reset the cc for each loop
             clock_gettime(CLOCK_REALTIME, &begin);
             ret = peci_WrEndPointConfigMmio_dom(
                 address, domainId, u8Seg, u8PciBus, u8PciDev, u8PciFunc, u8Bar,