diff options
author | Furquan Shaikh <furquan@google.com> | 2019-05-30 17:24:12 -0700 |
---|---|---|
committer | Furquan Shaikh <furquan@google.com> | 2019-06-04 02:40:08 +0000 |
commit | b2709ae0aed724278dfaa3d0af1d68e8fe18cbb1 (patch) | |
tree | f3aae22acb5ada05563b01e68040822198090175 /src/superio/smsc/lpc47b397 | |
parent | ef1ab4d6d47c87d976c8186d5d8651853e3c92a9 (diff) |
soc/intel/cannonlake: Do not read SPD again if index hasn't changed
With the recent refactoring of memory configuration in
CB:32513 ("soc/intel/cannonlake: Support different SPD read type for
each slot"), meminit_cbfs_spd_index ends up reading SPD from CBFS for
each slot. However, for mainboards that use the same SPD index for
each slot this is unneccessary. This change adds a check to see if
spd_data_ptr is not NULL and current spd index is the same as the last
call to decide if SPD read from CBFS should be skipped.
TEST=Verified that SPD gets read only once on hatch.
Change-Id: I91963b55cea534c92207b2cd9f0caa96df8f222b
Signed-off-by: Furquan Shaikh <furquan@google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/33137
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Reviewed-by: Philip Chen <philipchen@google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/superio/smsc/lpc47b397')
0 files changed, 0 insertions, 0 deletions