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

[Mathias Nyman]

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@xxxxxxxxxxxxxxxxx>
> ---

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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/