On Mon, Jun 23, 2014 at 1:57 PM, Benjamin Tissoires
<benjamin.tissoi...@redhat.com> wrote:
> Hi,
>
> This patch series aims at simplifying the handling of Wacom devices from the 
> user
> space point of view. As mentioned in the commit message from 1/5, the pad data
> are interleaved into the stylus input for some devices, and into the touch 
> input
> for others.
>
> It doesn't makes real sense to interleave pad with stylus because pad data are
> not linked to the absolute position of a stylus (or its presence). It is a
> device by itself which should have its own input node.
>
> It will also help the handling of the old Wacom Graphire 4 which shares its
> BTN_LEFT and BTN_RIGHT in its pad and in the mouse.
>
> Again, it's not a problem right now because the user space is currently 
> dealing
> with it, but for the Wayland libinput implementation, having something clear
> will really help.
>
> This patch series does not break current X11 behavior, the only change from 
> the
> user will be that an input device will appear as "Wacom ... Pad pad".
>
> I tried to test/debug it as extensively as possible, but I do not guarantee 
> any
> mistake :(
>
> Tested on:
> - Bamboo 16FG 4x5
> - Bamboo 2FG
> - BambooFun 4x5
> - Bamboo
> - Cintiq 12WX
> - Cintiq 21UX2
> - Cintiq 21UX
> - Cintiq 22HD
> - Cintiq 22HDT
> - Cintiq 24HD
> - DTU2231
> - Graphire3
> - Intuos3 9x12
> - Intuos4 WL
> - Intuos5 touch M
> - Intuos Pro S
>
> Note: this patch series might requesting to apply the patch for input->phys
> I just sent:
> https://patchwork.kernel.org/patch/4404391/
>
>
> Cheers,
> Benjamin
>
> Benjamin Tissoires (5):
>   Input - wacom: create a separate input device for pads
>   Input - wacom: split out the pad device for Intuos/Cintiq
>   Input - wacom: split out the pad device for Bamboos
>   Input - wacom: split out the pad device for DTUS
>   Input - wacom: split out the pad device for Graphire G4 and MO
>
>  drivers/input/tablet/wacom.h     |   2 +
>  drivers/input/tablet/wacom_sys.c |  63 +++++++-
>  drivers/input/tablet/wacom_wac.c | 328 
> ++++++++++++++++++++++++---------------
>  drivers/input/tablet/wacom_wac.h |   2 +
>  4 files changed, 259 insertions(+), 136 deletions(-)
>
> --
> 1.9.0
>

These look fine to me as well (aside from the one minor comment):

Reviewed-by: Jason Gerecke <killert...@gmail.com>

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....

------------------------------------------------------------------------------
Open source business process management suite built on Java and Eclipse
Turn processes into business applications with Bonita BPM Community Edition
Quickly connect people, data, and systems into organized workflows
Winner of BOSSIE, CODIE, OW2 and Gartner awards
http://p.sf.net/sfu/Bonitasoft
_______________________________________________
Linuxwacom-devel mailing list
Linuxwacom-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel

Reply via email to