Current tslib indeed does use ABS_PRESSURE to signal applied pressure. BTN_TOUCH is only used to indicate pen up events. This happens in the input_raw module.<div>Tslib could be modified to check whether the underlying input device supports ABS_PRESSURE. if not, it shoud use BTN_TOUCH to signal applied pressure.</div>
<div><br></div><div><br><div class="gmail_quote">On Wed, Apr 11, 2012 at 12:16 PM, Richard Purdie <span dir="ltr"><<a href="mailto:richard.purdie@linuxfoundation.org">richard.purdie@linuxfoundation.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Wed, 2012-04-11 at 10:59 +0200, Andrea Adami wrote:<br>
> On Mon, Apr 2, 2012 at 11:35 PM, Andrea Adami <<a href="mailto:andrea.adami@gmail.com">andrea.adami@gmail.com</a>> wrote:<br>
> > * The actual rule only works for devices reporting ts pressure<br>
> > * and ignores the other touchscreens (happens e.g. on ipaq h1940).<br>
> > * a0,1,*18 a = absolute axes (ABS_X 0x00 ABS_Y 0x01 ABS_PRESSURE 0x18)<br>
> > * (see include/linux/input.h)<br>
> > *<br>
> > * Guidelines for touchscreen seem suggesting only ABS_X and ABS_Y are mandatory<br>
> > * (<a href="http://kernel.org/doc/Documentation/input/event-codes.txt" target="_blank">http://kernel.org/doc/Documentation/input/event-codes.txt</a>)<br>
> > *<br>
> > * Bump PR<br>
> ><br>
> > Signed-off-by: Andrea Adami <<a href="mailto:andrea.adami@gmail.com">andrea.adami@gmail.com</a>><br>
> > ---<br>
> > meta/recipes-core/udev/udev/local.rules | 3 +--<br>
> > meta/recipes-core/udev/<a href="http://udev_164.bb" target="_blank">udev_164.bb</a> | 2 +-<br>
> > 2 files changed, 2 insertions(+), 3 deletions(-)<br>
> ><br>
> > diff --git a/meta/recipes-core/udev/udev/local.rules b/meta/recipes-core/udev/udev/local.rules<br>
> > index 625e49a..49e8d28 100644<br>
> > --- a/meta/recipes-core/udev/udev/local.rules<br>
> > +++ b/meta/recipes-core/udev/udev/local.rules<br>
> > @@ -31,5 +31,4 @@ KERNEL=="rtc0", SYMLINK+="rtc"<br>
> > ACTION=="add", DEVPATH=="/devices/*", ENV{MODALIAS}=="?*", RUN+="/sbin/modprobe $env{MODALIAS}"<br>
> ><br>
> > # Create a symlink to any touchscreen input device<br>
> > -SUBSYSTEM=="input", KERNEL=="event[0-9]*", ATTRS{modalias}=="input:*-e0*,3,*a0,1,*18,*", SYMLINK+="input/touchscreen0"<br>
> > -<br>
> > +SUBSYSTEM=="input", KERNEL=="event[0-9]*", ATTRS{modalias}=="input:*-e0*,3,*a0,1,*", SYMLINK+="input/touchscreen0"<br>
> > diff --git a/meta/recipes-core/udev/<a href="http://udev_164.bb" target="_blank">udev_164.bb</a> b/meta/recipes-core/udev/<a href="http://udev_164.bb" target="_blank">udev_164.bb</a><br>
> > index c5813ec..0462ff2 100644<br>
> > --- a/meta/recipes-core/udev/<a href="http://udev_164.bb" target="_blank">udev_164.bb</a><br>
> > +++ b/meta/recipes-core/udev/<a href="http://udev_164.bb" target="_blank">udev_164.bb</a><br>
> > @@ -1,6 +1,6 @@<br>
> > include udev.inc<br>
> ><br>
> > -PR = "r13"<br>
> > +PR = "r14"<br>
> ><br>
> > SRC_URI += "file://udev-166-v4l1-1.patch"<br>
> ><br>
> > --<br>
> > 1.7.3.4<br>
> ><br>
><br>
> Bump<br>
><br>
> I'm not linux-input specialist but this fix looks rather harmless.<br>
<br>
</div></div>I'm not 100% convinced about that. There needs to be some mechanism to<br>
signal to the system when there is pressure applied to the screen and<br>
when pressure was released. I think there are two ways you can do this,<br>
BTN_TOUCH and ABS_PRESSURE. Regardless of what the input events codes<br>
say, I'm really more interested in what tslib accepts.<br>
<br>
This code has been like this for a long time and I'm surprised that if<br>
there were issues it wasn't patched long before now. I'm therefore left<br>
wondering if the touchscreen driver you're using is doing the right<br>
things.<br>
<br>
Cheers,<br>
<br>
Richard<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
<br>
<br>
_______________________________________________<br>
Openembedded-core mailing list<br>
<a href="mailto:Openembedded-core@lists.openembedded.org">Openembedded-core@lists.openembedded.org</a><br>
<a href="http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core" target="_blank">http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core</a><br>
</div></div></blockquote></div><br></div>