Add check to verify SBE .ring section size

    -Add option to p9_xip_tool to verify the size of the
     DD specific .rings section intended for the SBE image
    -Update makefiles to correctly run p9_ring_apply to insert
     and extract the DD level specific rings.
    -Update tor_access_ring to correctly advance the pointer
     to the next DD level in the HW image so the size can
     be extracted correctlty.

Change-Id: I21a53b5d11341a14a4e9cc88388c9c2050394095
RTC:165558
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/33873
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Joseph J. McGill <jmcgill@us.ibm.com>
Reviewed-by: Thi N. Tran <thi@us.ibm.com>
Reviewed-by: Sachin Gupta <sgupta2m@in.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/33879
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>

1 files changed, 8 insertions, 7 deletions

diff --git a/src/import/chips/p9/utils/imageProcs/p9_tor.C b/src/import/chips/p9/utils/imageProcs/p9_tor.C
index 2f79ce294..23912ac6f 100644
--- a/src/import/chips/p9/utils/imageProcs/p9_tor.C
+++ b/src/import/chips/p9/utils/imageProcs/p9_tor.C
@@ -1191,12 +1191,14 @@ int tor_access_ring(  void*           i_ringSection,     // Ring section ptr
             MY_INF("tor_access_ring(): No of DD levels: %d \n", ddLevelCount);
         }
 
-        for (uint8_t i = 0; i < ddLevelCount; i++)
+        // start at one since we use that as an offset
+        for (uint8_t i = 1; i <= ddLevelCount; i++)
         {
-            local = 2;
+            local = 2 * (i);
             ddLevelOffset  =  *((uint32_t*)i_ringSection + local);
-            ddLevel = htobe32(ddLevelOffset) >> 24 & 0x000000FF;
-            ddLevelOffset = htobe32(ddLevelOffset) & 0x00FFFFFF;
+
+            ddLevel = be32toh(ddLevelOffset) >> 24 & 0x000000FF;
+            ddLevelOffset = be32toh(ddLevelOffset) & 0x00FFFFFF;
 
             if (i_dbgl > 1)
             {
@@ -1206,12 +1208,11 @@ int tor_access_ring(  void*           i_ringSection,     // Ring section ptr
 
             if ( ddLevel == i_ddLevel)
             {
-                ddLevelOffset  =  *((uint32_t*)i_ringSection + local);
-                ddLevelOffset = htobe32(ddLevelOffset) & 0x00FFFFFF;
                 ddLevelOffset = ddLevelOffset + sizeof(TorNumDdLevels_t);
                 local = local + 1;
                 ddBlockSize = *((uint32_t*)i_ringSection + local);
-                ddBlockSize = htobe32(ddBlockSize);
+                ddBlockSize = be32toh(ddBlockSize);
+
                 bDdCheck = 1;
                 break;
             }