On Tue, Oct 19, 2021 at 11:44 AM Jiri Kosina <jikos@xxxxxxxxxx> wrote: > > On Tue, 19 Oct 2021, Jiri Kosina wrote: > > > > Rebased onto Linus' tree (sha 926de8c4326c14fcf35f1de142019043597a4fac) > > > Depends on Roderick's patch to add the player LED defines: > > > https://patchwork.kernel.org/project/linux-input/patch/20210908165539.3102929-3-roderick.colenbrander@xxxxxxxx/ > > > > I just got Ack for the joydev part from Dmitry. > > > > v16 is now queued in hid.git#for-5.16/nintendo > > Benjamin noticed that I pushed wrong version of the branch -- the one that > still doesn't contain the LED_FUNCTION_PLAYER[1-5] defines, which I've had > staged here locally, waiting for Pavel's Ack (which is taking time, > unfortunately). > > So please ignore this branch for now, I'll push v2 once that situation is > cleared out. > > CCing Pavel as well here to make him aware of the issues this is causing > all over the place (see .e.g my mail [1] from yesterday). > > [1] https://lore.kernel.org/all/nycvar.YFH.7.76.2110181739310.12554@xxxxxxxxxxxxx/ > I am also jumping on this to raise a concern I recently had with all of the work we do for HID devices in the kernel regarding game controllers. Foreword: this is definitely not a NACK on the series, but more trying to open a discussion to take a step back. The SDL developers recently took a hard turn in how they are handling game controllers on Linux: libsdl now directly talks to hidraw or libusb to handle the controllers in user space, bypassing the kernel processing. Which means that a game working on a recent SDL release already has the PS5 player LEDs ready for instance. I think that part of this work was the integration of the steam client database, which already supports a lot of fix ups for game controllers. So I am starting to wonder if we are not adding dead code in the kernel, because both steam and SDL will disable the input handling of the controllers when they open the hidraw node (IIRC about what was done in this series). I know that Android/Chrome is interested in having actual input nodes created, but are they the sole users of those now? What is the benefit of having game controllers in the kernel? I am opening the discussion and we probably want to bring in the SDL folks here too. But I'd like to have some sort of confirmation that these series which are adding game controllers are not just adding dead code. Cheers, Benjamin
