Skip to content

Commit

Permalink
Only update SBEs on chips connected to powerbus
Browse files Browse the repository at this point in the history
  - Fixes issue in istep 7 where HB was attempting
    to update the slave chip's SBE, as SBE update
    expects SBE PSU interface to be active

Change-Id: Id90f1b0f23c668a153a7e2725e762c761ee3c490
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/36792
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Martin Gloff <mgloff@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
  • Loading branch information
sannerd authored and dcrowell77 committed Feb 21, 2017
1 parent bc4bd5e commit 988d8d5
Showing 1 changed file with 18 additions and 0 deletions.
18 changes: 18 additions & 0 deletions src/usr/sbe/sbe_update.C
Expand Up @@ -299,6 +299,24 @@ namespace SBE
}
}

//Can only update the SBE once the powerbus is up (secureboot)
//Use the scom switch Xscom capability flag as a proxy for
//powerbus access. If we can't access via powerbus, then skip
TARGETING::ScomSwitches scomSetting =
sbeState.target->getAttr<TARGETING::ATTR_SCOM_SWITCHES>();

if(!(scomSetting.useXscom))
{
//Xscom is not viable on this chip, thus powerbus isn't
//up to chip -- skip it
TRACFCOMP( g_trac_sbe,
INFO_MRK"updateProcessorSbeSeeproms(): "
"Power bus not established to chip 0x%X,"
" not performing update",
TARGETING::get_huid(sbeState.target));
continue;
}

err = getSbeInfoState(sbeState);

if (err)
Expand Down

0 comments on commit 988d8d5

Please sign in to comment.