linux-input.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Josh Leivenzon <joshleivenzon@outlook.com>
To: "luke@ljones.dev" <luke@ljones.dev>
Cc: "bentiss@kernel.org" <bentiss@kernel.org>,
	"corentin.chary@gmail.com" <corentin.chary@gmail.com>,
	"hdegoede@redhat.com" <hdegoede@redhat.com>,
	"ilpo.jarvinen@linux.intel.com" <ilpo.jarvinen@linux.intel.com>,
	"jikos@kernel.org" <jikos@kernel.org>,
	"linux-input@vger.kernel.org" <linux-input@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"platform-driver-x86@vger.kernel.org"
	<platform-driver-x86@vger.kernel.org>
Subject: Re: [PATCH v3 1/1] hid-asus: use hid for brightness control on keyboard
Date: Sun, 12 Jan 2025 12:18:28 +0000	[thread overview]
Message-ID: <20250112121811.434552-1-hacker1024@users.sourceforge.net> (raw)
In-Reply-To: <20240713074733.77334-2-luke@ljones.dev>

Hello,

Thanks for this patch. I'm looking at doing something similar for the Zenbook 
Duo 2024 keyboard, which, I'm discovering, is similar to the ROG_NKEY_KEYBOARD 
devices in a number of ways.

Something unique about the Zenbook Duo keyboard is that it has an external 
data-capable USB-C port, meaning it can be used as a regular USB keyboard for 
any device. This means that even if WMI features work, it is highly preferable 
to use HID alternatives where possible in order to allow those features to work 
on any Linux device that the keyboard is plugged in to.

This leads me to my question: Why the DMI check on the HID side? This is a 
little problematic for me, as it blocks functionallity on other devices. Would 
it be sufficient to use quirks only in that case? The important part, to my 
understanding, is that the WMI module disables itself properly, which is still 
covered by its own DMI check.

Are there devices that use the ROG_NKEY_KEYBOARDs where HID doesn't work?

  parent reply	other threads:[~2025-01-12 12:18 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-07-13  7:47 [PATCH v3 0/1] hid-asus: use hid for keyboard brightness control Luke D. Jones
2024-07-13  7:47 ` [PATCH v3 1/1] hid-asus: use hid for brightness control on keyboard Luke D. Jones
2024-08-05 15:03   ` Hans de Goede
2025-01-12 12:18   ` Josh Leivenzon [this message]
2025-01-13  0:37     ` Luke Jones

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20250112121811.434552-1-hacker1024@users.sourceforge.net \
    --to=joshleivenzon@outlook.com \
    --cc=bentiss@kernel.org \
    --cc=corentin.chary@gmail.com \
    --cc=hdegoede@redhat.com \
    --cc=ilpo.jarvinen@linux.intel.com \
    --cc=jikos@kernel.org \
    --cc=linux-input@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=luke@ljones.dev \
    --cc=platform-driver-x86@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).