diff options
author | Martin Peschke <mpeschke@de.ibm.com> | 2017-02-06 15:12:56 +0100 |
---|---|---|
committer | Sachin Gupta <sgupta2m@in.ibm.com> | 2017-02-15 09:17:53 -0500 |
commit | 4f78e01b13a675dba9a9e43cfaa956090ffffbbe (patch) | |
tree | 8229e2e4a68f85224089b15b8ce8d0aef95cc522 /src/import/chips/p9 | |
parent | d82dbcf6c22684793c3eedd4c0ed70b3ca507851 (diff) | |
download | talos-sbe-4f78e01b13a675dba9a9e43cfaa956090ffffbbe.tar.gz talos-sbe-4f78e01b13a675dba9a9e43cfaa956090ffffbbe.zip |
p9_xip_tool: cleanup - ring dissect output warnings
Usually we don't want debug messages.
Change-Id: I59333814751fdeeed6027945f47424af9dabecf0
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/33182
Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Martin Peschke <mpeschke@de.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/33308
Reviewed-by: Hostboot Team <hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Sachin Gupta <sgupta2m@in.ibm.com>
Diffstat (limited to 'src/import/chips/p9')
-rw-r--r-- | src/import/chips/p9/xip/p9_xip_tool.C | 35 |
1 files changed, 14 insertions, 21 deletions
diff --git a/src/import/chips/p9/xip/p9_xip_tool.C b/src/import/chips/p9/xip/p9_xip_tool.C index 534c3a82..d07db3e7 100644 --- a/src/import/chips/p9/xip/p9_xip_tool.C +++ b/src/import/chips/p9/xip/p9_xip_tool.C @@ -41,6 +41,8 @@ #include <string.h> #include <unistd.h> +#undef P9_XIP_TOOL_VERBOSE + #include "p9_xip_image.h" #ifdef XIP_TOOL_ENABLE_DISSECT // Needed on ppe side to avoid TOR API #include "p9_tor.H" @@ -1791,13 +1793,13 @@ int dissectRingSectionTor( void* i_ringSection, if (i_imageMagicNo == P9_XIP_MAGIC_HW) { numDdLevels = htobe32( ( (TorNumDdLevels_t*)i_ringSection )->TorNumDdLevels ); - fprintf( stdout, "numDdLevels (=%d) read from top of ring section.\n", numDdLevels); + fprintf( stderr, "numDdLevels (=%d) read from top of ring section.\n", numDdLevels); } else { numDdLevels = 1; ddLevel = 0xff; // This means it's unknown. - fprintf( stdout, "Image contains only one DD level set of rings.\n"); + fprintf( stderr, "Image contains only one DD level set of rings.\n"); } //---------------- @@ -1835,14 +1837,15 @@ int dissectRingSectionTor( void* i_ringSection, // - Only loop once if ringId is a common ring. for (instanceId = 0; instanceId <= CHIPLET_ID_MAX && ringType != COMMON; instanceId++) { - - fprintf( stdout, "Processing: " +#ifdef P9_XIP_TOOL_VERBOSE + fprintf( stderr, "Processing: " "DD=0x%02x " "PPE=%s " "Variant=%s " "RingID=%d " "InstanceID=0x%02x\n", ddLevel, ppeTypeName[ppeType], ringVariantName[ringVariant], ringId, instanceId); +#endif ringBlockSize = RING_BUF_SIZE_MAX; rc = tor_access_ring( i_ringSection, @@ -1979,25 +1982,15 @@ int dissectRingSectionTor( void* i_ringSection, sizeListMax = sizeListMax + sizeListIncr; disList = (char*)realloc( (void*)(disList), sizeListMax); } - - fprintf(stdout, "%s\n", disList); - - } - else if (rc == TOR_RING_NOT_FOUND) - { - fprintf(stdout, "tor_access_ring() returned rc=%d=TOR_RING_NOT_FOUND\n", rc); } - else if (rc == TOR_INVALID_INSTANCE_ID) + else if (rc == TOR_RING_NOT_FOUND || + rc == TOR_INVALID_INSTANCE_ID || + rc == TOR_AMBIGUOUS_API_PARMS || + rc == TOR_INVALID_RING_ID) { - fprintf(stdout, "tor_access_ring() returned rc=%d=TOR_INVALID_INSTANCE_ID\n", rc); - } - else if (rc == TOR_AMBIGUOUS_API_PARMS) - { - fprintf(stdout, "tor_access_ring() returned rc=%d=TOR_AMBIGUOUS_API_PARMS\n", rc); - } - else if (rc == TOR_INVALID_RING_ID) - { - fprintf(stdout, "tor_access_ring() returned rc=%d=TOR_INVALID_RING_ID\n", rc); +#ifdef P9_XIP_TOOL_VERBOSE + fprintf(stderr, "tor_access_ring() returned error code rc=%d\n", rc); +#endif } else { |