[coreboot-gerrit] Change in coreboot[master]: device/dram/ddr2.c: Improve error returning and debug output

Arthur Heymans (Code Review) gerrit at coreboot.org
Sun Sep 10 21:09:19 CEST 2017


Hello Patrick Rudolph, build bot (Jenkins), Nico Huber, 

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

    https://review.coreboot.org/21480

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

Change subject: device/dram/ddr2.c: Improve error returning and debug output
......................................................................

device/dram/ddr2.c: Improve error returning and debug output

This patch outputs decoding errors with BIOS_WARNING instead of
depending on CONFIG_DEBUG_RAM_SETUP.

Returns SPD_STATUS_INVALID on invalid settings for tRR, bcd and tCK.

Change-Id: Iee434d1fa1a9d911cc3683b88b260881ed6434ea
Signed-off-by: Arthur Heymans <arthur at aheymans.xyz>
---
M src/device/dram/ddr2.c
1 file changed, 88 insertions(+), 36 deletions(-)


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

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iee434d1fa1a9d911cc3683b88b260881ed6434ea
Gerrit-Change-Number: 21480
Gerrit-PatchSet: 2
Gerrit-Owner: Arthur Heymans <arthur at aheymans.xyz>
Gerrit-Reviewer: Nico Huber <nico.h at gmx.de>
Gerrit-Reviewer: Patrick Rudolph <siro at das-labor.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply at coreboot.org>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20170910/d2a85f52/attachment.html>


More information about the coreboot-gerrit mailing list