[coreboot-gerrit] Change in coreboot[master]: nb/amd/amdk8: Link raminit_f.c

Arthur Heymans (Code Review) gerrit at coreboot.org
Tue Apr 11 17:37:41 CEST 2017


Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/19030 )

Change subject: nb/amd/amdk8: Link raminit_f.c
......................................................................


Patch Set 10:

(1 comment)

https://review.coreboot.org/#/c/19030/10/src/northbridge/amd/amdk8/raminit_f_dqs.c
File src/northbridge/amd/amdk8/raminit_f_dqs.c:

PS10, Line 21: #if IS_ENABLED(SOUTHBRIDGE_AMD_SB700)
Placed the guards here. Does that make more sense?


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I9cd1ffff2c39021693fe1d5d3f90ec5f70891f57
Gerrit-PatchSet: 10
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Arthur Heymans <arthur at aheymans.xyz>
Gerrit-Reviewer: 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



More information about the coreboot-gerrit mailing list