[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20230829165627.156542-4-mario.limonciello@amd.com>
Date: Tue, 29 Aug 2023 11:56:27 -0500
From: Mario Limonciello <mario.limonciello@....com>
To: <linus.walleij@...aro.org>
CC: <Shyam-sundar.S-k@....com>, <Basavaraj.Natikar@....com>,
<linux-gpio@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
<regressions@...ts.linux.dev>, <lucapgl2001@...il.com>,
Mario Limonciello <mario.limonciello@....com>
Subject: [PATCH 3/3] pinctrl: amd: Add a quirk for Lenovo Ideapad 5
Lenovo ideapad 5 doesn't use interrupts for GPIO 0, and so internally
debouncing with WinBlue debounce behavior means that the GPIO doesn't
clear until a separate GPIO is used (such as touchpad).
Prefer to use legacy debouncing to avoid problems.
Reported-by: Luca Pigliacampo <lucapgl2001@...il.com>
Closes: https://bugzilla.kernel.org/show_bug.cgi?id=217833
Signed-off-by: Mario Limonciello <mario.limonciello@....com>
---
drivers/pinctrl/pinctrl-amd.c | 34 ++++++++++++++++++++++++++++++++--
1 file changed, 32 insertions(+), 2 deletions(-)
diff --git a/drivers/pinctrl/pinctrl-amd.c b/drivers/pinctrl/pinctrl-amd.c
index a2468a988be3..2e1721a9249a 100644
--- a/drivers/pinctrl/pinctrl-amd.c
+++ b/drivers/pinctrl/pinctrl-amd.c
@@ -8,6 +8,7 @@
*
*/
+#include <linux/dmi.h>
#include <linux/err.h>
#include <linux/bug.h>
#include <linux/kernel.h>
@@ -41,6 +42,27 @@ module_param(powerbtn, int, 0444);
MODULE_PARM_DESC(powerbtn,
"Power button debouncing: 0=traditional, 1=windows, -1=auto");
+struct pinctrl_amd_dmi_quirk {
+ int powerbtn;
+};
+
+static const struct dmi_system_id pinctrl_amd_dmi_quirks[] __initconst = {
+ {
+ /*
+ * Lenovo Ideapad 5
+ * Power button GPIO not cleared until touchpad movement
+ * https://bugzilla.kernel.org/show_bug.cgi?id=217833
+ */
+ .matches = {
+ DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
+ DMI_MATCH(DMI_PRODUCT_NAME, "82LM"),
+ },
+ .driver_data = &(struct pinctrl_amd_dmi_quirk) {
+ .powerbtn = 0,
+ },
+ }
+};
+
static int amd_gpio_get_direction(struct gpio_chip *gc, unsigned offset)
{
unsigned long flags;
@@ -1084,8 +1106,16 @@ static void handle_powerbtn(struct amd_gpio *gpio_dev)
{
u32 pin_reg;
- if (powerbtn == -1)
- return;
+ if (powerbtn == -1) {
+ const struct pinctrl_amd_dmi_quirk *quirk = NULL;
+ const struct dmi_system_id *id;
+
+ id = dmi_first_match(pinctrl_amd_dmi_quirks);
+ if (!id)
+ return;
+ quirk = id->driver_data;
+ powerbtn = quirk->powerbtn;
+ }
pin_reg = readl(gpio_dev->base + WAKE_INT_MASTER_REG);
switch (powerbtn) {
--
2.34.1
Powered by blists - more mailing lists