<p><a href="https://review.coreboot.org/28767">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/28767/1/src/soc/amd/stoneyridge/southbridge.c">File src/soc/amd/stoneyridge/southbridge.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/28767/1/src/soc/amd/stoneyridge/southbridge.c@914">Patch Set #1, Line 914:</a> <code style="font-family:monospace,monospace">1;</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">No, not certain.  This will eventually be used for SD.  I'll update the source.</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Search for "AOACx[7F:41:step2] Device D3 State" in the NDA BKDG (probably also available in the public one), and you'll understand the values of 0x5F, 0x71 and 0x77. The even offsets is for D3 control, the odd ones for D3 state. And yes, procedure is_aoac_device_enabled() was created to use specifically these offsets when checking if a device is enabled (D0) or disabled (D3).</p></li></ul></li></ul><p>To view, visit <a href="https://review.coreboot.org/28767">change 28767</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/28767"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I8af69c030eb2353ad75beeb2bfd3bef24abff04c </div>
<div style="display:none"> Gerrit-Change-Number: 28767 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Marshall Dawson <marshalldawson3rd@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-CC: Richard Spiegel <richard.spiegel@silverbackltd.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 27 Sep 2018 21:58:28 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>