Re: [RFC][PATCH] Add spurious wakeup quirk for Lynxpoint controllers

[Laura Abbott <labbott@redhat.com>]

On 09/18/2015 03:18 AM, Mathias Nyman wrote:
> On 10.09.2015 20:27, Laura Abbott wrote:
>>
>> We received several reports of systems rebooting and powering on
>> after an attempted shutdown. Testing showed that setting
>> XHCI_SPURIOUS_WAKEUP quirk in addition to the XHCI_SPURIOUS_REBOOT
>> quirk allowed the system to shutdown as expected for Lynxpoint
>> xHCI controllers. Set the qurik.
>>
>> Signed-off-by: Laura Abbott <labbott@fedoraproject.org>
>> ---
>
> We used to have the XHCI_SPURIOUS_WAKEUP flag set for lynxpoint controllers,
> but it was removed in commit:
>
> commit b45abacde3d551c6696c6738bef4a1805d0bf27a
>      xhci: no switching back on non-ULT Haswell
>      The switch back is limited to ULT even on HP. The contrary
>      finding arose by bad luck in BIOS versions for testing.
>      This fixes spontaneous resume from S3 on some HP laptops.
>
> Adding the SPURIOUS_WAKEUP flag back looks reasonable to me,
> but I don't want to break suspend.
> I don't understand how it could have caused spontaneous resume in HP laptops
> in the first place, it really shouldn't do anything before shutdown.
>
> Better ask Oliver,
> Do you still have access to the HP laptop?
> Any chance you could see if the flag still causes spontaneous resume?
>
> -Mathias

Would you rather see a revert of the patch you gave rather than a new
one re-introducing the flag?

Thanks,
Laura