Enabling FSI to remote Centaurs (SW107421 is fixed)

Change-Id: I2eaef856d94a4d84db729e1f9d1e9b5b0f8d0845
Reviewed-on: http://gfw160.austin.ibm.com:8080/gerrit/496
Tested-by: Jenkins Server
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>

1 files changed, 4 insertions, 6 deletions

diff --git a/src/usr/scom/test/scomtest.H b/src/usr/scom/test/scomtest.H
index f08fd7851..c9069acf7 100644
--- a/src/usr/scom/test/scomtest.H
+++ b/src/usr/scom/test/scomtest.H
@@ -96,7 +96,6 @@ public:
           { 0x120F0166, 0xFEDCBA9876543210}, 
           { 0x00040005, 0x0000000000000000}, 
           { 0x02040004, 0xFFFFFFFFFFFFFFFF}, 
-
       };
       const uint64_t NUM_ADDRS = sizeof(test_data)/sizeof(test_data[0]);
 
@@ -160,9 +159,7 @@ public:
    */
   void test_FSISCOMreadWrite_centaur(void)
   {
-
       TRACFCOMP( g_trac_scom, "ScomTest::test_FSISCOMreadWrite_centaur> Start" );
-
       uint64_t fails = 0;
       uint64_t total = 0;
       errlHndl_t l_err = NULL;
@@ -203,10 +200,9 @@ public:
           uint64_t data;
       } test_data[] = {
           { 0x00012345 , 0x1111222233334444 }, 
-          /* Simics is chopping off the upper nibble... SW108655
-          { 0x02011403 , 0x123456789ABCDEF0 }, 
+          //@fixme - should be 0x02011403 but simics is adding parity...
+          { 0x02011402 , 0x123456789ABCDEF0 }, 
           { 0x02011672 , 0x1122334455667788 },
-          */
 
       };
       const uint64_t NUM_ADDRS = sizeof(test_data)/sizeof(test_data[0]);
@@ -268,6 +264,8 @@ public:
 
   //@todo - write tests to verify connection between XSCOM and FSISCOM
 
+  //@todo - write error path testcase for FSI scom using bad address 
+
 };
 
 #endif