From: Chris Bagwell <ch...@cnpbagwell.com>

Over time, we've merged device specific filter routines
into single version.  So there is no need for historic
hook to register at runtime.

ISDv4 devices never registered any filter which seems
like it would result in jitter.

Removed RawFilter() from WacomDevice structure and call
wcmFilterCoord() directly from wcmCommon.c.

Signed-off-by: Chris Bagwell <ch...@cnpbagwell.com>
---

Updated an older patch to work against current HEAD. Resending
as requested by David to forward to users that are complaining
about jitter on ISDv4 devices for further testing.

 src/wcmCommon.c     |    5 ++---
 src/wcmUSB.c        |    2 --
 src/xf86WacomDefs.h |    2 --
 3 files changed, 2 insertions(+), 7 deletions(-)

diff --git a/src/wcmCommon.c b/src/wcmCommon.c
index 315aa4e..f7a8f63 100644
--- a/src/wcmCommon.c
+++ b/src/wcmCommon.c
@@ -906,14 +906,13 @@ void wcmEvent(WacomCommonPtr common, unsigned int channel,
        }
 
        /* Optionally filter values only while in proximity */
-       if (common->wcmModel->FilterRaw &&
-           ds.proximity && ds.device_type != PAD_ID)
+       if (ds.proximity && ds.device_type != PAD_ID)
        {
                /* Start filter fresh when entering proximity */
                if (!pLast->proximity)
                        wcmResetSampleCounter(pChannel);
 
-               common->wcmModel->FilterRaw(common,pChannel,&ds);
+               wcmFilterCoord(common,pChannel,&ds);
        }
 
        /* skip event if we don't have enough movement */
diff --git a/src/wcmUSB.c b/src/wcmUSB.c
index e1997aa..06f8655 100644
--- a/src/wcmUSB.c
+++ b/src/wcmUSB.c
@@ -22,7 +22,6 @@
 #endif
 
 #include "xf86Wacom.h"
-#include "wcmFilter.h"
 
 #include <asm/types.h>
 #include <linux/input.h>
@@ -76,7 +75,6 @@ static struct _WacomModel mname =             \
        .GetRanges = usbWcmGetRanges,           \
        .Start = usbStart,                      \
        .Parse = usbParse,                      \
-       .FilterRaw = wcmFilterCoord,            \
        .DetectConfig = usbDetectConfig,        \
 };
 
diff --git a/src/xf86WacomDefs.h b/src/xf86WacomDefs.h
index 891f6a6..8b15548 100644
--- a/src/xf86WacomDefs.h
+++ b/src/xf86WacomDefs.h
@@ -128,8 +128,6 @@ struct _WacomModel
        int (*GetRanges)(InputInfoPtr pInfo);
        int (*Start)(InputInfoPtr pInfo);
        int (*Parse)(InputInfoPtr pInfo, const unsigned char* data, int len);
-       int (*FilterRaw)(WacomCommonPtr common, WacomChannelPtr pChannel,
-               WacomDeviceStatePtr ds);
        int (*DetectConfig)(InputInfoPtr pInfo);
 };
 
-- 
1.7.3.4


------------------------------------------------------------------------------
Colocation vs. Managed Hosting
A question and answer guide to determining the best fit
for your organization - today and in the future.
http://p.sf.net/sfu/internap-sfd2d
_______________________________________________
Linuxwacom-devel mailing list
Linuxwacom-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel

Reply via email to