Subject: [PATCH v2] platform/x86: ideapad-laptop: Fix fn-lock LED on Yoga
14ITL5 laptops
Date: Friday, November 11, 2022, 8:32:09 AM CST
From: Rawat, Arnav <arnavr3@xxxxxxxxxxxx>
To: Ike Panhc <ike.pan@xxxxxxxxxxxxx>, hdegoede@xxxxxxxxxx
<hdegoede@xxxxxxxxxx>, markgross@xxxxxxxxxx <markgross@xxxxxxxxxx>
CC: Rawat, Arnav <arnavr3@xxxxxxxxxxxx>
>From 90832a9a373570db2a62c3edf00cf129b59b0ba3 Mon Sep 17 00:00:00 2001
From: arawat <rawat.arnav@xxxxxxxxx>
Date: Thu, 10 Nov 2022 13:13:11 -0600
Subject: [PATCH v2] platform/x86: ideapad-laptop: Fix fn-lock LED on Yoga
14ITL5
laptops
The commit 3ae86d2d4704796ee658a34245cb86e68c40c5d7: Fix Legion 5 Fnlock LED
set the WMI id for the fn-lock event on some Legion 5 laptops. However,
the same WMI ID is also sent on some Yoga laptops. Here, setting the fn-lock
state is not valid behavior, and causes the ec to spam interrupts until the
laptop is rebooted, so include a check for this line of laptops.
Signed-off-by: Arnav Rawat <arnavr3@xxxxxxxxxxxx>
---
drivers/platform/x86/ideapad-laptop.c | 14 ++++++++++++-
1 file changed, 12 insertions(+), 1 deletion(-)
diff --git a/drivers/platform/x86/ideapad-laptop.c b/drivers/platform/x86/
ideapad-laptop.c
index abd0c81d62c4..d1dcf57ce596 100644
--- a/drivers/platform/x86/ideapad-laptop.c
+++ b/drivers/platform/x86/ideapad-laptop.c
@@ -1491,6 +1492,17 @@ static void ideapad_acpi_notify(acpi_handle handle, u32
event, void *data)
}
#if IS_ENABLED(CONFIG_ACPI_WMI)
+// Set fnesc state only on certain ideapads
+static const struct dmi_system_id ideapad_fnesc_allow_list[] = {
+ {
+ .matches = {
+ DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
+ DMI_MATCH(DMI_PRODUCT_VERSION, "Lenovo Legion
R7000P2020H"),
+ DMI_MATCH(DMI_PRODUCT_NAME, "82GR")
+ }
+ }
+};
+
static void ideapad_wmi_notify(u32 value, void *context)
{
struct ideapad_private *priv = context;
@@ -1501,7 +1513,7 @@ static void ideapad_wmi_notify(u32 value, void *context)
ideapad_input_report(priv, value);
break;
case 208:
- if (!eval_hals(priv->adev->handle, &result)) {
+ if (!eval_hals(priv->adev->handle, &result) &&
dmi_check_system(ideapad_fnesc_allow_list)) {
bool state = test_bit(HALS_FNLOCK_STATE_BIT,
&result);
exec_sals(priv->adev->handle, state ?
SALS_FNLOCK_ON : SALS_FNLOCK_OFF);
--
2.37.3
-----------------------------------------
