Always leave A/XBUS chiplets enabled

Since the XBUS chiplet contains logic required for other functions
(specifically PSI), we need to ensure that the HWPs are told the
chiplet is enabled.  The bus itself will be powered down at the IO
level so there are no power concerns.

Change-Id: I19ebf75c6377901937114bc6188d0793a3e78cd1
CQ: SW228043
Reviewed-on: http://gfw160.austin.ibm.com:8080/gerrit/6697
Tested-by: Jenkins Server
Reviewed-by: Andrew J. Geissler <andrewg@us.ibm.com>
Reviewed-by: MIKE J. JONES <mjjones@us.ibm.com>
Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>

1 files changed, 8 insertions, 6 deletions

diff --git a/src/usr/hwpf/plat/fapiPlatAttributeService.C b/src/usr/hwpf/plat/fapiPlatAttributeService.C
index 638163ec5..af9aacc68 100644
--- a/src/usr/hwpf/plat/fapiPlatAttributeService.C
+++ b/src/usr/hwpf/plat/fapiPlatAttributeService.C
@@ -1321,14 +1321,16 @@ fapi::ReturnCode fapiPlatGetEnableAttr ( fapi::AttributeId i_id,
                 o_enable = l_buses.size() ? 1 : 0;
                 break;
             case fapi::ATTR_PROC_A_ENABLE:
-                // The enable flag is 1 if one of the abus target is functional
-                getChildChiplets( l_buses, l_pTarget, TARGETING::TYPE_ABUS );
-                o_enable = l_buses.size() ? 1 : 0;
+                // The enable flag reflects the state of the pervasive chiplet,
+                //  NOT the bus logic, so always return true since we don't
+                //  support partial good on the ABUS chiplet
+                o_enable = 1;
                 break;
             case fapi::ATTR_PROC_X_ENABLE:
-                // The enable flag is 1 if one of the xbus target is functioanl
-                getChildChiplets( l_buses, l_pTarget, TARGETING::TYPE_XBUS );
-                o_enable = l_buses.size() ? 1 : 0;
+                // The enable flag reflects the state of the pervasive chiplet,
+                //  NOT the bus logic, so always return true since we don't
+                //  support partial good on the XBUS chiplet
+                o_enable = 1;
                 break;
             default:
                 o_enable = 0;