From ffdc6b255981c8b52e3f5513425ebbaa8dfeb675 Mon Sep 17 00:00:00 2001 From: Mike Baiocchi Date: Mon, 25 Mar 2019 23:15:24 -0500 Subject: Add check to determine if a UCD's I2C Master is functional Inside the code that loops over all the supported UCD devices for possibly updating their flash image, a check was added to make sure that the UCD's I2C Master target is functional. Change-Id: Id03bf20f0295a27f67f9da8da207934af710630f CQ:SW460918 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/75002 Tested-by: Jenkins Server Tested-by: FSP CI Jenkins Reviewed-by: Matthew Raybuck Reviewed-by: Ilya Smirnov Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Reviewed-by: Nicholas E. Bofferding --- src/usr/isteps/ucd/updateUcdFlash.C | 15 +++++++++++++++ 1 file changed, 15 insertions(+) (limited to 'src/usr/isteps/ucd') diff --git a/src/usr/isteps/ucd/updateUcdFlash.C b/src/usr/isteps/ucd/updateUcdFlash.C index ce6eff490..d0e746c18 100644 --- a/src/usr/isteps/ucd/updateUcdFlash.C +++ b/src/usr/isteps/ucd/updateUcdFlash.C @@ -679,6 +679,21 @@ errlHndl_t updateAllUcdFlashImages( "with HUID of 0x%08X", TARGETING::get_huid(powerSequencer)); + // Check that pI2cMasterTarget is functional because otherwise the UCD + // can't be accessed + TARGETING::HwasState l_hwasState = + pI2cMasterTarget->getAttr(); + if (l_hwasState.functional != true) + { + TRACFCOMP(g_trac_ucd, INFO_MRK + "updateAllUcdFlashImages: Skipping update on power sequencer HUID " + "= 0x%08X because its I2C Master HUID = 0x%08X is non-functional", + TARGETING::get_huid(powerSequencer), + TARGETING::get_huid(pI2cMasterTarget)); + + break; + } + const auto position = pI2cMasterTarget-> getAttr(); -- cgit v1.2.1