<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Again I don't think that statement is true, a value in _PR3 does not mean to <br>keep device power in D3Hot. For your reference, the following code <br>https://chromium.googlesource.com/chromiumos/third_party/kernel/+/stabilize-nocturne.10646.B-chromeos-4.14/drivers/acpi/power.c did explain that.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Is there a particular place that is supposed to illustrate this?  It looks to me like acpi_power_transition() calls acpi_power_on_list() on items in the list for the new power state and calls acpi_power_off_list() on items in old list.  So, having a _PR3 keeps the power on in D3Hot.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">May I know what kind of problem you are facing now?<br>The steps in _OFF had not been executed?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">The device (or at least the privacy light) never turns off in Windows.  If no driver is installed for the camera then the light stays on.  If I install a driver for the device then the light stays on.  If I install a driver and modify ACPI then the light does turn off.  I might be able to convince the light to go off by having the driver try harder to claim support for D3cold but this is going beyond the Intel reference examples and again I don't see any other MIPI camera outside of coreboot that uses _PR3.</p><p><a href="https://review.coreboot.org/28073">View Change</a></p><ul style="list-style: none; padding: 0;"></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 20:43:31 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>