Re: [PATCH] pinctrl: amd: Mask wake bits on probe again

[Linus Walleij]

On Fri, Aug 18, 2023 at 4:49 PM Mario Limonciello
<mario.limonciello@xxxxxxx> wrote:

> Shubhra reports that their laptop is heating up over s2idle. Even though
> it's getting into the deepest state, it appears to be having spurious
> wakeup events.
>
> While debugging a tangential issue with the RTC Carsten reports that recent
> 6.1.y based kernel face a similar problem.
>
> Looking at acpidump and GPIO register comparisons these spurious wakeup
> events are from the GPIO associated with the I2C touchpad on both laptops
> and occur even when the touchpad is not marked as a wake source by the
> kernel.
>
> This means that the boot firmware has programmed these bits and because
> Linux didn't touch them lead to spurious wakeup events from that GPIO.
>
> To fix this issue, restore most of the code that previously would clear all
> the bits associated with wakeup sources. This will allow the kernel to only
> program the wake up sources that are necessary.
>
> This is similar to what was done previously; but only the wake bits are
> cleared by default instead of interrupts and wake bits.  If any other
> problems are reported then it may make sense to clear interrupts again too.
>
> Cc: Sachi King <nakato@xxxxxxxxx>
> Cc: stable@xxxxxxxxxxxxxxx
> Cc: Thorsten Leemhuis <regressions@xxxxxxxxxxxxx>
> Fixes: 65f6c7c91cb2 ("pinctrl: amd: Revert "pinctrl: amd: disable and mask interrupts on probe"")
> Reported-by: "Shubhra Prakash Nandi" <email2shubhra@xxxxxxxxx>
> Closes: https://bugzilla.kernel.org/show_bug.cgi?id=217754
> Reported-by: "Carsten Hatger" <xmb8dsv4@xxxxxxxxx>
> Link: https://bugzilla.kernel.org/show_bug.cgi?id=217626#c28
> Signed-off-by: Mario Limonciello <mario.limonciello@xxxxxxx>

Patch applied for fixes so we get some rotation in linux-next.

Thanks for looking into this Mario!

Yours,
Linus Walleij