Key Clear: Fix Hostboot GPIO processing on multi-node systems

This commit fixes a bug where hostboot code was not properly setting
up ATTR_GPIO_INFO_PHYS_PRES for fleetwood systems which can have
different primary processors on multiple nodes.  Now, each processor
is given a default setting that works when it is the active primary
processor on that node.

Change-Id: I75923dda95f8185464696f295a594fa2d5e71510
CQ:SW495867
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/100311
Reviewed-by: Nicholas E Bofferding <bofferdn@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Christopher J Engel <cjengel@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Reviewed-by: Ilya Smirnov <ismirno@us.ibm.com>
Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>

Author: mabaiocchi

src/usr/gpio/gpiodd.C

@@ -5,7 +5,7 @@
 /*                                                                        */
 /* OpenPOWER HostBoot Project                                             */
 /*                                                                        */
-/* Contributors Listed Below - COPYRIGHT 2014,2019                        */
+/* Contributors Listed Below - COPYRIGHT 2014,2020                        */
 /* [+] Google Inc.                                                        */
 /* [+] International Business Machines Corp.                              */
 /*                                                                        */
@@ -89,14 +89,16 @@ errlHndl_t gpioPerformOp(DeviceFW::OperationType i_opType,
             break;
         }
 
+        char * path_str = gpioInfo.i2cMasterPath.toString();
+
         TARGETING::TargetService& ts = TARGETING::targetService();
         TARGETING::Target * i2c_master = ts.toTarget(gpioInfo.i2cMasterPath);
 
         if( i2c_master == NULL )
         {
             TRACFCOMP( g_trac_gpio,ERR_MRK"gpioPerformOp() - "
-                       "I2C Target not found. Device type %d.",
-                       gpioInfo.deviceType );
+                       "I2C Target not found. Device type %d. entity path=%s",
+                       gpioInfo.deviceType, path_str );
             /*@
              * @errortype
              * @reasoncode       GPIO_I2C_TARGET_NOT_FOUND
@@ -116,8 +118,19 @@ errlHndl_t gpioPerformOp(DeviceFW::OperationType i_opType,
                                           true /*Add HB SW Callout*/ );
 
             err->collectTrace( GPIO_COMP_NAME );
+            ERRORLOG::ErrlUserDetailsString(path_str).addToLog(err);
+            free(path_str);
+            path_str=nullptr;
             break;
         }
+        else
+        {
+            TRACDCOMP( g_trac_gpio,INFO_MRK"gpioPerformOp() - "
+                       "I2C Target entity path=%s",
+                       path_str );
+        }
+        free(path_str);
+        path_str = nullptr;
 
         if( i_opType == DeviceFW::READ )
         {

src/usr/targeting/common/processMrw.pl

@@ -1271,6 +1271,27 @@ sub processProcessor
                      "PROC_MEM_TO_USE", ( $targetObj->getAttribute($target,
                      "FABRIC_GROUP_ID") << 3));
     processPowerRails ($targetObj, $target);
+
+    # Set GPIO_INFO_PHYS_PRES for each proc to the same default even though
+    # it's only valid on the primary and alt-primary proc.
+    # It will only get called on the active primary processor during the IPL.
+
+    $targetObj->setAttributeField($target,"GPIO_INFO_PHYS_PRES",
+        "i2cMasterPath",$path);
+    $targetObj->setAttributeField($target,"GPIO_INFO_PHYS_PRES",
+        "port","0");
+    $targetObj->setAttributeField($target,"GPIO_INFO_PHYS_PRES",
+        "devAddr","0xC0");
+    $targetObj->setAttributeField($target,"GPIO_INFO_PHYS_PRES",
+        "engine","2");
+    $targetObj->setAttributeField($target,"GPIO_INFO_PHYS_PRES",
+        "windowOpenPin","0");
+    $targetObj->setAttributeField($target,"GPIO_INFO_PHYS_PRES",
+        "physicalPresencePin","1");
+    $targetObj->setAttributeField($target,"GPIO_INFO_PHYS_PRES",
+        "i2cMuxBusSelector","0xFF");
+    $targetObj->setAttributeField($target,"GPIO_INFO_PHYS_PRES",
+        "i2cMuxPath","physical:sys-0");
 }
 
 sub processPowerRails