Update addresses used in XSCOM/HWPF test cases to work in VBU

Disable XSCOM and HWPF test cases so automated test doesn't fail

Added XSCOM address in trace

Re-enable XSCOM and HWPF test cases to be used with new fips build

Change to backing build b0720a_1132.750 & fix test cases failure

Change-Id: Iac54bdcde54d2d8168455e1f51f6ff309f14918f
Reviewed-on: http://gfw160.austin.ibm.com:8080/gerrit/200
Tested-by: Jenkins Server
Reviewed-by: MIKE J. JONES <mjjones@us.ibm.com>
Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>

2 files changed, 70 insertions, 13 deletions

diff --git a/src/usr/xscom/test/xscomtest.H b/src/usr/xscom/test/xscomtest.H
index f1c7a017a..b2934bf4b 100644
--- a/src/usr/xscom/test/xscomtest.H
+++ b/src/usr/xscom/test/xscomtest.H
@@ -12,6 +12,7 @@
 #include <errl/errlentry.H>
 #include <errl/errltypes.H>
 #include <devicefw/userif.H>
+#include <xscom/xscomreasoncodes.H>
 
 extern trace_desc_t* g_trac_xscom;
 
@@ -25,11 +26,11 @@ struct testXscomAddrData
 };
 
 // Test table values
-//@todo - Select scratch register so chip doesn't get messed-up
 const testXscomAddrData g_xscomAddrTable[] =
 {
-    {0x08030007, 0x8000000000000001},
-    {0x08010587, 0x9000000000000003},
+        // Write data to be ORed with read value
+        {0x13030007, 0x0000040000000000},
+        {0x130F0012, 0x00000C0000000000},
 
 };
 const uint32_t g_xscomAddrTableSz =
@@ -46,6 +47,7 @@ public:
      */
     void testXscom1(void)
     {
+
         TARGETING::Target* l_testTarget = MASTER_PROCESSOR_CHIP_TARGET_SENTINEL;
         size_t l_size = sizeof(uint64_t);
 
@@ -56,9 +58,11 @@ public:
              testXscomAddrData l_testEntry = g_xscomAddrTable[l_num];
 
              // Perform XSComOM read
-             uint64_t l_data = 0;
+             uint64_t l_readData = 0;
+             uint64_t l_writeData = 0;
+             uint64_t l_savedData = 0;
              l_err = deviceRead(l_testTarget,
-                                &l_data,
+                                &l_readData,
                                 l_size,
                                 DEVICE_SCOM_ADDRESS(l_testEntry.addr));
              if (l_err)
@@ -66,10 +70,18 @@ public:
                  TS_FAIL("testXscom1: XSCom read: deviceRead() fails! Error committed.");
                  break;
              }
+             else
+             {
+                 TS_TRACE("testXscom1: XSCom read, Address 0x%.8X, Data %llx",
+                                                      l_testEntry.addr,
+                                                      (long long unsigned)l_readData);
+             }
 
              // Perform an XSCom write
+             l_savedData = l_readData;
+             l_writeData = (l_readData | l_testEntry.data);
              l_err = deviceWrite(l_testTarget,
-                     &l_testEntry.data,
+                     &l_writeData,
                      l_size,
                      DeviceFW::SCOM,
                      l_testEntry.addr);
@@ -79,11 +91,17 @@ public:
                  TS_FAIL("testXscom1: XSCom write: deviceWrite() fails!");
                  break;
              }
+             else
+             {
+                 TS_TRACE("testXscom1: XSCom write, Address 0x%.8X, Data %llx",
+                         l_testEntry.addr,
+                         (long long unsigned)l_writeData);
+             }
 
              // Read back
-             l_data = 0;
+             l_readData = 0;
              l_err = deviceRead(l_testTarget,
-                                &l_data,
+                                &l_readData,
                                 l_size,
                                 DEVICE_SCOM_ADDRESS(l_testEntry.addr));
              if (l_err)
@@ -91,6 +109,39 @@ public:
                  TS_FAIL("testXscom1: XSCom read back: deviceRead() fails!");
                  break;
              }
+
+             if( l_readData != l_writeData )
+             {
+                 TS_FAIL("testXscom1: XSCom read back doesn't match write!");
+                 /*@
+                  * @errortype
+                  * @moduleid     XSCOM_TEST_XSCOM1
+                  * @reasoncode   XSCOM_DATA_UNMATCHED
+                  * @userdata1    Write value
+                  * @userdata2    Read back value
+                  * @devdesc      Read back value doesn't match write
+                  */
+                 l_err = new ERRORLOG::ErrlEntry(
+                         ERRORLOG::ERRL_SEV_INFORMATIONAL,
+                         XSCOM::XSCOM_TEST_XSCOM1,
+                         XSCOM::XSCOM_DATA_UNMATCHED,
+                         l_writeData,
+                         l_readData);
+                 break;
+             }
+
+             // Write back original value
+             l_err = deviceWrite(l_testTarget,
+                      &l_savedData,
+                      l_size,
+                      DeviceFW::SCOM,
+                      l_testEntry.addr);
+
+              if (l_err)
+              {
+                  TS_FAIL("testXscom1: XSCom write back original fails!");
+                  break;
+              }
         }
 
         if (l_err)
diff --git a/src/usr/xscom/xscom.C b/src/usr/xscom/xscom.C
index 969b81db0..215d0e11b 100644
--- a/src/usr/xscom/xscom.C
+++ b/src/usr/xscom/xscom.C
@@ -181,6 +181,7 @@ errlHndl_t xscomPerformOp(DeviceFW::OperationType i_opType,
     errlHndl_t l_err = NULL;
     HMER l_hmer;
     mutex_t* l_XSComMutex;
+    uint64_t l_addr = va_arg(i_args,uint64_t);
 
     // Retry loop
     bool l_retry = false;
@@ -213,7 +214,7 @@ errlHndl_t xscomPerformOp(DeviceFW::OperationType i_opType,
         }
 
         // Build the XSCom address
-        XSComP8Address l_mmioAddr(va_arg(i_args,uint64_t), l_xscomChipInfo.nodeId,
+        XSComP8Address l_mmioAddr(l_addr, l_xscomChipInfo.nodeId,
                                   l_xscomChipInfo.chipId, l_XSComBaseAddr);
 
         // Re-init l_retry for loop
@@ -266,10 +267,15 @@ errlHndl_t xscomPerformOp(DeviceFW::OperationType i_opType,
 
         // Done, un-pin
         task_affinity_unpin();
-
-        TRACFCOMP(g_trac_xscom, "xscomPerformOp: OpType 0x%.8X, Address %llx, Page %llx; Offset %llx; VirtAddr %llx; l_virtAddr+l_offset %llx",
-                i_opType, static_cast<uint64_t>(l_mmioAddr), l_page,
-                l_offset_64, l_virtAddr, l_virtAddr + l_offset_64);
+        TRACFCOMP(g_trac_xscom, "xscomPerformOp: OpType %llx, Address 0%llx, MMIO Address %llx",
+                       static_cast<uint64_t>(i_opType),
+                       l_addr,
+                       static_cast<uint64_t>(l_mmioAddr));
+        TRACFCOMP(g_trac_xscom, "xscomPerformOp: Page %llx; Offset %llx; VirtAddr %llx; l_virtAddr+l_offset %llx",
+                       l_page,
+                       l_offset_64,
+                       l_virtAddr,
+                       l_virtAddr + l_offset_64);
 
         if (i_opType == DeviceFW::READ)
         {