Thread (6 messages) flat view 6 messages, 2 authors, 2016-11-15

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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help