ProcessMRW changes for dynamic i2c devices

HDAT has a requirement from hostboot to provide I2C information
from the connections in the mrw.  This change pulls the changes
out of the mrw and stores them under the master proc target.
Each index in the arrays created is a separate connection.
The information is then pulled out in i2c.C getDeviceInfo()

Change-Id: I378ef6520dc6d32ef623fd438e73881ce928b37d
RTC:165485
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/42363
Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Reviewed-by: Prachi Gupta <pragupta@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Marshall J. Wilks <mjwilks@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>

5 files changed, 230 insertions, 24 deletions

diff --git a/src/usr/targeting/common/Targets.pm b/src/usr/targeting/common/Targets.pm
index a3ea5fa5f..1a41dde98 100644
--- a/src/usr/targeting/common/Targets.pm
+++ b/src/usr/targeting/common/Targets.pm
@@ -1808,6 +1808,18 @@ sub getBusAttribute
       ->{default};
 }
 
+## returns a boolean for if a given bus attribute is defined
+sub isBusAttributeDefined
+{
+    my $self       = shift;
+    my $target     = shift;
+    my $busnum     = shift;
+    my $attr       = shift;
+    my $target_ptr = $self->getTarget($target);
+
+    return defined($target_ptr->{CONNECTION}->{BUS}->[$busnum]->{bus_attribute}
+            ->{$attr}->{default});
+}
 
 ## returns a pointer to an array of children target names
 sub getTargetChildren
@@ -2151,6 +2163,10 @@ to value C<VALUE>.  This is for complex attributes.
 Gets the attribute C<ATTRIBUTE_NAME> from bus C<TARGET_STRING> bus number
 C<INDEX>.
 
+=item isBusAttributeDefined(C<TARGET_STRING>,C<INDEX>.C<ATTRIBUTE_NAME>)
+
+Looks for a specific attribute and returns if it exists or not
+
 =item getTargetChildren(C<TARGET_STRING>)
 
 Returns an array of target strings representing all the children of target
diff --git a/src/usr/targeting/common/processMrw.pl b/src/usr/targeting/common/processMrw.pl
index 40c483394..fee956c24 100755
--- a/src/usr/targeting/common/processMrw.pl
+++ b/src/usr/targeting/common/processMrw.pl
@@ -516,6 +516,15 @@ sub processProcessor
        $targetObj->setMasterProc($target);
     }
 
+    # I2C arrays
+    my @engine = ();
+    my @port = ();
+    my @slavePort = ();
+    my @addr = ();
+    my @speed = ();
+    my @type = ();
+    my @purpose = ();
+
     $targetObj->log($target, "Processing PROC");
     foreach my $child (@{ $targetObj->getTargetChildren($target) })
     {
@@ -572,8 +581,58 @@ sub processProcessor
         {
             processOcc($targetObj, $child, $target);
         }
+        # Ideally this should be $child_type eq "I2C", but we need a change
+        # in serverwiz and the witherspoon.xml first
+        elsif (index($child,"i2c-master") != -1)
+        {
+            my ($i2cEngine, $i2cPort, $i2cSlavePort, $i2cAddr,
+                $i2cSpeed, $i2cType, $i2cPurpose) =
+                    processI2C($targetObj, $child, $target);
+
+            # Add this I2C device's information to the proc array
+            push(@engine,@$i2cEngine);
+            push(@port,@$i2cPort);
+            push(@slavePort,@$i2cSlavePort);
+            push(@addr,@$i2cAddr);
+            push(@speed,@$i2cSpeed);
+            push(@type,@$i2cType);
+            push(@purpose,@$i2cPurpose);
+
+        }
     }
 
+    # Add final I2C arrays to processor
+    my $size         = scalar @engine;
+    my $engine_attr  = $engine[0];
+    my $port_attr    = $port[0];
+    my $slave_attr   = $slavePort[0];
+    my $addr_attr    = $addr[0];
+    my $speed_attr   = $speed[0];
+    my $type_attr    = $type[0];
+    my $purpose_attr = $purpose[0];
+
+    # Parse out array to print as a string
+    foreach my $n (1..($size-1))
+    {
+        $engine_attr    .= ",".$engine[$n];
+        $port_attr      .= ",".$port[$n];
+        $slave_attr     .= ",".$slavePort[$n];
+        $addr_attr      .= ",".$addr[$n];
+        $speed_attr     .= ",".$speed[$n];
+        $type_attr      .= ",".$type[$n];
+        $purpose_attr   .= ",".$purpose[$n];
+    }
+
+    # Set the arrays to the corresponding attribute on the proc
+    $targetObj->setAttribute($target,"HDAT_I2C_ENGINE",$engine_attr);
+    $targetObj->setAttribute($target,"HDAT_I2C_MASTER_PORT",$port_attr);
+    $targetObj->setAttribute($target,"HDAT_I2C_SLAVE_PORT",$slave_attr);
+    $targetObj->setAttribute($target,"HDAT_I2C_ADDR",$addr_attr);
+    $targetObj->setAttribute($target,"HDAT_I2C_BUS_FREQ",$speed_attr);
+    $targetObj->setAttribute($target,"HDAT_I2C_DEVICE_TYPE",$type_attr);
+    $targetObj->setAttribute($target,"HDAT_I2C_DEVICE_PURPOSE",$purpose_attr);
+    $targetObj->setAttribute($target,"HDAT_I2C_ELEMENTS",$size);
+
     ## update path for mvpd's and sbe's
     my $path  = $targetObj->getAttribute($target, "PHYS_PATH");
     my $model = $targetObj->getAttribute($target, "MODEL");
@@ -1540,6 +1599,137 @@ sub getI2cMapField
     return $hexfield;
 }
 
+#------------------------------------------------------------------------------
+# I2C
+#
+sub processI2C
+{
+    my $targetObj    = shift; # Top Hierarchy of targeting structure
+    my $target       = shift; # I2C targetInstance
+    my $parentTarget = shift; # Processor target
+
+    # Initialize output arrays
+    my @i2cEngine = ();
+    my @i2cPort = ();
+    my @i2cSlave = ();
+    my @i2cAddr = ();
+    my @i2cSpeed = ();
+    my @i2cType = ();
+    my @i2cPurpose = ();
+
+    # Step 1: get I2C_ENGINE and PORT from <targetInstance>
+
+    my $engine = $targetObj->getAttribute($target, "I2C_ENGINE");
+    if($engine eq "") {$engine = "0xFF";}
+
+    my $port = $targetObj->getAttribute($target, "I2C_PORT");
+    if($port eq "") {$port = "0xFF";}
+
+    # Step 2: get I2C_ADDRESS and I2C_SPEED from <bus>
+    #         This is different for each connection.
+
+    my $i2cs = $targetObj->findConnections($parentTarget, "I2C","");
+    if ($i2cs ne "")
+    {
+        # This gives all i2c connections
+        foreach my $i2c (@{$i2cs->{CONN}})
+        {
+            # Here we are checking that the i2c source matches our target
+            my $source = $i2c->{SOURCE};
+            if ($source ne $target)
+            {
+                next;
+            }
+
+            # Most I2C devices will default the slave port, it is only valid
+            # for gpio expanders.
+            my $slavePort = "0xFF";
+
+            my @source_array = split(/-/,$source);
+            my $source_idx = scalar @source_array;
+
+            # If the last part of the source only includes numbers
+            if($source_array[$source_idx-1] =~ /^[0-9,.E]+$/)
+            {
+                $slavePort = $source_array[$source_idx-1];
+            }
+
+            my $addr;
+            my $speed;
+            my $type;
+            my $purpose;
+
+            # For all these attributes, we need to check if they're defined,
+            # and if not we set them to a default value.
+            if ($targetObj->isBusAttributeDefined(
+                     $i2c->{SOURCE},$i2c->{BUS_NUM},"I2C_ADDRESS"))
+            {
+                $addr = $targetObj->getBusAttribute(
+                           $i2c->{SOURCE},$i2c->{BUS_NUM},"I2C_ADDRESS");
+            }
+
+            if ($addr eq "") {$addr = "0xFF";}
+
+            if ($targetObj->isBusAttributeDefined(
+                     $i2c->{SOURCE},$i2c->{BUS_NUM},"I2C_SPEED"))
+            {
+                $speed = $targetObj->getBusAttribute(
+                           $i2c->{SOURCE},$i2c->{BUS_NUM},"I2C_SPEED");
+            }
+
+            if ($speed eq "") {$speed = "0";}
+
+            if ($targetObj->isBusAttributeDefined(
+                     $i2c->{SOURCE},$i2c->{BUS_NUM},"I2C_TYPE"))
+            {
+                $type = $targetObj->getBusAttribute(
+                           $i2c->{SOURCE},$i2c->{BUS_NUM},"I2C_TYPE");
+            }
+
+            if ($type eq "")
+            {
+                $type = "0xFF";
+            }
+            else
+            {
+                $type = $targetObj->getEnumValue("HDAT_I2C_DEVICE_TYPE",$type);
+            }
+
+            if ($targetObj->isBusAttributeDefined(
+                     $i2c->{SOURCE},$i2c->{BUS_NUM},"I2C_PURPOSE"))
+            {
+                $purpose = $targetObj->getBusAttribute(
+                           $i2c->{SOURCE},$i2c->{BUS_NUM},"I2C_PURPOSE");
+            }
+
+            if ($purpose eq "")
+            {
+                $purpose = "0xFF";
+            }
+            else
+            {
+                $purpose = $targetObj->getEnumValue("HDAT_I2C_DEVICE_PURPOSE",
+                                                    $purpose);
+            }
+
+            # Step 3: For each connection, create an instance in the array
+            #         for the DeviceInfo_t struct.
+            push @i2cEngine, $engine;
+            push @i2cPort, $port;
+            push @i2cSlave, $slavePort;
+            push @i2cAddr, $addr;
+            push @i2cSpeed, $speed;
+            push @i2cType, $type;
+            push @i2cPurpose, $purpose;
+
+        }
+    }
+
+    # Return this i2c device's information back to the processor
+    return (\@i2cEngine, \@i2cPort, \@i2cSlave, \@i2cAddr,
+            \@i2cSpeed, \@i2cType, \@i2cPurpose);
+}
+
 
 sub setEepromAttributes
 {
diff --git a/src/usr/targeting/common/xmltohb/attribute_types.xml b/src/usr/targeting/common/xmltohb/attribute_types.xml
index c66d002d6..da75d4697 100644
--- a/src/usr/targeting/common/xmltohb/attribute_types.xml
+++ b/src/usr/targeting/common/xmltohb/attribute_types.xml
@@ -34429,31 +34429,31 @@ Measured in GB</description>
     </description>
     <enumerator>
         <name>955X</name>
-        <value>1</value>
+        <value>0x1</value>
     </enumerator>
     <enumerator>
         <name>SEEPROM</name>
-        <value>2</value>
+        <value>0x2</value>
     </enumerator>
     <enumerator>
         <name>NUVOTON_TPM</name>
-        <value>3</value>
+        <value>0x3</value>
     </enumerator>
     <enumerator>
         <name>MEX_FPGA</name>
-        <value>4</value>
+        <value>0x4</value>
     </enumerator>
     <enumerator>
         <name>UCX90XX</name>
-        <value>5</value>
+        <value>0x5</value>
     </enumerator>
     <enumerator>
         <name>NVLINK</name>
-        <value>6</value>
+        <value>0x6</value>
     </enumerator>
     <enumerator>
         <name>UNKNOWN</name>
-        <value>FF</value>
+        <value>0xFF</value>
     </enumerator>
 </enumerationType>
 
@@ -34464,47 +34464,47 @@ Measured in GB</description>
     </description>
     <enumerator>
         <name>CABLE_CARD_PRES</name>
-        <value>1</value>
+        <value>0x1</value>
     </enumerator>
     <enumerator>
         <name>CABLE_CARD_POWER_SENSE</name>
-        <value>2</value>
+        <value>0x2</value>
     </enumerator>
     <enumerator>
         <name>CABLE_CARD_POWER_CONTROL</name>
-        <value>3</value>
+        <value>0x3</value>
     </enumerator>
     <enumerator>
         <name>TPM</name>
-        <value>4</value>
+        <value>0x4</value>
     </enumerator>
     <enumerator>
         <name>MODULE_VPD</name>
-        <value>5</value>
+        <value>0x5</value>
     </enumerator>
     <enumerator>
         <name>DIMM_SPD</name>
-        <value>6</value>
+        <value>0x6</value>
     </enumerator>
     <enumerator>
         <name>PROC_MODULE_VPD</name>
-        <value>7</value>
+        <value>0x7</value>
     </enumerator>
     <enumerator>
         <name>SBE_SEEPROM</name>
-        <value>8</value>
+        <value>0x8</value>
     </enumerator>
     <enumerator>
         <name>PLANAR_VPD</name>
-        <value>9</value>
+        <value>0x9</value>
     </enumerator>
     <enumerator>
         <name>PCI_HOTPLUG</name>
-        <value>A</value>
+        <value>0xA</value>
     </enumerator>
     <enumerator>
         <name>NVLINK</name>
-        <value>B</value>
+        <value>0xB</value>
     </enumerator>
     <enumerator>
         <name>WINDOW_OPEN</name>
@@ -34516,7 +34516,7 @@ Measured in GB</description>
     </enumerator>
     <enumerator>
         <name>UNKNOWN</name>
-        <value>FF</value>
+        <value>0xFF</value>
     </enumerator>
 </enumerationType>
 
diff --git a/src/usr/targeting/common/xmltohb/attribute_types_openpower.xml b/src/usr/targeting/common/xmltohb/attribute_types_openpower.xml
index 1e705f111..a7a476588 100644
--- a/src/usr/targeting/common/xmltohb/attribute_types_openpower.xml
+++ b/src/usr/targeting/common/xmltohb/attribute_types_openpower.xml
@@ -191,7 +191,7 @@
 </attribute>
 
 <attribute>
-    <id>HDAT_I2C_DEVTYPE</id>
+    <id>HDAT_I2C_DEVICE_TYPE</id>
     <description>
         This attribute holds the values of the I2C device type from the i2c
         device connections as defined in the MRW.  It is parsed into a
@@ -255,7 +255,7 @@
 </attribute>
 
 <attribute>
-    <id>HDAT_I2C_DEV_PURPOSE</id>
+    <id>HDAT_I2C_DEVICE_PURPOSE</id>
     <description>
         This attribute holds the values of the I2C device purpose from the i2c
         device connections as defined in the MRW.  It is parsed into a
diff --git a/src/usr/targeting/common/xmltohb/target_types_openpower.xml b/src/usr/targeting/common/xmltohb/target_types_openpower.xml
index f85eb2b0b..e56f7ab00 100644
--- a/src/usr/targeting/common/xmltohb/target_types_openpower.xml
+++ b/src/usr/targeting/common/xmltohb/target_types_openpower.xml
@@ -32,14 +32,14 @@
      =====================================================================
  -->
 <targetTypeExtension>
-    <id>base</id>
+    <id>chip-processor</id>
     <attribute><id>HDAT_I2C_ENGINE</id></attribute>
     <attribute><id>HDAT_I2C_MASTER_PORT</id></attribute>
-    <attribute><id>HDAT_I2C_DEVTYPE</id></attribute>
+    <attribute><id>HDAT_I2C_DEVICE_TYPE</id></attribute>
     <attribute><id>HDAT_I2C_ADDR</id></attribute>
     <attribute><id>HDAT_I2C_SLAVE_PORT</id></attribute>
     <attribute><id>HDAT_I2C_BUS_FREQ</id></attribute>
-    <attribute><id>HDAT_I2C_DEV_PURPOSE</id></attribute>
+    <attribute><id>HDAT_I2C_DEVICE_PURPOSE</id></attribute>
     <attribute><id>HDAT_I2C_ELEMENTS</id></attribute>
     <attribute>
         <id>IPMI_INSTANCE</id>