Re: [PATCH v2] HID: udraw: Add support for the uDraw tablet for PS3
From: Bastien Nocera <hadess@hadess.net>
Date: 2016-11-12 17:26:03
Also in:
lkml
On Tue, 2016-11-08 at 14:30 +0100, Bastien Nocera wrote:
On Tue, 2016-11-08 at 14:23 +0100, Jiri Kosina wrote:quoted
<snip>
quoted
Hi Bastien, overall the driver looks ok to me, thanks. Probably the only concern I'd have -- traditionally, we've always been naming drivers by the vendor, as it turned out that over time they get extended by adding support for multiple devices with the same vendor, and usually there is quite some overlap in shared code.In that particular case, I'd rather it got renamed to "hid-udraw-ps3" in this case. The XBox 360 and Wii versions use completely different protocols: http://brandonw.net/udraw/ The XBox 360 version is more likely to be integrated in with other XBox pads, and the Wii version along with other Wii Bluetooth remotes.quoted
Hence I'd propose to call this driver hid-thq (if that's really the vendor's proper name, doesn't seem to be present in latest usb.ids release).THQ went under in 2012, as a result of making too many of those drawing tablets: https://en.wikipedia.org/wiki/THQ I'm fine with it either way. Just let me know which one you prefer.
So, any preference? The sooner you make that choice, the sooner I can send an updated patch and push this into my distribution's kernel. I have a machine waiting on the hardware driver for control :) Cheers