From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from relay10.grserver.gr (relay10.grserver.gr [37.27.248.198]) (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 97ED526E6E1; Sat, 18 Oct 2025 10:20:14 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=37.27.248.198 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1760782816; cv=none; b=FROd+ykXdI1UHz8UmoxIKvDFGW+qxKAxYy2T6CsD4eVuvCvkKx0KhfVFmeyxXlT8s1sgS3udWxT4ntHY5MxLVvrw3tFCD9vaLjDsPZphUPY2mdazQTRUkdEGVoCT36fjG6TAS6YENowycwDnijmpCQDz1o25x5AUSD6s+JVsp08= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1760782816; c=relaxed/simple; bh=bAVsTtVVadOWp97QgVy+yErNXpIetJbrzqa/Gn3C1Ss=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=irO9SRJ857MD9m/W/CfDlAW4q3iJUV5dhCWi8VwO0/2G7knubmrdl91r5ZmywTKBIB7yCmHRWa1C3EKVa8AxVO3eSzxVta8pcEF8hJHYOUmse6i8Fy8NSGCWEY/arqSteqdGBGZBy2Rqr2MvdRh5FwJmNBMpCyvpdlHGA70hois= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=antheas.dev; spf=pass smtp.mailfrom=antheas.dev; dkim=temperror (0-bit key) header.d=antheas.dev header.i=@antheas.dev header.b=E2jxEqsy; arc=none smtp.client-ip=37.27.248.198 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=antheas.dev Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=antheas.dev Authentication-Results: smtp.subspace.kernel.org; dkim=temperror (0-bit key) header.d=antheas.dev header.i=@antheas.dev header.b="E2jxEqsy" Received: from relay10 (localhost.localdomain [127.0.0.1]) by relay10.grserver.gr (Proxmox) with ESMTP id BF3183FD44; Sat, 18 Oct 2025 13:20:06 +0300 (EEST) Received: from linux3247.grserver.gr (linux3247.grserver.gr [213.158.90.240]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by relay10.grserver.gr (Proxmox) with ESMTPS id 487853FD2C; Sat, 18 Oct 2025 13:20:06 +0300 (EEST) Received: from antheas-z13 (x5996a8de.customers.hiper-net.dk [89.150.168.222]) by linux3247.grserver.gr (Postfix) with ESMTPSA id 007B91FC4B1; Sat, 18 Oct 2025 13:20:03 +0300 (EEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=antheas.dev; s=default; t=1760782805; bh=127svcBDXgweZY4aI/q7KZPt4QURB379Gsk87bd/Plc=; h=From:To:Subject; b=E2jxEqsyM8e3NpEC1QzQrmXphj0Pmw1AQm/1BW/nbvm5+IxFCnx8Gn7a7krMD5/rm W1q7KGHnLMpqq6JaDJkI4PpwRdbn/knPtf6sIgQFydPxsBxLlnv4NLDqOV+fR5yEkS DDAFG1cOb6HECB1GgI/Fltdf8JtZ83EEwIHWjhIrA8R6jR49ye0WV6IHDWEp2BE+tj MEPe4htN0AWhxlniFQ6Dmf3EMthuyoc482IVRvJULqAOTGeJllrSh9jHNHiFxZghb5 DKkBIH2QJRH2EPWBjUUiwYHb1gBcuy/EPiIcO84dVXnuU1DiE/UfN27e0FT9RbS3JY gFnO3xkXUPAwg== Authentication-Results: linux3247.grserver.gr; spf=pass (sender IP is 89.150.168.222) smtp.mailfrom=lkml@antheas.dev smtp.helo=antheas-z13 Received-SPF: pass (linux3247.grserver.gr: connection is authenticated) From: Antheas Kapenekakis To: platform-driver-x86@vger.kernel.org, linux-input@vger.kernel.org Cc: linux-kernel@vger.kernel.org, Jiri Kosina , Benjamin Tissoires , Corentin Chary , "Luke D . Jones" , Hans de Goede , =?UTF-8?q?Ilpo=20J=C3=A4rvinen?= , Denis Benato , Antheas Kapenekakis Subject: [PATCH v7 1/9] HID: asus: simplify RGB init sequence Date: Sat, 18 Oct 2025 12:17:51 +0200 Message-ID: <20251018101759.4089-2-lkml@antheas.dev> X-Mailer: git-send-email 2.51.0 In-Reply-To: <20251018101759.4089-1-lkml@antheas.dev> References: <20251018101759.4089-1-lkml@antheas.dev> Precedence: bulk X-Mailing-List: linux-input@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-PPP-Message-ID: <176078280579.1198150.338744444978924506@linux3247.grserver.gr> X-PPP-Vhost: antheas.dev X-Virus-Scanned: clamav-milter 1.4.3 at linux3247.grserver.gr X-Virus-Status: Clean Currently, RGB initialization forks depending on whether a device is NKEY. Then, NKEY devices are initialized using 0x5a, 0x5d, 0x5e endpoints, and non-NKEY devices with 0x5a and then a backlight check, which is omitted for NKEY devices. Remove the fork, using a common initialization sequence for both, where they are both only initialized with 0x5a, then checked for backlight support. This patch should not affect existing functionality. 0x5d and 0x5e endpoint initializations are performed by Windows userspace programs associated with different usages that reside under the vendor HID. Specifically, 0x5d is used by Armoury Crate, which controls RGB and 0x5e by an animation program for certain Asus laptops. Neither is used currently in the driver. Signed-off-by: Antheas Kapenekakis --- drivers/hid/hid-asus.c | 56 ++++++++++++++---------------------------- 1 file changed, 19 insertions(+), 37 deletions(-) diff --git a/drivers/hid/hid-asus.c b/drivers/hid/hid-asus.c index a444d41e53b6..7ea1037c3979 100644 --- a/drivers/hid/hid-asus.c +++ b/drivers/hid/hid-asus.c @@ -638,50 +638,32 @@ static int asus_kbd_register_leds(struct hid_device *hdev) unsigned char kbd_func; int ret; - if (drvdata->quirks & QUIRK_ROG_NKEY_KEYBOARD) { - /* Initialize keyboard */ - ret = asus_kbd_init(hdev, FEATURE_KBD_REPORT_ID); - if (ret < 0) - return ret; - - /* The LED endpoint is initialised in two HID */ - ret = asus_kbd_init(hdev, FEATURE_KBD_LED_REPORT_ID1); - if (ret < 0) - return ret; - - ret = asus_kbd_init(hdev, FEATURE_KBD_LED_REPORT_ID2); - if (ret < 0) - return ret; - - if (dmi_match(DMI_PRODUCT_FAMILY, "ProArt P16")) { - ret = asus_kbd_disable_oobe(hdev); - if (ret < 0) - return ret; - } - - if (drvdata->quirks & QUIRK_ROG_ALLY_XPAD) { - intf = to_usb_interface(hdev->dev.parent); - udev = interface_to_usbdev(intf); - validate_mcu_fw_version(hdev, - le16_to_cpu(udev->descriptor.idProduct)); - } + ret = asus_kbd_init(hdev, FEATURE_KBD_REPORT_ID); + if (ret < 0) + return ret; - } else { - /* Initialize keyboard */ - ret = asus_kbd_init(hdev, FEATURE_KBD_REPORT_ID); - if (ret < 0) - return ret; + /* Get keyboard functions */ + ret = asus_kbd_get_functions(hdev, &kbd_func, FEATURE_KBD_REPORT_ID); + if (ret < 0) + return ret; - /* Get keyboard functions */ - ret = asus_kbd_get_functions(hdev, &kbd_func, FEATURE_KBD_REPORT_ID); + if (dmi_match(DMI_PRODUCT_FAMILY, "ProArt P16")) { + ret = asus_kbd_disable_oobe(hdev); if (ret < 0) return ret; + } - /* Check for backlight support */ - if (!(kbd_func & SUPPORT_KBD_BACKLIGHT)) - return -ENODEV; + if (drvdata->quirks & QUIRK_ROG_ALLY_XPAD) { + intf = to_usb_interface(hdev->dev.parent); + udev = interface_to_usbdev(intf); + validate_mcu_fw_version( + hdev, le16_to_cpu(udev->descriptor.idProduct)); } + /* Check for backlight support */ + if (!(kbd_func & SUPPORT_KBD_BACKLIGHT)) + return -ENODEV; + drvdata->kbd_backlight = devm_kzalloc(&hdev->dev, sizeof(struct asus_kbd_leds), GFP_KERNEL); -- 2.51.0