From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f49.google.com (mail-wm1-f49.google.com [209.85.128.49]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 44B44201278 for ; Wed, 26 Nov 2025 19:57:12 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.49 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1764187034; cv=none; b=Sb3RqXkHi5JQZECFrotqYRpwBOXA/YT9uQLQdWpvR4Q0W1flMilFjZHYtCXCP32/PKi6/Dn792E86Lv5BxyM928ddtxOlwV7w7S/AuN5ksCaupiiub/uYFB2yCuA6jpa2h22RQcJBtvsc1ceYUKsN5XO3loqCOcW77M3Bufy4OM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1764187034; c=relaxed/simple; bh=sFxdQJq77FHB9tvA6CD8aNGyL+khIcZI1+iJPTYgK0o=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=j9KVdxqZ0D0wBUF+oDpQyXEMfVRod4O0F2RtZr81XrssV0X3gEWcdwqSkg+PHZ85AuFJNHacjgC6TaAeRhavSwFCom72w/pRjOTRxsN4SmxBS+3hy1M5CeR6la57E2fthuDSPA+QSyuxn9J2S0A/+Rypbg085O21JUzvBVPKKkw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=aoeplI1a; arc=none smtp.client-ip=209.85.128.49 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="aoeplI1a" Received: by mail-wm1-f49.google.com with SMTP id 5b1f17b1804b1-47775fb6c56so588315e9.1 for ; Wed, 26 Nov 2025 11:57:12 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1764187031; x=1764791831; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=5d9lydYHfYTR/ACeyVzgTNDMdFsWzBAH+5B+oqVuOrk=; b=aoeplI1aLf71KS/Lee+Zi0U+oIokFBj1dNIRENWTHxHF+XMVFjlBh7cNGTA2SvVGII +TZDzRAhwOodta9nvijt1pZTTx8oNN89zRzxGz+k6STQDVfK12fpq92Ep9gZ0734pN7i FkwfvfEXmM4hvKWAXsamN9LcvSEh4NkriRCON/CfM8zYCnrGSjb0o74qW1WI4hMOQUbX 8IhQfMNtFTAEv9fuIyh8mcIdOlObUel0fsHot7u6F97KERqTVXRwxA9kjwHtV7phwx6f YnCpVrtPxQr9/vt8pQeVHBHbtoO8SxkolwuX4Rv/1HAKzsr6mcoDUKs0tzTIzceQ32D5 3Rng== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1764187031; x=1764791831; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=5d9lydYHfYTR/ACeyVzgTNDMdFsWzBAH+5B+oqVuOrk=; b=Azz3O4KNwjx1+ghso7eT+qr+I4wYdPxHMl+6NS8dU3r37VvSzIYkxGvHE1J8fb2nv8 ZAWAjMvbirrQbFLAoOEHYnoDb3ilHVTybYMsX6BsZzPzV4+YN4m1B2qA8umIHqID29uR eMbr1fqISwk7nXl5sSKRWKmrf8UHhRxgz1biyzbq6H656RQe/DNS/dFxhBQ2bnUvP3UR dh7jOJav/02SlcN33oikApefnpVLEl2goIUb24k9BSauMwYullewdqtMSt9USk3q0FJ6 T9yjAA7shv7pohaWNm0hBPYDBsOsSTZiKYvq9Aq892sOYqTMO0CZUyzyB8r88bKw2Cam d4PA== X-Forwarded-Encrypted: i=1; AJvYcCUzy6wiN3GdHPwoT1g8rR5tRLA+E5kdFnkK8kCXsA7yZZW7Fg0RXZ2fNnhE7DI8ydq02FurKw1E+/vmgSU=@vger.kernel.org X-Gm-Message-State: AOJu0YxswspyF+cQ8oTpW7lrdt3TAy0kt3pZ3xgdR/JRnaI2+M7xOjO1 fw15nhKgctpkvXMk3rjxBHSG21Af/jDoohnbseOsUIIVzS0kBHRCGQVV X-Gm-Gg: ASbGncvWH3jRERLHorvlrPVnhmUgx9Yo1hgGwsKsWUpMcUCeanmz36boUrptwsnkKhQ sUec2Dy4YnfaYNdS9fkZUOprBm9em29tmG4q7FxL/PBTqXV96dSF3hq00x1P8ssQWcmoZjA4wU+ 96xzrWabHicPPbhPQiialBznoRg8lfV5p/N8QxFZV7ee7e76lkiR90DnFygN16kT2AwTGOMjJ+k N3ZCj9HFMXCLqLfPuheG9Xf4sGZcqlw3XM6CIQJjIEiM0vmp5stQSzEHk86fb5cRlt89f53YFj7 t1jFTIyGrPpLh5iS2SsUcLXc8atm5gRnWN7aq/621CsR9iH0rLRafj69USUq+SeZ6Aco8JZiFCt wz2+0AZXYvYF0JDwhZlw024LbTWvrdHuLubhkO/T5gEY9Iw0Pb4v+c87H82PsbJk9iemHM3Cjk1 x6a0ivCFdA85JQa9IF7CplYaA= X-Google-Smtp-Source: AGHT+IGGyxkOHK8GZV6Ekt6wPcd/FqevLtVSgweKGQGNG+bJX6KjbUrpGirBnJ4u9ZEjKg1XNiVySQ== X-Received: by 2002:a05:600c:19c6:b0:477:7975:30ea with SMTP id 5b1f17b1804b1-47904b25e46mr73013525e9.29.1764187030408; Wed, 26 Nov 2025 11:57:10 -0800 (PST) Received: from [192.168.1.121] ([176.206.93.222]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-479040cfe17sm51335465e9.5.2025.11.26.11.57.09 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 26 Nov 2025 11:57:09 -0800 (PST) Message-ID: <577d33c5-bf91-4a23-813b-43df8135aa0d@gmail.com> Date: Wed, 26 Nov 2025 20:57:09 +0100 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v10 00/11] HID: asus: Fix ASUS ROG Laptop's Keyboard backlight handling To: Hans de Goede , =?UTF-8?Q?Ilpo_J=C3=A4rvinen?= , Antheas Kapenekakis Cc: platform-driver-x86@vger.kernel.org, linux-input@vger.kernel.org, LKML , Jiri Kosina , Benjamin Tissoires , Corentin Chary , "Luke D . Jones" References: <20251122110032.4274-1-lkml@antheas.dev> <4bcc6708-77ff-403d-83cf-10a40934263a@kernel.org> Content-Language: en-US, it-IT, en-US-large From: Denis Benato In-Reply-To: <4bcc6708-77ff-403d-83cf-10a40934263a@kernel.org> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit On 11/26/25 18:34, Hans de Goede wrote: > Hi, > > On 26-Nov-25 4:23 PM, Ilpo Järvinen wrote: >> On Wed, 26 Nov 2025, Antheas Kapenekakis wrote: > ... > >>> As for "HID: asus: early return for ROG devices" changing the name of >>> the devices of this driver, I will veto backporting it if it happens, >>> so inputplumber will have the two full months to remove the name >>> match. This is not a breaking change in the sense that software cannot >>> be made to work on both previous and latter versions and there is no >>> other software to my knowledge relying on name matches for Asus >>> keyboards. >> Did Hans give some opinion about this rename earlier, at least I don't >> remember nor could find from lore archives? > I don't remember commenting on this myself either. > > So generally speaking there are plenty of cases where /dev/input/event# > nodes for a specific device have their name changed by some kernel patches. > > Typically HID input devices are matched in userspace by their > bus:vend-id:prod-id triplet not by the name. The name might even > change by a fwupdate of the device itself. > > So I'm not overly worried about this and inputplumber seems nice > enough and already is very much not a plug-and-play tool. > > One possible concern with laptop keyboard input-device name changes > though is hwdb entries to fixup scancode -> ev-key-code mappings. > > See: /lib/udev/hwdb.d/60-keyboard.hwdb on any standard Linux systems > an then the big comment at the top. > > An input-device name change might break this match pattern: > > # - Input driver device name and DMI data match: > # evdev:name::dmi:bvn*:bvr*:bd*:svn:pn* > # is the name device specified by the > # driver, is the firmware-provided string exported > # by the kernel DMI modalias, see /sys/class/dmi/id/modalias > > As well as the extended version of this and for laptops with USB > keyboards this is the only match type which allows a DMI match > which is what we want for laptop kbd mappings. Looking at the Asus > section of the upstream 60-keyboard.hwdb I do not see any such > matches though. > > There not being such matches kinda make sense since for USB-HID > devices any special scancode -> ev-key-code mappings are typically > handled in a vendor specific HID driver like hid-asus. > > TL;DR: I think that the input-device name should be fine. Thank you very much for this write-up! Now I feel much more confident in giving approval! > Regards, > > Hans Thank you, Denis