Attention is currently required from: Eric Lai, Felix Held, Fred Reitberger, Jason Glenesk, Konrad Adamczyk, Matt DeVillier, Raul Rangel.
Hello Eric Lai, Felix Held, Fred Reitberger, Jason Glenesk, Matt DeVillier, Raul Rangel, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/76108?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed: Code-Review+2 by Eric Lai, Code-Review+2 by Fred Reitberger, Verified+1 by build bot (Jenkins)
The change is no longer submittable: Code-Review and Verified are unsatisfied now.
Change subject: soc/amd/common/fsp: Fix dimm_num assignment ......................................................................
soc/amd/common/fsp: Fix dimm_num assignment
The dimm_num shall be dimm, not channel.
BUG=b:288520486 TEST=In kernel, see output from `dmidecode -t 17`. Observe that Locator reflects proper location of the module.
Change-Id: Id876a5c245ed1a145c930b3456830d7b42780b74 Signed-off-by: Konrad Adamczyk konrada@google.com --- M src/soc/amd/common/fsp/dmi.c 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/08/76108/2