[coreboot-gerrit] Change in coreboot[master]: amd/stoneyridge: Load AOAC and USB gnvs values

[Marshall Dawson (Code Review)]

Marshall Dawson has posted comments on this change. ( https://review.coreboot.org/28767 )

Change subject: amd/stoneyridge: Load AOAC and USB gnvs values
......................................................................


Patch Set 1:

(1 comment)

> Sorry, 0x71 is the state
Sorry, 0x5F is the state
Sorry, 0x77 is the state

I'm not certain what you're saying.  Do you think these are OK as-is?

https://review.coreboot.org/#/c/28767/1/src/soc/amd/stoneyridge/southbridge.c
File src/soc/amd/stoneyridge/southbridge.c:

https://review.coreboot.org/#/c/28767/1/src/soc/amd/stoneyridge/southbridge.c@914
PS1, Line 914: 1;
> Can you be sure it'll be enabled? ESPI is not the same as SPI. […]
No, not certain.  This will eventually be used for SD.  I'll update the source.



-- 
To view, visit https://review.coreboot.org/28767
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8af69c030eb2353ad75beeb2bfd3bef24abff04c
Gerrit-Change-Number: 28767
Gerrit-PatchSet: 1
Gerrit-Owner: Marshall Dawson <marshalldawson3rd at gmail.com>
Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd at gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply at coreboot.org>
Gerrit-CC: Richard Spiegel <richard.spiegel at silverbackltd.com>
Gerrit-Comment-Date: Thu, 27 Sep 2018 21:34:40 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20180927/9d5e0173/attachment.html>