From nobody Sat Feb 7 15:15:26 2026 Received: from out-185.mta0.migadu.com (out-185.mta0.migadu.com [91.218.175.185]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7385D14BF92 for ; Thu, 25 Dec 2025 14:30:14 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=91.218.175.185 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766673016; cv=none; b=NGg18aFxcyFaCinczznYQPefX3d/PhlXi2F9wX1WCiPHmLQHmCIHNIuA5B9+rD32vAJMjwMMfHAS/m9TJaxTTO6Pru647VlsnOSpS2c64dYsaqY6S9Gaa5m/ACedgtj3kQne8E0J5x0Sufx4gbJbQ+W4gJ/V19WjZMoY7PYyHyk= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766673016; c=relaxed/simple; bh=XCQHYdq3spbZE86t6SluvclGO9w1CrqV32bMnR1uFeI=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=q3POhqibSD4HoKnpnhMAaeM5oYY2tuL8aWNmnbTkSS6Gt4hldPOX3UbytDniwYlYeCtvMwYCmrNWnUtqYoMp9G71ZjgHMZEW1o/IRV1l4vb1d/spbgfJjuJuEAiW0PlCldWEf5DwyhYoRXop9TO4sJTMJjMKfcbQwiiTwVVyF7Y= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.dev; spf=pass smtp.mailfrom=linux.dev; dkim=pass (1024-bit key) header.d=linux.dev header.i=@linux.dev header.b=Vt9b02sm; arc=none smtp.client-ip=91.218.175.185 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.dev Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linux.dev Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linux.dev header.i=@linux.dev header.b="Vt9b02sm" X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.dev; s=key1; t=1766673012; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=3m20kD7PwGCTH43gt8FUGu7YcWOl9a9RmHjQnwvr5Pg=; b=Vt9b02smnBCOZQK2tPLDiR4SzvqJxEdVx260PLhnzC2uDbcsGJYSZNedlFb4fwlLUoHnNM TwoTY3ubUkZSKo2tE6gZ3JYaGkxdodbX+WqHsJLDnsMswyT+DCuHSx99juCoM60W/Gl5Cc q/uLpT8zjXe37RwpMOl00Ej2XueEN6Y= From: Denis Benato To: linux-kernel@vger.kernel.org Cc: platform-driver-x86@vger.kernel.org, "Hans de Goede" , =?UTF-8?q?Ilpo=20J=C3=A4rvinen?= , "Luke D . Jones" , "Mateusz Schyboll" , "Denis Benato" , Denis Benato Subject: [PATCH 1/3] platform/x86: asus-wmi: explicitly mark more code with CONFIG_ASUS_WMI_DEPRECATED_ATTRS Date: Thu, 25 Dec 2025 15:30:05 +0100 Message-ID: <20251225143007.2684239-2-denis.benato@linux.dev> In-Reply-To: <20251225143007.2684239-1-denis.benato@linux.dev> References: <20251225143007.2684239-1-denis.benato@linux.dev> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Migadu-Flow: FLOW_OUT Content-Type: text/plain; charset="utf-8" Be more explicit in code that will be excluded when compiling with CONFIG_ASUS_WMI_DEPRECATED_ATTRS disabled. Signed-off-by: Denis Benato --- drivers/platform/x86/asus-wmi.c | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/drivers/platform/x86/asus-wmi.c b/drivers/platform/x86/asus-wm= i.c index 4aec7ec69250..a49447eff4f4 100644 --- a/drivers/platform/x86/asus-wmi.c +++ b/drivers/platform/x86/asus-wmi.c @@ -302,7 +302,11 @@ struct asus_wmi { u32 nv_temp_target; =20 u32 kbd_rgb_dev; + +#if IS_ENABLED(CONFIG_ASUS_WMI_DEPRECATED_ATTRS) bool kbd_rgb_state_available; +#endif /* IS_ENABLED(CONFIG_ASUS_WMI_DEPRECATED_ATTRS) */ + bool oobe_state_available; =20 u8 throttle_thermal_policy_mode; @@ -1060,6 +1064,7 @@ static const struct attribute_group kbd_rgb_mode_grou= p =3D { }; =20 /* TUF Laptop Keyboard RGB State *****************************************= *****/ +#if IS_ENABLED(CONFIG_ASUS_WMI_DEPRECATED_ATTRS) static ssize_t kbd_rgb_state_store(struct device *dev, struct device_attribute *attr, const char *buf, size_t count) @@ -1106,6 +1111,8 @@ static const struct attribute_group kbd_rgb_state_gro= up =3D { .attrs =3D kbd_rgb_state_attrs, }; =20 +#endif /* IS_ENABLED(CONFIG_ASUS_WMI_DEPRECATED_ATTRS) */ + static const struct attribute_group *kbd_rgb_mode_groups[] =3D { NULL, NULL, @@ -1861,8 +1868,11 @@ static int asus_wmi_led_init(struct asus_wmi *asus) =20 if (asus->kbd_rgb_dev) kbd_rgb_mode_groups[num_rgb_groups++] =3D &kbd_rgb_mode_group; + +#if IS_ENABLED(CONFIG_ASUS_WMI_DEPRECATED_ATTRS) if (asus->kbd_rgb_state_available) kbd_rgb_mode_groups[num_rgb_groups++] =3D &kbd_rgb_state_group; +#endif /* IS_ENABLED(CONFIG_ASUS_WMI_DEPRECATED_ATTRS) */ =20 asus->led_workqueue =3D create_singlethread_workqueue("led_workqueue"); if (!asus->led_workqueue) --=20 2.52.0 From nobody Sat Feb 7 15:15:26 2026 Received: from out-172.mta0.migadu.com (out-172.mta0.migadu.com [91.218.175.172]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7010E1397 for ; Thu, 25 Dec 2025 14:30:15 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=91.218.175.172 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766673017; cv=none; b=Q2ynztVdXQYQ5eXNiO1rDys0Z07X+J/wUHUM98ROSu6UfHZG/HZ/PMixqSsXXd78+BJDxQipqfsWpwLlWh30WPx3bzTvbpZSbCgVlKxeRDD+MrHy/fa+0RLWJMfFuguzv2CeD58q2OHOLziR5papC7a2OieSEey3TT1+he2V7e8= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766673017; c=relaxed/simple; bh=X/L3uuVFYSgyBij6ji1XU7UH7su430duIjXhDgZc1+Y=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=LjtjkwmJp3XUWsWXQ9wnaGtqukrwHLFTL7nxSr0lI0h310MwIpyXNjQtMYN+h7rPuzRvzfuViH7de9nFUZt6HU536tN6Od3SlWy8bgcTiK6+OOqzbAK1Na0W5CsWo45OMkLKDoYPGvEFKWFO25+iX05jk6gbwDgmNGxQ8rHZSy8= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.dev; spf=pass smtp.mailfrom=linux.dev; dkim=pass (1024-bit key) header.d=linux.dev header.i=@linux.dev header.b=EkpNdX4r; arc=none smtp.client-ip=91.218.175.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.dev Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linux.dev Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linux.dev header.i=@linux.dev header.b="EkpNdX4r" X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.dev; s=key1; t=1766673013; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=O047kvp7XWl7YkggHvF0RN/6JImVsvXOkwpYjNSppcM=; b=EkpNdX4r8zis0XVjU4UZW3j7MEOj7blorGNBgd5mEAKIjTKwryMJ+zUEmkYPvbkLfFQm6f DIDwoViW3n8e6GK+aZPlhuKwNxiOO6tuAmeuvBNbDg/DpdF136VIx5AnkAeSfqSQsaVt8C iLO1mmgjpPmC2AtmypiRl9ubahOWurs= From: Denis Benato To: linux-kernel@vger.kernel.org Cc: platform-driver-x86@vger.kernel.org, "Hans de Goede" , =?UTF-8?q?Ilpo=20J=C3=A4rvinen?= , "Luke D . Jones" , "Mateusz Schyboll" , "Denis Benato" , Denis Benato Subject: [PATCH 2/3] platform/x86: asus-wmi: fix sending OOBE at probe Date: Thu, 25 Dec 2025 15:30:06 +0100 Message-ID: <20251225143007.2684239-3-denis.benato@linux.dev> In-Reply-To: <20251225143007.2684239-1-denis.benato@linux.dev> References: <20251225143007.2684239-1-denis.benato@linux.dev> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Migadu-Flow: FLOW_OUT Content-Type: text/plain; charset="utf-8" Disabling oobe is an important step to be able to fully control the hardware in TUF laptops that supports this feature, but the command has been incorrectly tied to deprecated attributes: restore sending the OOBE exit command. Signed-off-by: Denis Benato --- drivers/platform/x86/asus-wmi.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/platform/x86/asus-wmi.c b/drivers/platform/x86/asus-wm= i.c index a49447eff4f4..8dfdde7877a8 100644 --- a/drivers/platform/x86/asus-wmi.c +++ b/drivers/platform/x86/asus-wmi.c @@ -4899,7 +4899,6 @@ static int asus_wmi_add(struct platform_device *pdev) asus->egpu_enable_available =3D asus_wmi_dev_is_present(asus, ASUS_WMI_DE= VID_EGPU); asus->dgpu_disable_available =3D asus_wmi_dev_is_present(asus, ASUS_WMI_D= EVID_DGPU); asus->kbd_rgb_state_available =3D asus_wmi_dev_is_present(asus, ASUS_WMI_= DEVID_TUF_RGB_STATE); - asus->oobe_state_available =3D asus_wmi_dev_is_present(asus, ASUS_WMI_DEV= ID_OOBE); =20 if (asus_wmi_dev_is_present(asus, ASUS_WMI_DEVID_MINI_LED_MODE)) asus->mini_led_dev_id =3D ASUS_WMI_DEVID_MINI_LED_MODE; @@ -4912,6 +4911,8 @@ static int asus_wmi_add(struct platform_device *pdev) asus->gpu_mux_dev =3D ASUS_WMI_DEVID_GPU_MUX_VIVO; #endif /* IS_ENABLED(CONFIG_ASUS_WMI_DEPRECATED_ATTRS) */ =20 + asus->oobe_state_available =3D asus_wmi_dev_is_present(asus, ASUS_WMI_DEV= ID_OOBE); + if (asus_wmi_dev_is_present(asus, ASUS_WMI_DEVID_THROTTLE_THERMAL_POLICY)) asus->throttle_thermal_policy_dev =3D ASUS_WMI_DEVID_THROTTLE_THERMAL_PO= LICY; else if (asus_wmi_dev_is_present(asus, ASUS_WMI_DEVID_THROTTLE_THERMAL_PO= LICY_VIVO)) --=20 2.52.0 From nobody Sat Feb 7 15:15:26 2026 Received: from out-178.mta0.migadu.com (out-178.mta0.migadu.com [91.218.175.178]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7CFC8824BD for ; Thu, 25 Dec 2025 14:30:16 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=91.218.175.178 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766673018; cv=none; b=K6HIXuWhjmONqRC8Qu0GHHeeZsNghrN+392UnnlJPg173zHNTbofEpARpgkQ+nMzlGDmS43DQN0EtsakmoYpUt7ZMa3ZAXHwND2ShgQ3jHDDe2Rw89/2K3LC/JqZ7RZm9qH7w6uIi/JY70L2TEAZ8K5GxJCxP/orXQA3WAa2otI= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766673018; c=relaxed/simple; bh=Bgc7f+zcDXtUSZxuCVW0o8XX0Fg9ampdvxiKc6dwmBY=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=pqCkGrUEM+x1ykWdL9FQSOgx8emI5H5p8pP67HdBusgzfFx0ZFYoemMB/QkPehztz2I2oc+FhtUQT+6IgVaaZ0IwAk/ZGgw1aSTTMJlWUbFcbOdjeqvySBG2u5xpe3Sm18+aGa8VTiTfCmmWB4wlgos4QVMlxLAMjlHoYuKhEw8= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.dev; spf=pass smtp.mailfrom=linux.dev; dkim=pass (1024-bit key) header.d=linux.dev header.i=@linux.dev header.b=O9WB5ZrB; arc=none smtp.client-ip=91.218.175.178 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.dev Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linux.dev Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linux.dev header.i=@linux.dev header.b="O9WB5ZrB" X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.dev; s=key1; t=1766673014; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=NPy2y/WeFTE5qeVxrFoySqM7SqNk3yZh3X3YJSnkXig=; b=O9WB5ZrBBHwmMgKx/umZYq3qN/jqUqkFLgI+orjziShMya5XqqNHRlpPfkX1zG42k6r44d H+kzE4CbUNmL7PDPwBt4tudBgDjnRVo1cbGD+z0JA1VWzZtBsgZPZ5khHB6IpTvWpiRi7k 1z8C3iz3Z0iYNVJdH2QJAn2qALC7hqQ= From: Denis Benato To: linux-kernel@vger.kernel.org Cc: platform-driver-x86@vger.kernel.org, "Hans de Goede" , =?UTF-8?q?Ilpo=20J=C3=A4rvinen?= , "Luke D . Jones" , "Mateusz Schyboll" , "Denis Benato" , Denis Benato Subject: [PATCH 3/3] platform/x86: asus-armoury: add keyboard control firmware attributes Date: Thu, 25 Dec 2025 15:30:07 +0100 Message-ID: <20251225143007.2684239-4-denis.benato@linux.dev> In-Reply-To: <20251225143007.2684239-1-denis.benato@linux.dev> References: <20251225143007.2684239-1-denis.benato@linux.dev> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Migadu-Flow: FLOW_OUT Content-Type: text/plain; charset="utf-8" Implement keyboard status attributes in asus-armoury after deprecating those attribute(s) in asus-wmi to avoid losing the ability to control LEDs status. Signed-off-by: Denis Benato --- drivers/platform/x86/asus-armoury.c | 258 +++++++++++++++++++++ include/linux/platform_data/x86/asus-wmi.h | 15 ++ 2 files changed, 273 insertions(+) diff --git a/drivers/platform/x86/asus-armoury.c b/drivers/platform/x86/asu= s-armoury.c index 9c1a9ad42bc4..a37713854691 100644 --- a/drivers/platform/x86/asus-armoury.c +++ b/drivers/platform/x86/asus-armoury.c @@ -76,10 +76,22 @@ struct rog_tunables { u32 nv_tgp; }; =20 +struct asus_armoury_kbd_status { + bool boot; + bool awake; + bool sleep; + bool shutdown; +}; + struct asus_armoury_priv { struct device *fw_attr_dev; struct kset *fw_attr_kset; =20 + struct mutex keyboard_mutex; + + /* Current TUF keyboard RGB state tracking */ + struct asus_armoury_kbd_status *kbd_state; + /* * Mutex to protect eGPU activation/deactivation * sequences and dGPU connection status: @@ -97,6 +109,7 @@ struct asus_armoury_priv { =20 static struct asus_armoury_priv asus_armoury =3D { .egpu_mutex =3D __MUTEX_INITIALIZER(asus_armoury.egpu_mutex), + .keyboard_mutex =3D __MUTEX_INITIALIZER(asus_armoury.keyboard_mutex), }; =20 struct fw_attrs_group { @@ -433,6 +446,169 @@ static ssize_t mini_led_mode_possible_values_show(str= uct kobject *kobj, } ASUS_ATTR_GROUP_ENUM(mini_led_mode, "mini_led_mode", "Set the mini-LED bac= klight mode"); =20 +/* Keyboard power management *********************************************= *****/ + +static int armoury_kbd_state(struct kobj_attribute *attr, + const struct asus_armoury_kbd_status *status) +{ + u32 kbd_state =3D 0xBD | BIT(2) << 8; + + if (status->boot) + kbd_state |=3D BIT(1) << 16; + if (status->awake) + kbd_state |=3D BIT(3) << 16; + if (status->sleep) + kbd_state |=3D BIT(5) << 16; + if (status->shutdown) + kbd_state |=3D BIT(7) << 16; + + return armoury_set_devstate(attr, kbd_state, NULL, + ASUS_WMI_DEVID_TUF_RGB_STATE); +} + +enum asus_armoury_kbd_state_field { + ASUS_ARMOURY_KBD_STATE_BOOT, + ASUS_ARMOURY_KBD_STATE_AWAKE, + ASUS_ARMOURY_KBD_STATE_SLEEP, + ASUS_ARMOURY_KBD_STATE_SHUTDOWN, +}; + +static ssize_t armoury_kbd_state_write(struct kobject *kobj, struct kobj_a= ttribute *attr, + const char *buf, size_t count, + enum asus_armoury_kbd_state_field field) +{ + ssize_t err; + bool enable; + struct asus_armoury_kbd_status kbd_status; + + err =3D kstrtobool(buf, &enable); + if (err) + return err; + + scoped_guard(mutex, &asus_armoury.keyboard_mutex) { + memcpy(&kbd_status, asus_armoury.kbd_state, sizeof(kbd_status)); + + switch (field) { + case ASUS_ARMOURY_KBD_STATE_BOOT: + kbd_status.boot =3D enable; + break; + case ASUS_ARMOURY_KBD_STATE_AWAKE: + kbd_status.awake =3D enable; + break; + case ASUS_ARMOURY_KBD_STATE_SLEEP: + kbd_status.sleep =3D enable; + break; + case ASUS_ARMOURY_KBD_STATE_SHUTDOWN: + kbd_status.shutdown =3D enable; + break; + default: + return -EINVAL; + } + + err =3D armoury_kbd_state(attr, &kbd_status); + if (err) + return err; + + memcpy(asus_armoury.kbd_state, &kbd_status, sizeof(kbd_status)); + } + + sysfs_notify(kobj, NULL, attr->attr.name); + + return count; +} + +static ssize_t armoury_kbd_state_read(struct kobject *kobj, struct kobj_at= tribute *attr, + char *buf, enum asus_armoury_kbd_state_field field) +{ + bool *field_ptr, field_enabled; + + switch (field) { + case ASUS_ARMOURY_KBD_STATE_AWAKE: + field_ptr =3D &asus_armoury.kbd_state->awake; + break; + case ASUS_ARMOURY_KBD_STATE_SLEEP: + field_ptr =3D &asus_armoury.kbd_state->sleep; + break; + case ASUS_ARMOURY_KBD_STATE_BOOT: + field_ptr =3D &asus_armoury.kbd_state->boot; + break; + case ASUS_ARMOURY_KBD_STATE_SHUTDOWN: + field_ptr =3D &asus_armoury.kbd_state->shutdown; + break; + default: + return -EINVAL; + } + + scoped_guard(mutex, &asus_armoury.keyboard_mutex) + field_enabled =3D *field_ptr; + + return sysfs_emit(buf, field_enabled ? "1\n" : "0\n"); +} + +static ssize_t kbd_leds_sleep_current_value_store(struct kobject *kobj, + struct kobj_attribute *attr, + const char *buf, size_t count) +{ + return armoury_kbd_state_write(kobj, attr, buf, count, ASUS_ARMOURY_KBD_S= TATE_SLEEP); +} + +static ssize_t kbd_leds_sleep_current_value_show(struct kobject *kobj, + struct kobj_attribute *attr, char *buf) +{ + return armoury_kbd_state_read(kobj, attr, buf, ASUS_ARMOURY_KBD_STATE_SLE= EP); +} + +ASUS_ATTR_GROUP_BOOL(kbd_leds_sleep, "kbd_leds_sleep", + "Keyboard backlight while system is sleeping"); + +static ssize_t kbd_leds_boot_current_value_store(struct kobject *kobj, + struct kobj_attribute *attr, + const char *buf, size_t count) +{ + return armoury_kbd_state_write(kobj, attr, buf, count, ASUS_ARMOURY_KBD_S= TATE_BOOT); +} + +static ssize_t kbd_leds_boot_current_value_show(struct kobject *kobj, + struct kobj_attribute *attr, char *buf) +{ + return armoury_kbd_state_read(kobj, attr, buf, ASUS_ARMOURY_KBD_STATE_BOO= T); +} + +ASUS_ATTR_GROUP_BOOL(kbd_leds_boot, "kbd_leds_boot", + "Keyboard backlight while system is booting"); + +static ssize_t kbd_leds_awake_current_value_store(struct kobject *kobj, + struct kobj_attribute *attr, + const char *buf, size_t count) +{ + return armoury_kbd_state_write(kobj, attr, buf, count, ASUS_ARMOURY_KBD_S= TATE_AWAKE); +} + +static ssize_t kbd_leds_awake_current_value_show(struct kobject *kobj, + struct kobj_attribute *attr, char *buf) +{ + return armoury_kbd_state_read(kobj, attr, buf, ASUS_ARMOURY_KBD_STATE_AWA= KE); +} + +ASUS_ATTR_GROUP_BOOL(kbd_leds_awake, "kbd_leds_awake", + "Keyboard backlight while system is awake"); + +static ssize_t kbd_leds_shutdown_current_value_store(struct kobject *kobj, + struct kobj_attribute *attr, + const char *buf, size_t count) +{ + return armoury_kbd_state_write(kobj, attr, buf, count, ASUS_ARMOURY_KBD_S= TATE_SHUTDOWN); +} + +static ssize_t kbd_leds_shutdown_current_value_show(struct kobject *kobj, + struct kobj_attribute *attr, char *buf) +{ + return armoury_kbd_state_read(kobj, attr, buf, ASUS_ARMOURY_KBD_STATE_SHU= TDOWN); +} + +ASUS_ATTR_GROUP_BOOL(kbd_leds_shutdown, "kbd_leds_shutdown", + "Keyboard backlight while system is shutdown"); + static ssize_t gpu_mux_mode_current_value_store(struct kobject *kobj, struct kobj_attribute *attr, const char *buf, size_t count) @@ -867,6 +1043,35 @@ static bool has_valid_limit(const char *name, const s= truct power_limits *limits) return limit_value > 0; } =20 +static struct asus_armoury_kbd_status *asus_init_kbd_state(void) +{ + int err; + u32 kbd_status; + struct asus_armoury_kbd_status *kbd_state __free(kfree) =3D NULL; + + err =3D armoury_get_devstate(NULL, &kbd_status, ASUS_WMI_DEVID_TUF_RGB_ST= ATE); + if (err) { + pr_err("ACPI does not support keyboard power control: %d\n", err); + return ERR_PTR(-ENODEV); + } + + pr_info("Detected keyboard backlight support\n"); + + kbd_state =3D kzalloc(sizeof(*kbd_state), GFP_KERNEL); + if (!kbd_state) + return ERR_PTR(-ENODEV); + + /* + * By default leds are off for all states (to spare power) + * except for when laptop is awake, where leds color and + * brightness are controllable by userspace. + */ + memset(kbd_state, 0, sizeof(*kbd_state)); + kbd_state->awake =3D true; + + return no_free_ptr(kbd_state); +} + static int asus_fw_attr_add(void) { const struct rog_tunables *const ac_rog_tunables =3D @@ -926,6 +1131,51 @@ static int asus_fw_attr_add(void) } } =20 + asus_armoury.kbd_state =3D NULL; + if (armoury_has_devstate(ASUS_WMI_DEVID_TUF_RGB_STATE)) { + asus_armoury.kbd_state =3D asus_init_kbd_state(); + if (IS_ERR(asus_armoury.kbd_state)) { + err =3D PTR_ERR(asus_armoury.kbd_state); + pr_err("Failed to get keyboard status: %d\n", err); + goto err_remove_kbd_state; + } + + err =3D sysfs_create_group(&asus_armoury.fw_attr_kset->kobj, &kbd_leds_s= leep_attr_group); + if (err) { + pr_err("Failed to create sysfs-group for keyboard backlight sleep state= : %d\n", err); + goto err_remove_kbd_state; + } + + err =3D sysfs_create_group(&asus_armoury.fw_attr_kset->kobj, &kbd_leds_b= oot_attr_group); + if (err) { + pr_err("Failed to create sysfs-group for keyboard backlight boot state:= %d\n", err); + goto err_remove_kbd_state; + } + + err =3D sysfs_create_group(&asus_armoury.fw_attr_kset->kobj, &kbd_leds_a= wake_attr_group); + if (err) { + pr_err("Failed to create sysfs-group for keyboard backlight awake state= : %d\n", err); + goto err_remove_kbd_state; + } + + err =3D sysfs_create_group(&asus_armoury.fw_attr_kset->kobj, &kbd_leds_s= hutdown_attr_group); + if (err) { + pr_err("Failed to create sysfs-group for keyboard backlight shutdown st= ate: %d\n", err); + goto err_remove_kbd_state; + } + + /* + * The attribute is write-only and for the state to be coherent + * a default state has to written: userspace is expected to + * modify it based on user preference. + */ + err =3D armoury_kbd_state(&attr_kbd_leds_awake_current_value, asus_armou= ry.kbd_state); + if (err) { + pr_err("Failed to initialize keyboard backlight states: %d\n", err); + goto err_remove_kbd_state; + } + } + for (i =3D 0; i < ARRAY_SIZE(armoury_attr_groups); i++) { if (!armoury_has_devstate(armoury_attr_groups[i].wmi_devid)) continue; @@ -965,6 +1215,14 @@ static int asus_fw_attr_add(void) } if (asus_armoury.gpu_mux_dev_id) sysfs_remove_group(&asus_armoury.fw_attr_kset->kobj, &gpu_mux_mode_attr_= group); +err_remove_kbd_state: + kfree(asus_armoury.kbd_state); + if (asus_armoury.kbd_state) { + sysfs_remove_group(&asus_armoury.fw_attr_kset->kobj, &kbd_leds_sleep_att= r_group); + sysfs_remove_group(&asus_armoury.fw_attr_kset->kobj, &kbd_leds_boot_attr= _group); + sysfs_remove_group(&asus_armoury.fw_attr_kset->kobj, &kbd_leds_awake_att= r_group); + sysfs_remove_group(&asus_armoury.fw_attr_kset->kobj, &kbd_leds_shutdown_= attr_group); + } err_remove_mini_led_group: if (asus_armoury.mini_led_dev_id) sysfs_remove_group(&asus_armoury.fw_attr_kset->kobj, &mini_led_mode_attr= _group); diff --git a/include/linux/platform_data/x86/asus-wmi.h b/include/linux/pla= tform_data/x86/asus-wmi.h index 419491d4abca..6667b3b474c2 100644 --- a/include/linux/platform_data/x86/asus-wmi.h +++ b/include/linux/platform_data/x86/asus-wmi.h @@ -153,6 +153,21 @@ /* TUF laptop RGB power/state */ #define ASUS_WMI_DEVID_TUF_RGB_STATE 0x00100057 =20 +/* + * Flags for TUF RGB state to be used with + * ASUS_WMI_DEVID_TUF_RGB_STATE: + * 0xBD | BIT(1) << 8 | flags + * + * where 0xBD is required for the method call + * to not be discarded, BIT(2)<<8 specified + * this is a command and flags is a combination + * of one or more of the following flags. + */ +#define ASUS_WMI_DEVID_TUF_RGB_STATE_BOOT (BIT(1) << 16u) +#define ASUS_WMI_DEVID_TUF_RGB_STATE_AWAKE (BIT(3) << 16u) +#define ASUS_WMI_DEVID_TUF_RGB_STATE_SLEEP (BIT(5) << 16u) +#define ASUS_WMI_DEVID_TUF_RGB_STATE_SHUTDOWN (BIT(7) << 16u) + /* Bootup sound control */ #define ASUS_WMI_DEVID_BOOT_SOUND 0x00130022 =20 --=20 2.52.0