Re: Subject: Re: [PATCH] Wacom Graphire Bluetooth driver (updated with a few fixes)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 2009-04-28 at 08:34 -0700, Ping wrote:
> Hi Bastien,
> 
> Thank you for putting the Wacom Bluethooth support together.  I have
> two questions (I have to admit that I know nothing about Bluethooth.
> I know something about Wacom devices though :):
> 
> 1.          Should we follow the USB way to blacklist all Wacom
> Bluetooth devices so we don't have to add each product ID later on?
> Something like this should work (correct me if I am wrong):
> 
> { HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_WACOM, HID_ANY_ID) }, 
> 
> instead of 
> 
>  { HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_WACOM,
> USB_DEVICE_ID_WACOM_GRAPHIRE_BLUETOOTH) }, 

We don't know whether the other Bluetooth tablets that appear will use
the same protocol. New devices wouldn't work as tablets, and they
wouldn't work as mice either, because we blacklisted them.

> 2.         Is it possible to merge hid/hid-wacom.c into
> input/tablet/wacom*?  From maintenance point of view, this would be
> easier for people to update the driver with future support (one place
> handles all common data between USB and Bluetooth devices of the
> similar models).

Yes, that's really what should happen, but until somebody actually does
the job, it's unlikely to happen. It's not that big a problem right now,
as the Bluetooth and USB devices don't share a protocol.

> I am glad to see that Wacom Bluetooth support is getting into the main
> stream.  

I'm actually still waiting for the patch to make it into the input
tree...

--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Media Devel]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Linux Wireless Networking]     [Linux Omap]

  Powered by Linux