pinctrl: amd: Revert "pinctrl: amd: disable and mask interrupts on probe"
authorMario Limonciello <mario.limonciello@amd.com>
Fri, 21 Apr 2023 12:06:24 +0000 (07:06 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 23 Jul 2023 11:49:30 +0000 (13:49 +0200)
commit57f6d48af46f0ffd23ef0dc463c8625d8855f657
tree1eee5e509626dfd6305ebd005bb56f7dc8aac669
parent1cd1a0151fdeab2683125ef98d313eaf7878b178
pinctrl: amd: Revert "pinctrl: amd: disable and mask interrupts on probe"

commit 65f6c7c91cb2ebacbf155e0f881f81e79f90d138 upstream.

commit 4e5a04be88fe ("pinctrl: amd: disable and mask interrupts on probe")
was well intentioned to mask a firmware issue on a surface laptop, but it
has a few problems:
1. It had a bug in the loop handling for iteration 63 that lead to other
   problems with GPIO0 handling.
2. It disables interrupts that are used internally by the SOC but masked
   by default.
3. It masked a real firmware problem in some chromebooks that should have
   been caught during development but wasn't.

There has been a lot of other development around s2idle; particularly
around handling of the spurious wakeups.  If there is still a problem on
the original reported surface laptop it should be avoided by adding a quirk
to gpiolib-acpi for that system instead.

Signed-off-by: Mario Limonciello <mario.limonciello@amd.com>
Link: https://lore.kernel.org/r/20230421120625.3366-5-mario.limonciello@amd.com
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/pinctrl/pinctrl-amd.c