[coreboot-gerrit] Change in coreboot[master]: nb/pineview/raminit: Don't do Jedec init on resume from S3

[Martin Roth (Code Review)]

Martin Roth has posted comments on this change. ( https://review.coreboot.org/19425 )

Change subject: nb/pineview/raminit: Don't do Jedec init on resume from S3
......................................................................


Patch Set 1:

(1 comment)

https://review.coreboot.org/#/c/19425/1/src/northbridge/intel/pineview/raminit.c
File src/northbridge/intel/pineview/raminit.c:

PS1, Line 2622: sdram_jedecinit(&si);
Just move it into the above if statement instead of checking for the same thing again?


-- 
To view, visit https://review.coreboot.org/19425
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id19a00c1546b7a71d90aa8c7e43e6efde1e9fbbc
Gerrit-PatchSet: 1
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Arthur Heymans <arthur at aheymans.xyz>
Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki at gmail.com>
Gerrit-Reviewer: Martin Roth <martinroth at google.com>
Gerrit-Reviewer: Paul Menzel <paulepanter at users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins)
Gerrit-HasComments: Yes