Re: [PATCH] xwayland: Implement smooth scrolling

2015-05-11 Thread Keith Packard
Dima Ryazanov writes: > We don't even need to simulate button clicks; it's done automatically. > This also fixes scrolling in Qt5 apps. Merged. 9ff89a2..81a51a6 master -> master -- -keith signature.asc Description: PGP signature ___ xorg-devel@

Re: [PATCH] xwayland: Implement smooth scrolling

2015-04-30 Thread Peter Hutterer
, wl_fixed_t value) > { > struct xwl_seat *xwl_seat = data; > -int index, count; > -int i, val; > +int index; > const int divisor = 10; > ValuatorMask mask; > > -if (time - xwl_seat->scroll_time > 2000) { > -xwl_se

[PATCH] xwayland: Implement smooth scrolling

2015-04-29 Thread Dima Ryazanov
int i, val; +int index; const int divisor = 10; ValuatorMask mask; -if (time - xwl_seat->scroll_time > 2000) { -xwl_seat->vertical_scroll = 0; -xwl_seat->horizontal_scroll = 0; -} -xwl_seat->scroll_time = time; - -/* FIXME: Need to do pro

Re: [PATCH evdev 1/5] Require multitouch/smooth scrolling dependencies

2015-03-11 Thread Hans de Goede
Hi, All patches in this series look good and are: Reviewed-by: Hans de Goede Regards, Hans On 11-03-15 06:53, Peter Hutterer wrote: Signed-off-by: Peter Hutterer --- configure.ac | 12 ++-- src/evdev.c | 93 src/evdev.h

[PATCH evdev 1/5] Require multitouch/smooth scrolling dependencies

2015-03-10 Thread Peter Hutterer
Signed-off-by: Peter Hutterer --- configure.ac | 12 ++-- src/evdev.c | 93 src/evdev.h | 16 --- 3 files changed, 2 insertions(+), 119 deletions(-) diff --git a/configure.ac b/configure.ac index 46f3dc7..e53177e 100644 -

[PATCH] evdev: smooth scrolling support for wheel emulation

2014-01-11 Thread Max Schwarz
Disables the button click generation (EvdevWheelEmuInertia) and generates scroll valuator events directly. Signed-off-by: Max Schwarz --- I tried to support turning wheel emulation on/off at runtime. This involves calling InitValuatorClassDeviceStruct() again to add the new valuators or remove th

Re: Smooth scrolling and button emulation

2012-10-04 Thread Peter Hutterer
On Thu, Oct 04, 2012 at 01:16:35PM +0200, Michal Suchanek wrote: > Hello, > > On 4 October 2012 11:23, Sascha Hlusiak wrote: > > >> I'm not sure what you mean by "Followed by smooth scrolling". Once you > >> enable smooth scrolling, you don

Re: Smooth scrolling and button emulation

2012-10-04 Thread Michal Suchanek
Hello, On 4 October 2012 11:23, Sascha Hlusiak wrote: >> I'm not sure what you mean by "Followed by smooth scrolling". Once you >> enable smooth scrolling, you don't send button 4/5 presses anymore, only >> the valuator events. the rest is handled in the s

Re: Smooth scrolling and button emulation

2012-10-04 Thread Sascha Hlusiak
erate a > > XI1/legacy scroll button down/up sequence. Is there a way to reset that > > server internal delta to 0 after scrolling stops? It seems that smooth > > scrolling might leave that value at 99, which makes it impossible for > > me to predict when the next legacy button

Re: Smooth scrolling and button emulation

2012-10-03 Thread Peter Hutterer
On Wed, Oct 03, 2012 at 07:17:08PM +0200, Sascha Hlusiak wrote: > Hi Peter, > > I'd like to implement smooth scrolling in the joystick input module. By > calling > > SetScrollValuator(pDevice, 2, SCROLL_TYPE_VERTICAL, 100, 0); > > I can set the scroll resolution

Smooth scrolling and button emulation

2012-10-03 Thread Sascha Hlusiak
Hi Peter, I'd like to implement smooth scrolling in the joystick input module. By calling SetScrollValuator(pDevice, 2, SCROLL_TYPE_VERTICAL, 100, 0); I can set the scroll resolution of the valuator 2 to 100. As far as I understand, I now can send valuator movement that XI2 applica

Re: [PATCH synaptics] Fix scoll increment for non-smooth scrolling (#46617)

2012-03-16 Thread Chase Douglas
018, the delta was incremented by > the delta only, scroll dist was used as increment in the smooth scrolling > information. On non-smooth-scrolling servers, the driver would now send > too many events. Restore the old behaviour by dividing by the scroll > distance before entering the but

[PATCH synaptics] Fix scoll increment for non-smooth scrolling (#46617)

2012-03-01 Thread Peter Hutterer
as increment in the smooth scrolling information. On non-smooth-scrolling servers, the driver would now send too many events. Restore the old behaviour by dividing by the scroll distance before entering the button click loop. X.Org Bug 46617 <http://bugs.freedesktop.org/show_bug.cgi?id=46617>

[PATCH synaptics 1/2] Scroll: Initial smooth scrolling support

2011-10-18 Thread Peter Hutterer
From: Daniel Stone Post smooth-scrolling events through the new X server API when available, rather than legacy jerky button events. [Amended to use the final smooth scrolling API] Signed-off-by: Daniel Stone Amendments-by: Peter Hutterer Signed-off-by: Peter Hutterer --- src/synaptics.c

Re: [PATCH evdev, v2] Support smooth scrolling on wheel emulation

2011-10-12 Thread Peter Hutterer
On Thu, Oct 06, 2011 at 11:45:58PM +0200, Max Schwarz wrote: > Hi Peter, > > > you could make the support compiled in, but not compiled _out_. so even if > > you HAVE_SMOOTH_SCROLLING, the old bits are ready to go when enabled. > > if no smooth scrolling axis is otherwi

Re: [PATCH evdev, v2] Support smooth scrolling on wheel emulation

2011-10-06 Thread Max Schwarz
Hi Peter, > you could make the support compiled in, but not compiled _out_. so even if > you HAVE_SMOOTH_SCROLLING, the old bits are ready to go when enabled. > if no smooth scrolling axis is otherwise present on the device, just post > button events as previously. Well, the proper

Re: [PATCH evdev, v2] Support smooth scrolling on wheel emulation

2011-10-04 Thread Peter Hutterer
On Tue, Oct 04, 2011 at 09:53:56PM +0200, Max Schwarz wrote: > This adds support for the new smooth-scrolling valuator system to > the wheel emulation code. > > Caveats: > - Enabling wheel emulation at runtime does not work if the device >does not provide the necessary

Re: [PATCH evdev, v2] Support smooth scrolling on wheel emulation

2011-10-04 Thread Peter Hutterer
On Tue, Oct 04, 2011 at 09:53:56PM +0200, Max Schwarz wrote: > This adds support for the new smooth-scrolling valuator system to > the wheel emulation code. > > Caveats: > - Enabling wheel emulation at runtime does not work if the device >does not provide the nece

[PATCH evdev, v2] Support smooth scrolling on wheel emulation

2011-10-04 Thread Max Schwarz
This adds support for the new smooth-scrolling valuator system to the wheel emulation code. Caveats: - Enabling wheel emulation at runtime does not work if the device does not provide the necessary axes already. - Horizontal scrolling is always reported on the REL_HWHEEL axis, ignoring a

Re: [PULL] XI 2.1 - raw events and smooth scrolling

2011-10-04 Thread Daniel Stone
Hi, On 3 October 2011 11:50, Max Schwarz wrote: >>       Input: Add smooth-scrolling support to GetPointerEvents > There's a subtle bug on that one: Old-style scroll button presses create > inverted emulated presses (and maybe inverted valuator events, but I don't > kn

Re: [PULL] XI 2.1 - raw events and smooth scrolling

2011-10-03 Thread Keith Packard
.c:DeliverRawEvent > Support (and require) XI 2.1 > input: deliver raw events unconditionally for XI 2.1 clients. > dix: use 'rc' for return code in DeliverRawEvent > Merge branch 'raw-events' into next > Input: Add smooth-scrolling

Re: [PULL] XI 2.1 - raw events and smooth scrolling

2011-10-03 Thread Max Schwarz
Hi Peter, > Input: Add smooth-scrolling support to GetPointerEvents There's a subtle bug on that one: Old-style scroll button presses create inverted emulated presses (and maybe inverted valuator events, but I don't know which direction is 'up' or 'down'

Re: [PULL] XI 2.1 - raw events and smooth scrolling

2011-10-02 Thread Peter Hutterer
I've added Alan's compiler warning fix to this branch. New tip: 524e5445c0b6df5247d4aac5368470bb89ef4080 On Fri, Sep 30, 2011 at 09:51:38AM +1000, Peter Hutterer wrote: > This is the now finally reviewed smooth scrolling branch and the raw events, > along with the new input ABI ch

[PATCH evdev] Support smooth scrolling on wheel emulation

2011-10-02 Thread Max Schwarz
This adds support for the new smooth-scrolling valuator system to the wheel emulation code. Caveats: - Enabling wheel emulation at runtime does not work if the device does not provide the necessary axes already. - Horizontal scrolling is always reported on the REL_HWHEEL axis, ignoring a

[PULL] XI 2.1 - raw events and smooth scrolling

2011-09-29 Thread Peter Hutterer
This is the now finally reviewed smooth scrolling branch and the raw events, along with the new input ABI changes (which are spread over a set of commits but at least this way they're in one merge only) Commit f32c827d513c44f07e1d0fbcc0c96cef18c9a4d9 The following changes since c

[PATCH v3] Input: Add smooth-scrolling support to GetPointerEvents

2011-09-28 Thread Peter Hutterer
scroll_axis = pDev->valuator->v_scroll_axis; +ValuatorMask mask; +ValuatorMask scroll; +int i; /* refuse events from disabled devices */ if (!pDev->enabled) @@ -1204,8 +1301,73 @@ GetPointerEvents(InternalEvent *events, DeviceIntPtr pDev, int type, events = UpdateFromMas

Re: [PATCH v2] Input: Add smooth-scrolling support

2011-09-28 Thread Peter Hutterer
s in, > no? By the time we get here, we're in absolute coordinates. So what we need to do is if (!valuator_mask_isset(last, axis)) valuator_mask_set(last, axis, 0); This way, the very first event on this axis is handled as you described and for any future events we're fine. > >

Re: [PATCH evdev 1/1] Support smooth scrolling on REL_WHEEL, REL_HWHEEL and REL_DIAL

2011-09-27 Thread Daniel Stone
Hi, On 23 September 2011 00:14, Peter Hutterer wrote: > [general reassurance] Reviewed-by: Daniel Stone Cheers, Daniel ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/lis

Re: [PATCH v2] Input: Add smooth-scrolling support

2011-09-27 Thread Daniel Stone
is, we won't generate anything until the second event comes in, no? > @@ -1193,7 +1288,12 @@ GetPointerEvents(InternalEvent *events, DeviceIntPtr > pDev, int type, >                  int buttons, int flags, const ValuatorMask *mask_in) >  { >     CARD32 ms = GetTimeInMillis(); > -    in

[PATCH v2] Input: Add smooth-scrolling support

2011-09-22 Thread Peter Hutterer
I found a bug in the stack of scrolling patches that caused a server lockup. Introduced by a recent rebase, but I found a few other issues and incorporated daniels' comments into this patch and squashed them all together. This is just one big patch to add smooth scrolling support instead o

Re: [PATCH evdev 1/1] Support smooth scrolling on REL_WHEEL, REL_HWHEEL and REL_DIAL

2011-09-22 Thread Peter Hutterer
On Thu, Sep 22, 2011 at 11:05:42AM +0100, Daniel Stone wrote: > Hi, > > On Mon, Aug 22, 2011 at 03:35:15PM +1000, Peter Hutterer wrote: > > +#ifdef HAVE_SMOOTH_SCROLLING > > +if (axis == REL_WHEEL) > > +SetScrollValuator(device, axnum, SCROLL_TYPE_VERTICAL, 1.0, > > SCROLL_FLA

Re: [PATCH 0/a few] smooth scrolling support

2011-09-22 Thread Daniel Stone
On Mon, Aug 22, 2011 at 03:35:14PM +1000, Peter Hutterer wrote: > This is the patchset for the revised smooth scrolling support. The basic > principle stays the same (valuator information instead of button > clicks), the protocol itself chanages a bit. Instead of a dedicated virtual >

Re: [PATCH evdev 1/1] Support smooth scrolling on REL_WHEEL, REL_HWHEEL and REL_DIAL

2011-09-22 Thread Daniel Stone
Hi, On Mon, Aug 22, 2011 at 03:35:15PM +1000, Peter Hutterer wrote: > +#ifdef HAVE_SMOOTH_SCROLLING > +if (axis == REL_WHEEL) > +SetScrollValuator(device, axnum, SCROLL_TYPE_VERTICAL, 1.0, > SCROLL_FLAG_PREFERRED); > +else if (axis == REL_DIAL) > +SetScroll

[PULL RESEND] input -next, smooth scrolling, screen crossing fixes

2011-09-22 Thread Peter Hutterer
This is a resend of the pull request http://lists.freedesktop.org/archives/xorg-devel/2011-September/024938.html This time all three (smooth scrolling, -next and two-screen-coordinates) in one branch. Neither smooth scrolling nor two-screen-coordinates has reviews but I've stopped caring no

[PATCH 2/5] Input: Add smooth-scrolling support to GetPointerEvents

2011-09-08 Thread Peter Hutterer
mask_in); + +valuator_mask_copy(&mask, mask_in); + +/* Turn a scroll button press into a smooth-scrolling event if necessary. */ +if (type == ButtonPress) +{ +double val, adj; +int axis; + +switch (buttons) { +case 4: +adj =

[PULL] smooth-scrolling server support

2011-09-08 Thread Peter Hutterer
This patchset adds smooth scrolling support to the server. Unfortunatley, except for Daniel's patch the others are still unreviewed. Pull at your leisure. This branch merges on top of -next, no conflicts. commit 53f7e75ad6cfc01270ce01dfa79a4ab80eeaac3f Cheers, Peter The following ch

[PATCH 2/5] Input: Add smooth-scrolling support to GetPointerEvents

2011-09-04 Thread Peter Hutterer
mask_in); + +valuator_mask_copy(&mask, mask_in); + +/* Turn a scroll button press into a smooth-scrolling event if necessary. */ +if (type == ButtonPress) +{ +double val, adj; +int axis; + +switch (buttons) { +case 4: +adj =

[PATCH 0/5] smooth scrolling patches

2011-09-04 Thread Peter Hutterer
These need some review lovin'. Patches 1-3 were on the list already, please go through them so I can send my next branch to keith. Cheers, Peter ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http:

[PATCH xserver 3/4] Input: Add smooth-scrolling support to GetPointerEvents

2011-08-21 Thread Peter Hutterer
alEvent *events, DeviceIntPtr pDev, int type, events = UpdateFromMaster(events, pDev, DEVCHANGE_POINTER_EVENT, &num_events); -num_events += fill_pointer_events(events, pDev, type, buttons, ms, flags, - mask_in); + +valuator_

[PATCH inputproto 1/2] Document smooth-scrolling support

2011-08-21 Thread Peter Hutterer
From: Daniel Stone Two new axes are added to support smooth scrolling: Rel Vert Scroll and Rel Horiz Scroll. Cumulative values of 1.0 with either magnitude on these axes are considered to be equivalent to one legacy ButtonPress event on the scroll buttons. Signed-off-by: Daniel Stone Reviewed

[PATCH evdev 1/1] Support smooth scrolling on REL_WHEEL, REL_HWHEEL and REL_DIAL

2011-08-21 Thread Peter Hutterer
Automatic smooth scrolling setup for these axes, with REL_WHEEL and REL_DIAL both mapping into vscrolling. REL_WHEEL is the preferred axis. Mouse wheel emulation is not yet updated for smooth scrolling. Signed-off-by: Peter Hutterer --- src/evdev.c | 17 - src/evdev.h |4

[PATCH 0/a few] smooth scrolling support

2011-08-21 Thread Peter Hutterer
This is the patchset for the revised smooth scrolling support. The basic principle stays the same (valuator information instead of button clicks), the protocol itself chanages a bit. Instead of a dedicated virtual axis, mark those axes the hardware supplies as scrolling axis. The option of a

Re: [PATCH RFC inputproto v2] Use flags for smooth scrolling, not axis labels.

2011-08-16 Thread Daniel Stone
Hi, On Tue, Aug 16, 2011 at 10:00:41AM +1000, Peter Hutterer wrote: > diff --git a/XI2proto.h b/XI2proto.h > index 8977e87..03ead01 100644 > --- a/XI2proto.h > +++ b/XI2proto.h > @@ -186,9 +186,11 @@ typedef struct { > uint8_t mode; /**< ModeRelative or ModeAbsolute */ > ui

Re: [PATCH RFC inputproto] Use flags for smooth scrolling, not axis labels.

2011-08-16 Thread Daniel Stone
On Mon, Aug 15, 2011 at 09:40:55PM +1000, Peter Hutterer wrote: > On 15/08/11 21:09 , Daniel Stone wrote: > >On Mon, Aug 15, 2011 at 03:28:03PM +1000, Peter Hutterer wrote: > >>+The behavior of the server if there is more than one VertScroll or more > >>than > >>+one HorizScroll axes on the same d

Re: [PATCH smooth scrolling] dix: Don't reset the scrolling valuators when emulating events

2011-08-15 Thread Peter Hutterer
ranch, it won't apply to > master. > > dix/getevents.c | 41 + > 1 files changed, 25 insertions(+), 16 deletions(-) [...] > @@ -1259,11 +1268,14 @@ GetPointerEvents(InternalEvent *events, DeviceIntPtr > pDev, int type, > num_ev

[PATCH RFC inputproto v2] Use flags for smooth scrolling, not axis labels.

2011-08-15 Thread Peter Hutterer
Using labels only to mark smooth scrolling axes disallows scrolling from hardware events (e.g. a mouse wheel). If those axes are marked as scrolling axes instead, the clients lose information which hardware axis this event corresponds to. For example, on Wacom devices, the client can benefit from

Re: [PATCH RFC inputproto] Use flags for smooth scrolling, not axis labels.

2011-08-15 Thread Peter Hutterer
On 15/08/11 21:09 , Daniel Stone wrote: Hi, Looks good to me generally, but: On Mon, Aug 15, 2011 at 03:28:03PM +1000, Peter Hutterer wrote: +Newer X servers may provide scrolling information through specific +valuators to provide scroll events with more precision than the button +events. Valua

Re: [PATCH RFC inputproto] Use flags for smooth scrolling, not axis labels.

2011-08-15 Thread Daniel Stone
Hi, Looks good to me generally, but: On Mon, Aug 15, 2011 at 03:28:03PM +1000, Peter Hutterer wrote: > +Newer X servers may provide scrolling information through specific > +valuators to provide scroll events with more precision than the button > +events. Valuators for axes sending scrolling infor

[PATCH RFC inputproto] Use flags for smooth scrolling, not axis labels.

2011-08-14 Thread Peter Hutterer
After starting to write some more documentation on the smooth scrolling and trying to implement evdev support for it, I ran into an issue. evdev currently exports axes as the kernel labels them, allowing for clients to know _what_ an axis is (as opposed to XI 1.x where axis 3 was pressure only by

[PATCH smooth scrolling] dix: Don't reset the scrolling valuators when emulating events

2011-08-14 Thread Peter Hutterer
= (int)pDev->last.valuators[h_scroll_axis]; + /* First fill out the original event set, with smooth-scrolling axes. */ nev_tmp = fill_pointer_events(events, pDev, type, buttons, ms, flags, &mask); @@ -1259,11 +1268,14 @@ GetPointerEvents(InternalEve

Re: [PATCH inputproto 4/4] Document smooth-scrolling support

2011-08-14 Thread Peter Hutterer
y, this doesn't seem to > exist on master) > > >         XI 2.x event for legacy client support, and that this event should > > -        be ignored if the client listens for these events. > > +        be ignored if the client listens for these events.  This flag is &

Re: [PATCH inputproto 4/4] Document smooth-scrolling support

2011-08-12 Thread Rui Tiago Cação Matos
hese events.  This flag is > +        set on scroll ButtonPress and RawButtonPress events (buttons 4, 5, 6 > +        and 7) if a smooth-scrolling event on the Rel Vert Scroll or > +        Rel Horiz Scroll axes was also generated. Rui ___ xor

Re: [PATCH inputproto 4/4] Document smooth-scrolling support

2011-08-12 Thread Rui Tiago Cação Matos
2011/8/12 Rui Tiago Cação Matos : >>         PointerEmulated signaps that the event has been emulated from another >                          ^^^ >                          typo (interestingly, this doesn't seem to > exist on master) Gah, it's from the previous patch. I shouldn't read my mail

[PATCH inputproto 4/4] Document smooth-scrolling support

2011-08-11 Thread Peter Hutterer
From: Daniel Stone Two new axes are added to support smooth scrolling: Rel Vert Scroll and Rel Horiz Scroll. Cumulative values of 1.0 with either magnitude on these axes are considered to be equivalent to one legacy ButtonPress event on the scroll buttons. Signed-off-by: Daniel Stone Reviewed

Re: [PATCH v2 28/28] Input: Add smooth-scrolling support to GetPointerEvents

2011-06-24 Thread Daniel Stone
(well, the unreviewed ones) > > > > Thanks. Do you want everything but smooth-scrolling in my for-peter > > branch, or pend for later, or? > > yes, but on top of master please. not on top of inputproto-2.1-devel. Let's > see if keith pulls it ;) I've pushed the followi

Re: [PATCH v2 28/28] Input: Add smooth-scrolling support to GetPointerEvents

2011-06-23 Thread Peter Hutterer
hing but v_scroll_axis and h_scroll_axis > > from the local stack anyway, why not make the while loop a: > > int emulate_scroll_button_events(events, device, hscroll, vscroll) > > Ha, sure. > > > Reviewed-by: Peter Hutterer for the lot though > > (well, the unrev

Re: [PATCH inputproto 2/2] Document smooth-scrolling support

2011-06-23 Thread Daniel Stone
think we can just put that one in the 'well, don't do that' > > basket, to be honest. > > you can't choose which axes you listen to, you get all of them. with smooth > scrolling, a 1.x/2.0 client (built before the addition of the scrolling > labels) would see m

Re: [PATCH inputproto 2/2] Document smooth-scrolling support

2011-06-23 Thread Peter Hutterer
On Thu, Jun 23, 2011 at 10:50:41AM +0100, Daniel Stone wrote: > Hi, > > On Thu, Jun 23, 2011 at 10:44:44AM +1000, Peter Hutterer wrote: > > On Thu, Jun 09, 2011 at 06:26:01PM +0100, Daniel Stone wrote: > > > Two new axes are added to support smooth scrolling: Rel Vert Sc

[PATCH v3 23/23] Scroll: Initial smooth scrolling support

2011-06-23 Thread Daniel Stone
Post smooth-scrolling events through the new X server API when available, rather than legacy jerky button events. Signed-off-by: Daniel Stone --- src/synaptics.c| 54 +-- src/synapticsstr.h | 11 ++ 2 files changed, 62 insertions

[PATCH v3 22/23] Scroll: Prepare ScrollData for smooth scrolling

2011-06-23 Thread Daniel Stone
2 files changed, 52 insertions(+), 85 deletions(-) v3: Move ScrollData into priv->scroll to fix non-smooth-scrolling with small-delta packets. Fixed indentation. Variables renamed. Store values normalised to 1.0 per button event, as with old ScrollData, rather than pixel

[PATCH v3 0/23] Synaptics accel changes and smooth scrolling

2011-06-23 Thread Daniel Stone
'v3' in the subject is new; patches 2, 3, 4, 8, 9, 10, 11, 12, 15, 18, 20, 22, and 23 are still missing review. #23 is still subject to change as I work out how to implement the relative-but-values-don't-carry-over axis type that Simon pointed out during xserver smooth-scrolling rev

Re: [PATCH inputproto 2/2] Document smooth-scrolling support

2011-06-23 Thread Daniel Stone
Hi, On Thu, Jun 23, 2011 at 10:44:44AM +1000, Peter Hutterer wrote: > On Thu, Jun 09, 2011 at 06:26:01PM +0100, Daniel Stone wrote: > > Two new axes are added to support smooth scrolling: Rel Vert Scroll and > > Rel Horiz Scroll. Cumulative values of 1.0 with either magnitude on

Re: [PATCH v2 28/28] Input: Add smooth-scrolling support to GetPointerEvents

2011-06-23 Thread Daniel Stone
the while loop a: > int emulate_scroll_button_events(events, device, hscroll, vscroll) Ha, sure. > Reviewed-by: Peter Hutterer for the lot though > (well, the unreviewed ones) Thanks. Do you want everything but smooth-scrolling in my for-peter branch, or pend for

Re: [PATCH v2 28/28] Input: Add smooth-scrolling support to GetPointerEvents

2011-06-22 Thread Peter Hutterer
; if (!pDev->enabled) > @@ -1219,8 +1226,101 @@ GetPointerEvents(InternalEvent *events, DeviceIntPtr > pDev, int type, > > events = UpdateFromMaster(events, pDev, DEVCHANGE_POINTER_EVENT, >&num_events); > -num_events += fill

Re: [PATCH inputproto 2/2] Document smooth-scrolling support

2011-06-22 Thread Peter Hutterer
On Thu, Jun 09, 2011 at 06:26:01PM +0100, Daniel Stone wrote: > Two new axes are added to support smooth scrolling: Rel Vert Scroll and > Rel Horiz Scroll. Cumulative values of 1.0 with either magnitude on > these axes are considered to be equivalent to one legacy ButtonPress >

[PULL inputproto] smooth-scrolling (was: Re: [PATCH inputproto 2/2] Document smooth-scrolling support)

2011-06-22 Thread Daniel Stone
Hi, On Mon, Jun 20, 2011 at 04:28:17PM +1000, Peter Hutterer wrote: > Reviewed-by: Peter Hutterer otherwise though Thanks for this, I've fixed your comments from both patches, added your Reviewed-by, and pushed to my p.fd.o smooth-scrolling branch, which is on top of your inputproto-2

Re: [PATCH inputproto 2/2] Document smooth-scrolling support

2011-06-19 Thread Peter Hutterer
On Thu, Jun 09, 2011 at 06:26:01PM +0100, Daniel Stone wrote: > Two new axes are added to support smooth scrolling: Rel Vert Scroll and > Rel Horiz Scroll. Cumulative values of 1.0 with either magnitude on > these axes are considered to be equivalent to one legacy ButtonPress >

Re: [PATCH 15/15] Initial smooth scrolling support

2011-06-14 Thread Peter Hutterer
On Tue, Jun 14, 2011 at 05:35:22PM +0100, Daniel Stone wrote: > Hi, > > On Tue, Jun 14, 2011 at 03:44:44PM +1000, Peter Hutterer wrote: > > On Thu, Jun 09, 2011 at 08:57:36PM +0100, Daniel Stone wrote: > > > -#include "synaptics.h" > > > -#include "synapticsstr.h" > > > -#include "synaptics-proper

[PATCH synaptics 20/21] Prepare ScrollData for smooth scrolling

2011-06-14 Thread Daniel Stone
Convert ScrollData from up/down/left/right members for button presses, to more fine-grained x and y members. Signed-off-by: Daniel Stone Reviewed-by: Peter Hutterer --- src/synaptics.c | 119 ++ 1 files changed, 49 insertions(+), 70 deletions

[PATCH synaptics v2 21/21] Initial smooth scrolling support

2011-06-14 Thread Daniel Stone
Post smooth-scrolling events through the new X server API when available, rather than legacy jerky button events. Signed-off-by: Daniel Stone --- src/synaptics.c| 54 +-- src/synapticsstr.h | 11 ++ 2 files changed, 62 insertions

[PATCH synaptics v2 00/21] Predictable motion/accel, smooth scrolling

2011-06-14 Thread Daniel Stone
Hi, The attached v2 patchset (changed/new patches are marked with 'v2' in the subject line) should take care of pretty much all the review comments from Peter & Simon, thanks. Patches #2, #3, #4, #8, #10, #11, #12, #13, #16, and #21 still need review. Cheers, Daniel _

Re: [PATCH 15/15] Initial smooth scrolling support

2011-06-14 Thread Daniel Stone
Hi, On Tue, Jun 14, 2011 at 03:44:44PM +1000, Peter Hutterer wrote: > On Thu, Jun 09, 2011 at 08:57:36PM +0100, Daniel Stone wrote: > > -#include "synaptics.h" > > -#include "synapticsstr.h" > > -#include "synaptics-properties.h" > > - > > #if GET_ABI_MAJOR(ABI_XINPUT_VERSION) >= 7 > > #include

Re: [PATCH synaptics 0/15] Predictable motion, better accel & smooth-scrolling

2011-06-13 Thread Peter Hutterer
le: we > use hardware time, rather than the time we received the event in the > SIGIO handler, to estimate motion; we added a better acceleration > function; we added some strict filters to get rid of erroneous motion > and make palm detection more reliable, and lastly, we also added smooth

Re: [PATCH 15/15] Initial smooth scrolling support

2011-06-13 Thread Peter Hutterer
On Thu, Jun 09, 2011 at 08:57:36PM +0100, Daniel Stone wrote: > Post smooth-scrolling events through the new X server API when > available, rather than legacy jerky button events. > > Signed-off-by: Daniel Stone > --- > src/syn

Re: [PATCH 14/15] Prepare ScrollData for smooth scrolling

2011-06-13 Thread Peter Hutterer
On Thu, Jun 09, 2011 at 08:57:35PM +0100, Daniel Stone wrote: > Convert ScrollData from up/down/left/right members for button presses, > to more fine-grained x and y members. > > Signed-off-by: Daniel Stone > --- Reviewed-by: Peter Hutterer Cheers, Peter > src/synaptics.c | 119 ++

Re: [PATCH 27/27] Input: Add smooth-scrolling support to GetPointerEvents

2011-06-10 Thread Daniel Stone
On Fri, Jun 10, 2011 at 03:57:53PM +1000, Peter Hutterer wrote: > On Thu, Jun 09, 2011 at 04:19:14PM +0100, Daniel Stone wrote: > > > a bit of a side-issue with this approach is that the scroll events lose > > > valuator information. we hit that one with the wacom driver where one of > > > my > >

Re: [PATCH 27/27] Input: Add smooth-scrolling support to GetPointerEvents

2011-06-09 Thread Peter Hutterer
On Thu, Jun 09, 2011 at 04:19:14PM +0100, Daniel Stone wrote: > > a bit of a side-issue with this approach is that the scroll events lose > > valuator information. we hit that one with the wacom driver where one of my > > patches started submitting button events with first/num_valuators as 0/0. it

[PATCH 15/15] Initial smooth scrolling support

2011-06-09 Thread Daniel Stone
Post smooth-scrolling events through the new X server API when available, rather than legacy jerky button events. Signed-off-by: Daniel Stone --- src/synaptics.c| 58 +-- src/synapticsstr.h | 11 + 2 files changed, 62 insertions

[PATCH 14/15] Prepare ScrollData for smooth scrolling

2011-06-09 Thread Daniel Stone
Convert ScrollData from up/down/left/right members for button presses, to more fine-grained x and y members. Signed-off-by: Daniel Stone --- src/synaptics.c | 119 ++ 1 files changed, 49 insertions(+), 70 deletions(-) diff --git a/src/synapti

[PATCH synaptics 0/15] Predictable motion, better accel & smooth-scrolling

2011-06-09 Thread Daniel Stone
handler, to estimate motion; we added a better acceleration function; we added some strict filters to get rid of erroneous motion and make palm detection more reliable, and lastly, we also added smooth scrolling support (of course). The second part is rather more invasive, and I'll follow up wi

[PATCH inputproto 2/2] Document smooth-scrolling support

2011-06-09 Thread Daniel Stone
Two new axes are added to support smooth scrolling: Rel Vert Scroll and Rel Horiz Scroll. Cumulative values of 1.0 with either magnitude on these axes are considered to be equivalent to one legacy ButtonPress event on the scroll buttons. Signed-off-by: Daniel Stone --- specs/XI2proto.txt

[PATCH v2 28/28] Input: Add smooth-scrolling support to GetPointerEvents

2011-06-09 Thread Daniel Stone
&num_events); -num_events += fill_pointer_events(events, pDev, type, buttons, ms, flags, - mask_in); + +valuator_mask_copy(&mask, mask_in); + +/* Turn a scroll button press into a smooth-scrolling event if necessary.

[PATCH xserver v2 0/27] Input cleanup and smooth-scrolling support

2011-06-09 Thread Daniel Stone
esolve. > > There are quite a few patches that could go in right now to give them a few > months of testing before we merge the new features. Interested in that? > > In fact, all except the POINTER_EMULATED patch and the actual integration of > the smooth scrolling. None of

Re: [PATCH 27/27] Input: Add smooth-scrolling support to GetPointerEvents

2011-06-09 Thread Daniel Stone
v_scroll_axis = i; > > +} > > this may be better cached in the DeviceIntRec instead of recalculated on > every single pointer event. Done - I was worried about extra DeviceIntRec usage, but eh. > > + > > +/* Turn a scroll button press into a smooth-s

Re: [PATCH 0/27] Input cleanup and smooth scrolling support

2011-06-06 Thread Peter Hutterer
On Fri, Jun 03, 2011 at 03:54:58PM +0100, Daniel Stone wrote: > There's a few patches in this series, destined for 1.12. Most notable > is the last (but arguably least pleasant) of the series, which adds > support for smooth scrolling by way of new scroll valuators, which > emul

Re: [PATCH 27/27] Input: Add smooth-scrolling support to GetPointerEvents

2011-06-06 Thread Peter Hutterer
+ > +valuator_mask_copy(&mask, mask_in); > + > +/* Find the vertical and horizontal scroll axes, if any. */ > +for (i = 0; i < pDev->valuator->numAxes; i++) > +{ > +if (h_scroll_label && pDev->valuator->axes[i].label == > h_scroll_labe

Re: [PATCH 27/27] Input: Add smooth-scrolling support to GetPointerEvents

2011-06-06 Thread Peter Hutterer
event for legacy client support, and that this event should > be ignored if the client listens for these events. This flag will > be set on scroll ButtonPress events (buttons 4, 5, 6 and 7) if a > smooth-scrolling event on the Rel Vert Scroll or Rel Horiz Scroll > ax

Re: [PATCH 27/27] Input: Add smooth-scrolling support to GetPointerEvents

2011-06-06 Thread Daniel Stone
d round of patches. Here's the applicable text: PointerEmulated means that the event has been emulated from another XI 2.x event for legacy client support, and that this event should be ignored if the client listens for these events. This flag will be set on scroll ButtonPr

Re: [PATCH 27/27] Input: Add smooth-scrolling support to GetPointerEvents

2011-06-05 Thread Simon Thum
On 06/03/2011 05:00 PM, Daniel Stone wrote: > For scroll wheel support, we used to send buttons 4/5 and 6/7 for > horizontal/vertical positive/negative scroll events. For touchpads, we > really want more fine-grained scroll values. GetPointerEvents now > accepts both old-school scroll button pres

Re: [PATCH 0/27] Input cleanup and smooth scrolling support

2011-06-03 Thread Daniel Stone
On Fri, Jun 03, 2011 at 03:54:58PM +0100, Daniel Stone wrote: > There's a few patches in this series, destined for 1.12. Most notable > is the last (but arguably least pleasant) of the series, which adds > support for smooth scrolling by way of new scroll valuators, which > emul

[PATCH 27/27] Input: Add smooth-scrolling support to GetPointerEvents

2011-06-03 Thread Daniel Stone
r->axes[i].label == v_scroll_label) +v_scroll_axis = i; +} + +/* Turn a scroll button press into a smooth-scrolling event. */ +if (type == ButtonPress && +((v_scroll_axis != -1 && (buttons == 4 || buttons == 5)) || + (h_scroll_axis

[PATCH 0/27] Input cleanup and smooth scrolling support

2011-06-03 Thread Daniel Stone
Hi all, There's a few patches in this series, destined for 1.12. Most notable is the last (but arguably least pleasant) of the series, which adds support for smooth scrolling by way of new scroll valuators, which emulate button events, and vice-versa for older drivers too. Along the wa

Re: Smooth scrolling again

2010-11-18 Thread Simon Thum
On 11/17/10 22:26, Max Schwarz wrote: > I would propose specific methods for changing the resolution, axes ranges, > etc., and one final call to send out events. Sounds fine. Simon ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.

Re: Smooth scrolling again

2010-11-17 Thread Max Schwarz
On Wednesday 17 November 2010 at 23:55:20, Simon Thum wrote: > > if possible, wrap it. it's much easier to work around issues than when > > each driver just mangles the struct directly. But I don't think you need > > a new call here, just re-using the existing InitValuatorAxisStruct > > should do.

Re: Smooth scrolling again

2010-11-17 Thread Simon Thum
On 11/14/10 22:52, Peter Hutterer wrote: > On Sun, Nov 14, 2010 at 06:51:43PM +0100, Max Schwarz wrote: Are the drivers "allowed" to access the valuator structure (e.g. dev->valuator->axis[0].resolution = XY) or should I implement wrapper functions in the server to do that? >>> >>> I

Re: Smooth scrolling again

2010-11-17 Thread Max Schwarz
Hi, I implemented most of the dynamic-resolution feature today. Patches are at https://github.com/x-quadraht/pscroll/tree/master/patches/ . The wheel resolution property now resides in the emuWheel.c code, which reflects that it only affects wheel emulation. I added a manpage entry, too. Curren

Re: Smooth scrolling again

2010-11-14 Thread Peter Hutterer
On Sun, Nov 14, 2010 at 06:51:43PM +0100, Max Schwarz wrote: > Hi Simon, > > > > The clients depend on XIDeviceChangedEvents anyway to acquire the new > > > resolution when the user switches input devices (see pscrolltest.c). > > > > Though that sounds fine, I think the event is intended for mast

Re: Smooth scrolling again

2010-11-14 Thread Max Schwarz
Hi Simon, > > The clients depend on XIDeviceChangedEvents anyway to acquire the new > > resolution when the user switches input devices (see pscrolltest.c). > > Though that sounds fine, I think the event is intended for master/slave > issues. That's deep in spec land, however. Read up and see if

Re: Smooth scrolling again

2010-11-14 Thread Simon Thum
On 11/14/10 00:18, Max Schwarz wrote: > My proposal would be to make an XINotifyDeviceChanged(DeviceIntPtr dev) > function available to the drivers. It would simply copy the values from the > device to a DeviceChangedEvent and send it via XISendDeviceEvent(). > Such a function could also replace

Re: Smooth scrolling again

2010-11-13 Thread Max Schwarz
Hi Simon, > Still, the 42 is a bit odd. Caught me ;-) The value was a lucky guess, but it works pretty well for me (I'm using a TrackPoint to test it, other devices might yield other values...) > Towels anyone? No Bugblatter Beast in sight, so I'm safe for now ;-) > I guess that's mostly Peter

  1   2   >