diff options
author | Stephen Glancy <sglancy@us.ibm.com> | 2018-02-16 10:12:00 -0600 |
---|---|---|
committer | William G. Hoffa <wghoffa@us.ibm.com> | 2018-02-28 11:47:17 -0500 |
commit | c03b84b934673984cde4aa532ea2b56503e9d720 (patch) | |
tree | 8233da2858b8da585ec334a28a6325d2b715d831 /src/import/chips/p9/procedures/hwp/memory/lib/mcbist | |
parent | 443282a786eeac0fe87525123adb707f4ed3c1d7 (diff) | |
download | talos-hostboot-c03b84b934673984cde4aa532ea2b56503e9d720.tar.gz talos-hostboot-c03b84b934673984cde4aa532ea2b56503e9d720.zip |
Fixes broadcast mode address check for deconfigured ports
Change-Id: Ie6a16b8d0a13880085f93c9dbc0c3f07c05ee167
CQ:SW414722
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/54234
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Dev-Ready: STEPHEN GLANCY <sglancy@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: ANDRE A. MARIN <aamarin@us.ibm.com>
Reviewed-by: ANUWAT SAETOW <asaetow@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/54254
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
Diffstat (limited to 'src/import/chips/p9/procedures/hwp/memory/lib/mcbist')
-rw-r--r-- | src/import/chips/p9/procedures/hwp/memory/lib/mcbist/memdiags.C | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/import/chips/p9/procedures/hwp/memory/lib/mcbist/memdiags.C b/src/import/chips/p9/procedures/hwp/memory/lib/mcbist/memdiags.C index 584f8c9ad..211da063e 100644 --- a/src/import/chips/p9/procedures/hwp/memory/lib/mcbist/memdiags.C +++ b/src/import/chips/p9/procedures/hwp/memory/lib/mcbist/memdiags.C @@ -375,8 +375,8 @@ fapi2::ReturnCode broadcast_mode_start_address_check_helper( // Checking that we are received a valid address (port_dimm) that is no less than the first configured port_dimm // on this MCBIST. This vector is sorted to make sure this is true. - FAPI_ASSERT( i_start_addr >= l_portdimm_this_dimm_min && - i_start_addr <= l_portdimm_this_dimm_max, + // Note: cronus always passes in a 0 address, so we need to support an address that is on or before this port + FAPI_ASSERT( i_start_addr <= l_portdimm_this_dimm_max, fapi2::MSS_MEMDIAGS_BCMODE_INVALID_ADDRESS() .set_MCA_TARGET(l_first_configured_mca) .set_START_ADDRESS(i_start_addr) |