Hello Peter Tsung Ho Wu, Stefan Reinauer, David Hendricks, build bot (Jenkins), Patrick Georgi, Martin Roth,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/em100/+/34917
to look at the new patch set (#2).
Change subject: em100: Add read_sdram() error check
......................................................................
em100: Add read_sdram() error check
Skip the memcpy and write_sdram if the readback fails.
Change-Id: Ibcdde9548d5ecd55ebecf7d824cf679a28e2322c
Signed-off-by: Jacob Garber jgarber1@ualberta.ca
Found-by: Coverity CID 205421
---
M em100.c
1 file changed, 6 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/em100 refs/changes/17/34917/2
--
To view, visit
https://review.coreboot.org/c/em100/+/34917
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: em100
Gerrit-Branch: master
Gerrit-Change-Id: Ibcdde9548d5ecd55ebecf7d824cf679a28e2322c
Gerrit-Change-Number: 34917
Gerrit-PatchSet: 2
Gerrit-Owner: Jacob Garber
jgarber1@ualberta.ca
Gerrit-Reviewer: David Hendricks
david.hendricks@gmail.com
Gerrit-Reviewer: Jacob Garber
jgarber1@ualberta.ca
Gerrit-Reviewer: Martin Roth
martinroth@google.com
Gerrit-Reviewer: Patrick Georgi
pgeorgi@google.com
Gerrit-Reviewer: Peter Tsung Ho Wu
tsung@amazon.com
Gerrit-Reviewer: Stefan Reinauer
stefan.reinauer@coreboot.org
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-MessageType: newpatchset