[PATCH libinput] tablet: Do not create a tablet device for Wacom touchscreens

[Jason Gerecke]

On 8/17/2015 7:21 PM, Peter Hutterer wrote:
> On Mon, Aug 17, 2015 at 05:29:14PM -0700, Jason Gerecke wrote:
>> Similar to the issue mentioned in the commit message of 2365f7d, libwacom
>> assigns both ID_INPUT_TABLET and ID_INPUT_TOUCHSCREEN to touchscreens like
>> the Cintiq 24HDT. This patch ensures that neither touchpads nor touchscreens
>> will accidentally be handled by the tablet code.
> 
> I'm confused, isnt this the same as 10ca39cf80698cedf92?
> 
> Cheers,
>    Peter
> 

Looks like my wires got crossed. I was apparently working off of your
Github repository instead of FDO. Pardon the noise.

Jason
---
Now instead of four in the eights place /
you’ve got three, ‘Cause you added one /
(That is to say, eight) to the two, /
But you can’t take seven from three, /
So you look at the sixty-fours....

>>
>> Signed-off-by: Jason Gerecke <jason.gerecke at wacom.com>
>> ---
>>  src/evdev.c | 6 ++++--
>>  1 file changed, 4 insertions(+), 2 deletions(-)
>>
>> diff --git a/src/evdev.c b/src/evdev.c
>> index bbc3dce..a4bdb9a 100644
>> --- a/src/evdev.c
>> +++ b/src/evdev.c
>> @@ -1495,8 +1495,10 @@ evdev_configure_device(struct evdev_device *device)
>>  	/* libwacom assigns touchpad _and_ tablet to the tablet touch bits,
>>  	   so make sure we don't initialize the tablet interface for the
>>  	   touch device */
>> -	if ((udev_tags & (EVDEV_UDEV_TAG_TABLET|EVDEV_UDEV_TAG_TOUCHPAD)) ==
>> -	     EVDEV_UDEV_TAG_TABLET) {
>> +	if ((udev_tags & (EVDEV_UDEV_TAG_TABLET |
>> +			  EVDEV_UDEV_TAG_TOUCHPAD |
>> +			  EVDEV_UDEV_TAG_TOUCHSCREEN)) ==
>> +	    EVDEV_UDEV_TAG_TABLET) {
>>  		device->dispatch = evdev_tablet_create(device);
>>  		device->seat_caps |= EVDEV_DEVICE_TABLET;
>>  		log_info(libinput,
>> -- 
>> 2.5.0
>>