Unverified Commit d8b17a36 authored by Luke D. Jones's avatar Luke D. Jones Committed by Ilpo Järvinen
Browse files

platform/x86: asus-wmi: fix TUF laptop RGB variant 



In kbd_rgb_mode_store the dev_get_drvdata() call was assuming the device
data was asus_wmi when it was actually led_classdev.

This patch corrects this by making the correct chain of calls to get the
asus_wmi driver data.

Fixes: ae834a54 ("platform/x86: asus-wmi: add support variant of TUF RGB")
Tested-by: default avatarDenis Benato <benato.denis96@gmail.com>
Signed-off-by: default avatarLuke D. Jones <luke@ljones.dev>
Reviewed-by: default avatarHans de Goede <hdegoede@redhat.com>
Link: https://lore.kernel.org/r/20240716011130.17464-2-luke@ljones.dev


Reviewed-by: default avatarIlpo Järvinen <ilpo.jarvinen@linux.intel.com>
Signed-off-by: default avatarIlpo Järvinen <ilpo.jarvinen@linux.intel.com>
parent 182c6941

drivers/platform/x86/asus-wmi.c

+5 −1
Original line number Diff line number Diff line
@@ -880,10 +880,14 @@ static ssize_t kbd_rgb_mode_store(struct device *dev, 
				 struct device_attribute *attr,

				 const char *buf, size_t count)

{

	struct asus_wmi *asus = dev_get_drvdata(dev);

	u32 cmd, mode, r, g, b, speed;

	struct led_classdev *led;

	struct asus_wmi *asus;

	int err;



	led = dev_get_drvdata(dev);

	asus = container_of(led, struct asus_wmi, kbd_led);



	if (sscanf(buf, "%d %d %d %d %d %d", &cmd, &mode, &r, &g, &b, &speed) != 6)

		return -EINVAL;