[coreboot-gerrit] Change in coreboot[master]: device/dram/ddr2: Don't bail out on invalid SPD checksum

Arthur Heymans (Code Review) gerrit at coreboot.org
Sat Apr 8 12:25:46 CEST 2017


Hello build bot (Jenkins), Martin Roth,

I'd like you to reexamine a change.  Please visit

    https://review.coreboot.org/19182

to look at the new patch set (#2).

Change subject: device/dram/ddr2: Don't bail out on invalid SPD checksum
......................................................................

device/dram/ddr2: Don't bail out on invalid SPD checksum

One might want to pass an SPD data array with only bytes needed to
configure the memory controller (to speed up raminit). Such an
incomplete array will most likely have an invalid checksum (unless a
new one is computed).

Change-Id: I752d2ba6af667f26da5869aa803f7adb4d2ec211
Signed-off-by: Arthur Heymans <arthur at aheymans.xyz>
---
M src/device/dram/ddr2.c
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/82/19182/2
-- 
To view, visit https://review.coreboot.org/19182
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I752d2ba6af667f26da5869aa803f7adb4d2ec211
Gerrit-PatchSet: 2
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Arthur Heymans <arthur at aheymans.xyz>
Gerrit-Reviewer: Arthur Heymans <arthur at aheymans.xyz>
Gerrit-Reviewer: Martin Roth <martinroth at google.com>
Gerrit-Reviewer: Patrick Rudolph <siro at das-labor.org>
Gerrit-Reviewer: Paul Menzel <paulepanter at users.sourceforge.net>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer at coreboot.org>
Gerrit-Reviewer: build bot (Jenkins)



More information about the coreboot-gerrit mailing list