Re: [PATCH xserver] xwayland: Support for BTN_STYLUS3 kernel events

2018-01-25 Thread Peter Hutterer
On Wed, Jan 24, 2018 at 10:54:53AM -0800, Jason Gerecke wrote:
> On Thu, Nov 9, 2017 at 2:19 PM, Jason Gerecke  wrote:
> > On Tue, Nov 7, 2017 at 2:37 PM, Peter Hutterer  
> > wrote:
> >> On Tue, Nov 07, 2017 at 11:09:44AM -0800, Jason Gerecke wrote:
> >>> BTN_STYLUS3 has been introduced by the Linux 4.15 kernel to report the
> >>> status of the third button present on Wacom's new "Pro Pen 3D" stylus.
> >>> Treat this button like xf86-input-wacom and send a button 8 event
> >>> ("navigate back") when received from Wayland.
> >>>
> >>> Signed-off-by: Jason Gerecke 
> >>> ---
> >>> Note: this patch is pending the formal acceptance of BTN_STYLU3 into
> >>> Linux 4.15. I'll post an update once Jiri merges the patch.
> >>
> >> Reviewed-by: Peter Hutterer , I'll merge this
> >> after your reminder ;)
> >>
> >> Cheers,
> >>Peter
> >>
> >>
> >
> > Jiri's accepted the patch into his 'for-4.15/wacom' branch :)
> >
> > 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
> >
> 
> Just noticed that this doesn't appear to have been merged, so sending
> another poke :)

oops, sorry.

remote: Updating patchwork state for 
https://patchwork.freedesktop.org/project/Xorg/list/
remote: I: patch #186966 updated using rev 
29a5423abd4bc6cb7bb800924a27b5d90951e408.
remote: I: 1 patch(es) updated to state Accepted.
To git+ssh://git.freedesktop.org/git/xorg/xserver
   71348e99a..29a5423ab  master -> master

Cheers,
   Peter

> 
> 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
> 
> >>>  hw/xwayland/xwayland-input.c | 1 +
> >>>  1 file changed, 1 insertion(+)
> >>>
> >>> diff --git a/hw/xwayland/xwayland-input.c b/hw/xwayland/xwayland-input.c
> >>> index 68e365051..4325b4c6e 100644
> >>> --- a/hw/xwayland/xwayland-input.c
> >>> +++ b/hw/xwayland/xwayland-input.c
> >>> @@ -1698,6 +1698,7 @@ tablet_tool_button_state(void *data, struct 
> >>> zwp_tablet_tool_v2 *tool,
> >>>
> >>>  case 0x113: /* BTN_SIDE*/
> >>>  case 0x116: /* BTN_BACK*/
> >>> +case 0x149: /* BTN_STYLUS3 */
> >>>  xbtn = 8;
> >>>  break;
> >>>
> >>> --
> >>> 2.15.0
> >>>
> 
___
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel


Re: [PATCH xserver] xwayland: Support for BTN_STYLUS3 kernel events

2018-01-24 Thread Jason Gerecke
On Thu, Nov 9, 2017 at 2:19 PM, Jason Gerecke  wrote:
> On Tue, Nov 7, 2017 at 2:37 PM, Peter Hutterer  
> wrote:
>> On Tue, Nov 07, 2017 at 11:09:44AM -0800, Jason Gerecke wrote:
>>> BTN_STYLUS3 has been introduced by the Linux 4.15 kernel to report the
>>> status of the third button present on Wacom's new "Pro Pen 3D" stylus.
>>> Treat this button like xf86-input-wacom and send a button 8 event
>>> ("navigate back") when received from Wayland.
>>>
>>> Signed-off-by: Jason Gerecke 
>>> ---
>>> Note: this patch is pending the formal acceptance of BTN_STYLU3 into
>>> Linux 4.15. I'll post an update once Jiri merges the patch.
>>
>> Reviewed-by: Peter Hutterer , I'll merge this
>> after your reminder ;)
>>
>> Cheers,
>>Peter
>>
>>
>
> Jiri's accepted the patch into his 'for-4.15/wacom' branch :)
>
> 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
>

Just noticed that this doesn't appear to have been merged, so sending
another poke :)

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

>>>  hw/xwayland/xwayland-input.c | 1 +
>>>  1 file changed, 1 insertion(+)
>>>
>>> diff --git a/hw/xwayland/xwayland-input.c b/hw/xwayland/xwayland-input.c
>>> index 68e365051..4325b4c6e 100644
>>> --- a/hw/xwayland/xwayland-input.c
>>> +++ b/hw/xwayland/xwayland-input.c
>>> @@ -1698,6 +1698,7 @@ tablet_tool_button_state(void *data, struct 
>>> zwp_tablet_tool_v2 *tool,
>>>
>>>  case 0x113: /* BTN_SIDE*/
>>>  case 0x116: /* BTN_BACK*/
>>> +case 0x149: /* BTN_STYLUS3 */
>>>  xbtn = 8;
>>>  break;
>>>
>>> --
>>> 2.15.0
>>>
___
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel


Re: [PATCH xserver] xwayland: Support for BTN_STYLUS3 kernel events

2017-11-09 Thread Jason Gerecke
On Tue, Nov 7, 2017 at 2:37 PM, Peter Hutterer  wrote:
> On Tue, Nov 07, 2017 at 11:09:44AM -0800, Jason Gerecke wrote:
>> BTN_STYLUS3 has been introduced by the Linux 4.15 kernel to report the
>> status of the third button present on Wacom's new "Pro Pen 3D" stylus.
>> Treat this button like xf86-input-wacom and send a button 8 event
>> ("navigate back") when received from Wayland.
>>
>> Signed-off-by: Jason Gerecke 
>> ---
>> Note: this patch is pending the formal acceptance of BTN_STYLU3 into
>> Linux 4.15. I'll post an update once Jiri merges the patch.
>
> Reviewed-by: Peter Hutterer , I'll merge this
> after your reminder ;)
>
> Cheers,
>Peter
>
>

Jiri's accepted the patch into his 'for-4.15/wacom' branch :)

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

>>  hw/xwayland/xwayland-input.c | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/hw/xwayland/xwayland-input.c b/hw/xwayland/xwayland-input.c
>> index 68e365051..4325b4c6e 100644
>> --- a/hw/xwayland/xwayland-input.c
>> +++ b/hw/xwayland/xwayland-input.c
>> @@ -1698,6 +1698,7 @@ tablet_tool_button_state(void *data, struct 
>> zwp_tablet_tool_v2 *tool,
>>
>>  case 0x113: /* BTN_SIDE*/
>>  case 0x116: /* BTN_BACK*/
>> +case 0x149: /* BTN_STYLUS3 */
>>  xbtn = 8;
>>  break;
>>
>> --
>> 2.15.0
>>
___
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel


Re: [PATCH xserver] xwayland: Support for BTN_STYLUS3 kernel events

2017-11-07 Thread Peter Hutterer
On Tue, Nov 07, 2017 at 11:09:44AM -0800, Jason Gerecke wrote:
> BTN_STYLUS3 has been introduced by the Linux 4.15 kernel to report the
> status of the third button present on Wacom's new "Pro Pen 3D" stylus.
> Treat this button like xf86-input-wacom and send a button 8 event
> ("navigate back") when received from Wayland.
> 
> Signed-off-by: Jason Gerecke 
> ---
> Note: this patch is pending the formal acceptance of BTN_STYLU3 into
> Linux 4.15. I'll post an update once Jiri merges the patch.

Reviewed-by: Peter Hutterer , I'll merge this
after your reminder ;)

Cheers,
   Peter


>  hw/xwayland/xwayland-input.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/hw/xwayland/xwayland-input.c b/hw/xwayland/xwayland-input.c
> index 68e365051..4325b4c6e 100644
> --- a/hw/xwayland/xwayland-input.c
> +++ b/hw/xwayland/xwayland-input.c
> @@ -1698,6 +1698,7 @@ tablet_tool_button_state(void *data, struct 
> zwp_tablet_tool_v2 *tool,
>  
>  case 0x113: /* BTN_SIDE*/
>  case 0x116: /* BTN_BACK*/
> +case 0x149: /* BTN_STYLUS3 */
>  xbtn = 8;
>  break;
>  
> -- 
> 2.15.0
> 
___
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel


[PATCH xserver] xwayland: Support for BTN_STYLUS3 kernel events

2017-11-07 Thread Jason Gerecke
BTN_STYLUS3 has been introduced by the Linux 4.15 kernel to report the
status of the third button present on Wacom's new "Pro Pen 3D" stylus.
Treat this button like xf86-input-wacom and send a button 8 event
("navigate back") when received from Wayland.

Signed-off-by: Jason Gerecke 
---
Note: this patch is pending the formal acceptance of BTN_STYLU3 into
Linux 4.15. I'll post an update once Jiri merges the patch.

 hw/xwayland/xwayland-input.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/hw/xwayland/xwayland-input.c b/hw/xwayland/xwayland-input.c
index 68e365051..4325b4c6e 100644
--- a/hw/xwayland/xwayland-input.c
+++ b/hw/xwayland/xwayland-input.c
@@ -1698,6 +1698,7 @@ tablet_tool_button_state(void *data, struct 
zwp_tablet_tool_v2 *tool,
 
 case 0x113: /* BTN_SIDE*/
 case 0x116: /* BTN_BACK*/
+case 0x149: /* BTN_STYLUS3 */
 xbtn = 8;
 break;
 
-- 
2.15.0

___
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel