lkml.org 
[lkml]   [2022]   [Nov]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
SubjectFwd: [PATCH v2] platform/x86: ideapad-laptop: Fix fn-lock LED on Yoga 14ITL5 laptops
Date
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@illinois.edu>
To: Ike Panhc <ike.pan@canonical.com>, hdegoede@redhat.com
<hdegoede@redhat.com>, markgross@kernel.org <markgross@kernel.org>
CC: Rawat, Arnav <arnavr3@illinois.edu>

From 90832a9a373570db2a62c3edf00cf129b59b0ba3 Mon Sep 17 00:00:00 2001
From: arawat <rawat.arnav@gmail.com>
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@illinois.edu>
---
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


-----------------------------------------
\
 
 \ /
  Last update: 2022-11-14 20:16    [W:0.049 / U:0.544 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site