On Wed, Mar 18, 2015 at 3:22 PM, Benjamin Tissoires
<benjamin.tissoi...@gmail.com> wrote:
> On Sun, Mar 15, 2015 at 10:01 PM, Peter Hutterer
> <peter.hutte...@who-t.net> wrote:
>> On Fri, Mar 13, 2015 at 11:24:27AM -0400, Benjamin Tissoires wrote:
>>> commit 0cfe113 (Allow PAD only interface without setting up EV_ABS
>>> and/or EV_X/Y) allows xf86-input-wacom to handle Pad only interface but
>>> such interface do not report any events.
>>>
>>> We need to call usbWcmInitPadState() to be able to forward events.
>>>
>>> Signed-off-by: Benjamin Tissoires <benjamin.tissoi...@redhat.com>
>>> ---
>>>  src/wcmUSB.c | 5 +++--
>>>  1 file changed, 3 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/src/wcmUSB.c b/src/wcmUSB.c
>>> index bac3c66..1efcb5b 100644
>>> --- a/src/wcmUSB.c
>>> +++ b/src/wcmUSB.c
>>> @@ -549,7 +549,7 @@ int usbWcmGetRanges(InputInfoPtr pInfo)
>>>               /* may be an expresskey only interface */
>>>               if (ISBITSET(common->wcmKeys, BTN_FORWARD) ||
>>>                   ISBITSET(common->wcmKeys, BTN_0))
>>> -                     return Success;
>>> +                     goto pad_init;
>>>
>>>               xf86Msg(X_ERROR, "%s: no abs bits.\n", pInfo->name);
>>>               return !Success;
>>> @@ -568,7 +568,7 @@ int usbWcmGetRanges(InputInfoPtr pInfo)
>>>               /* may be a PAD only interface */
>>>               if (ISBITSET(common->wcmKeys, BTN_FORWARD) ||
>>>                   ISBITSET(common->wcmKeys, BTN_0))
>>> -                     return Success;
>>> +                     goto pad_init;
>>>
>>>               xf86Msg(X_ERROR, "%s: unable to ioctl xmax value.\n", 
>>> pInfo->name);
>>>               return !Success;
>>> @@ -782,6 +782,7 @@ int usbWcmGetRanges(InputInfoPtr pInfo)
>>>                       common->wcmHWTouchSwitchState = 1;
>>>       }
>>>
>>> +pad_init:
>>>       usbWcmInitPadState(pInfo);
>>>
>>>       return Success;
>>> --
>>> 2.3.1
>>
>> Acked-by: Peter Hutterer <peter.hutte...@who-t.net>

Reviewed-by: Ping Cheng <pi...@wacom.com>

Sorry for the delay...

Ping

------------------------------------------------------------------------------
Dive into the World of Parallel Programming The Go Parallel Website, sponsored
by Intel and developed in partnership with Slashdot Media, is your hub for all
things parallel software development, from weekly thought leadership blogs to
news, videos, case studies, tutorials and more. Take a look and join the 
conversation now. http://goparallel.sourceforge.net/
_______________________________________________
Linuxwacom-devel mailing list
Linuxwacom-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel

Reply via email to