Skip to content

Commit

Permalink
Remove System Level TCE Attributes
Browse files Browse the repository at this point in the history
For Fleetwood support the TCE attributes refering to memory regions
had to move from the system level to the node level.  Now that the
Hostboot and FSP code no longer use these attributes at the system
level we can remove them and their usage.

Change-Id: I4f71b940787689d4ee9f3b3b6ad081dd4632c65b
RTC:190014
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/57680
Reviewed-by: ILYA SMIRNOV <ismirno@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Nicholas E. Bofferding <bofferdn@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>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
  • Loading branch information
mabaiocchi authored and dcrowell77 committed May 11, 2018
1 parent 89bbfaf commit 8e601e3
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 23 deletions.
3 changes: 0 additions & 3 deletions src/usr/targeting/common/xmltohb/target_types.xml
Original file line number Diff line number Diff line change
Expand Up @@ -957,9 +957,6 @@
<attribute><id>MAX_DMI_PER_PROC</id></attribute>
<attribute><id>PIB_I2C_REFCLOCK</id></attribute>
<attribute><id>USE_TCES_FOR_DMAS</id></attribute>
<attribute><id>TCE_START_TOKEN_FOR_PAYLOAD</id></attribute>
<attribute><id>START_MEM_ADDRESS_FOR_PAYLOAD_TCE_TOKEN</id></attribute>
<attribute><id>TCE_START_TOKEN_FOR_HDAT</id></attribute>
<!--fsp requirement-->
<attribute>
<id>BOOT_FREQ_MHZ</id>
Expand Down
20 changes: 0 additions & 20 deletions src/usr/util/utiltcemgr.C
Original file line number Diff line number Diff line change
Expand Up @@ -164,19 +164,6 @@ errlHndl_t utilSetupPayloadTces(void)
// opened for it
pNodeTgt->setAttr<TARGETING::ATTR_START_MEM_ADDRESS_FOR_PAYLOAD_TCE_TOKEN>(addr);

// Legacy Support: Also set System Level Attributes on Node 0
// @TODO RTC 190014 Can Be Removed Once FSP Switches To Node Attributes
// Get Target Service and the system target to set attributes
TARGETING::TargetService& tS = TARGETING::targetService();
TARGETING::Target* sys = nullptr;
(void) tS.getTopLevelTarget( sys );
assert(sys, "utilSetupPayloadTces() system target is NULL");
if (nodeId == 0)
{
sys->setAttr<TARGETING::ATTR_TCE_START_TOKEN_FOR_PAYLOAD>(token);
sys->setAttr<TARGETING::ATTR_START_MEM_ADDRESS_FOR_PAYLOAD_TCE_TOKEN>(addr);
}

// Save for internal use since can't trust FSP won't change attribute
Singleton<UtilTceMgr>::instance().setToken(UtilTceMgr::PAYLOAD_TOKEN,
token);
Expand Down Expand Up @@ -212,13 +199,6 @@ errlHndl_t utilSetupPayloadTces(void)
// Set attribute to tell FSP what the HDAT token is
pNodeTgt->setAttr<TARGETING::ATTR_TCE_START_TOKEN_FOR_HDAT>(token);

// Legacy Support: Also set System Level Attributes on Node 0
// @TODO RTC 190014 Can Be Removed Once FSP Switches To Node Attributes
if (nodeId == 0)
{
sys->setAttr<TARGETING::ATTR_TCE_START_TOKEN_FOR_HDAT>(token);
}

// Save for internal use since we can't trust FSP won't change the attribute
Singleton<UtilTceMgr>::instance().setToken(UtilTceMgr::HDAT_TOKEN,
token);
Expand Down

0 comments on commit 8e601e3

Please sign in to comment.