Re: [PATCH v7 2/2] pinctrl: amd: Fix wakeups when IRQ is shared with SCI

From: Linus Walleij
Date: Mon Nov 08 2021 - 19:38:50 EST


On Mon, Nov 1, 2021 at 2:49 AM Mario Limonciello
<mario.limonciello@xxxxxxx> wrote:

> On some Lenovo AMD Gen2 platforms the IRQ for the SCI and pinctrl drivers
> are shared. Due to how the s2idle loop handling works, this case needs
> an extra explicit check whether the interrupt was caused by SCI or by
> the GPIO controller.
>
> To fix this rework the existing IRQ handler function to function as a
> checker and an IRQ handler depending on the calling arguments.
>
> BugLink: https://gitlab.freedesktop.org/drm/amd/-/issues/1738
> Reported-by: Joerie de Gram <j.de.gram@xxxxxxxxx>
> Signed-off-by: Mario Limonciello <mario.limonciello@xxxxxxx>
> Acked-by: Basavaraj Natikar <Basavaraj.Natikar@xxxxxxx>

This v7 patch applied for fixes.

Sorry for the delay, I was busy with the merge window.
Once the merge window concludes I will rebase this on
-rc1 and send upstream.

Yours,
Linus Walleij