<p><a href="https://review.coreboot.org/28073">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/28073/1//COMMIT_MSG">Commit Message:</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/28073/1//COMMIT_MSG@9">Patch Set #1, Line 9:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">This means that we're<br>telling the OS that the cams need power in both D0 and D3 (and not the<br>intermediate states)<br></pre></blockquote></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">I don't think that's how OSPM to determine the power needed or not in D3 state. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">My understanding is that the _PR3 means the devices will have power in D3hot.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Prior to this change the device would have power in D0 and D3hot, but not D1, D2, or D3cold.  After this change only D0 will have power.  All the cam devices I can find outside of coreboot have a _PR0 but not a _PR3.  It seems more appropriate to remove the _PR3 than add a _PR2 (and probably also a "Name(_S0W, 4)" to make Windows think it's okay to transition the device to D3cold).</p></li></ul></li></ul><p>To view, visit <a href="https://review.coreboot.org/28073">change 28073</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/28073"/><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: Id52c2499c3b7577f03395cc9ca2460f25b80e13f </div>
<div style="display:none"> Gerrit-Change-Number: 28073 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Matt Delco <delco@chromium.org> </div>
<div style="display:none"> Gerrit-Reviewer: Lijian Zhao <lijian.zhao@intel.com> </div>
<div style="display:none"> Gerrit-Reviewer: Matt Delco <delco@chromium.org> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-CC: Lance Zhao <lance.zhao@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 14 Aug 2018 04:54:14 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>