Skip to content

Commit

Permalink
Fix ddimm_get_efd for EFDs that support multiple ranks
Browse files Browse the repository at this point in the history
Change-Id: Iffbd33faa221d1012f4b79475693734826b1c5dc
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/79964
Reviewed-by: Mark Pizzutillo <mark.pizzutillo@ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@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: STEPHEN GLANCY <sglancy@us.ibm.com>
Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com>
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/79984
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: Christian R. Geddes <crgeddes@us.ibm.com>
  • Loading branch information
stermole authored and crgeddes committed Jul 10, 2019
1 parent 1b5a0bd commit 4cedc84
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/import/chips/p9/procedures/hwp/accessors/ddimm_get_efd.C
Original file line number Diff line number Diff line change
Expand Up @@ -834,12 +834,12 @@ extern "C"

// If the 'is implemented flag' is true for the EFD, AND if the EFD
// frequency mask contains the frequency mask we are looking for AND
// the EFD master rank matches the master rank we are looking for
// the EFD master rank bitmap includes the master rank we are looking for
// then copy the EFD block for the caller.
if ( (l_efdMetaDataNptr[SPD_EFD_META_DATA_EFD_BYTE_3_OFFSET] &
SPD_EFD_META_DATA_EFD_IS_IMPLEMENTED_MASK) &&
(l_efdFreqMask & l_freqMask) &&
(l_efdDataNptr[EFD_DDR4_MASTER_RANK_ADDR] == l_rankMask) )
(l_efdDataNptr[EFD_DDR4_MASTER_RANK_ADDR] & l_rankMask) )
{
// io_vpdInfo.iv_size and EFD block size compatibility
// have been verified above
Expand Down

0 comments on commit 4cedc84

Please sign in to comment.