Age | Commit message (Collapse) | Author |
|
Don't do anything, but don't print a warning either.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
The module would previously log a message if min > max, but not do anything
about it. Keep the original min/max around, swap on-the-fly.
Signed-off-by: Jaska Kivela <jaska@kivela.net>
Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
Signed-off-by: Søren Holm <sh@sgh.dk>
Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
The server uses the flags to decide if a device comes up as attached or
floating slave. Overwriting the flag after the server has processed the
CorePointer/AlwaysCore/Floating/etc options results in the device always
coming up as floating slave. This again makes the device appear
unresponsive.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
xf86Elo.c:74:5: warning: initialization discards 'const' qualifier from
pointer target type [enabled by default]
several of:
xf86Elo.c:915:3: warning: initialization discards 'const' qualifier from
pointer target type [enabled by default]
(input ABI 18 only)
xf86Elo.c:1012:9: warning: assignment discards 'const' qualifier from
pointer target type [enabled by default]
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
Error doesn't exist anymore, removed in xorg-server-1.11.0-182-g09dbfcb
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
Avoid double-frees when the server calls Uninit.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
Let the device be printed in the logs
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
Signed-off-by: Terry Lambert <tlambert@chromium.org>
Reviewed-by: Stephane Marchesin <marcheu@chromium.org>
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
Don't overwrite the user-assigned name and use XI_TOUCHSCREEN for type_name.
Don't allow Option "DeviceName", use Identifier instead.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
And remove the manual input ABI check.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Cyril Brulebois <kibi@debian.org>
|
|
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Cyril Brulebois <kibi@debian.org>
|
|
All three are not called by the server anymore.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Cyril Brulebois <kibi@debian.org>
|
|
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Cyril Brulebois <kibi@debian.org>
|
|
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Cyril Brulebois <kibi@debian.org>
|
|
Both describe the same struct, LocalDevicePtr has been removed from the
server.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Cyril Brulebois <kibi@debian.org>
|
|
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Cyril Brulebois <kibi@debian.org>
|
|
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
Signed-off-by: Ivan Afonichev <ivan.afonichev@gmail.com>
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
The fix for bug #14109 ensures all bytes are emptied from the OS buffer
by looping until xf86WaitForInput returns 0. This patch just changes
the timeout from 1 millisecond to 0: we don't want the X server to block
if there's no more serial data.
It also removes the Vmin and Vtime options, which were making the calls
to read() block until a complete 10-byte packet buffer could be filled.
At 9600 bps, this could pause the X server for up to 9 ms. The code can
already handle partial buffers, so all we have to do is get rid of the
Vmin.
Also, if xf86EloGetPacket() returns !Success, we should continue rather
than break so the xf86WaitForInput call can decide whether to exit, in
case there's more data in the buffer.
Before the fix, glxgears was giving me about 390 FPS normally and down
to 140 FPS when dragging an empty area of the touchscreen. Now it's
basically unchanged when the touchscreen is in use (390 -> 385 FPS).
X.Org Bug 14109 <https://bugs.freedesktop.org/show_bug.cgi?id=14109>
Signed-off-by: Michael Smith <msmith@cbnco.com>
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
X.Org Bug 23239 <http://bugs.freedesktop.org/show_bug.cgi?id=23239>
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
|
|
This caused X server to access free()'d memory on exit path.
Thanks to msys.ch for providing a test machine.
|
|
This model does not reply to queries about touchscreen identification. After
opening the fd, we can skip right to enabling the device.
Red Hat Bug <https://bugzilla.redhat.com/show_bug.cgi?id=445193>
|
|
Some touchscreens supported by this driver need special handling, hence the
explicit specification of the model. Note that this commit does not actually
do anything with the information, it just sets some internal state.
Signed-off-by: Peter Hutterer <peter.hutterer@redhat.com>
|
|
|
|
These servers do the coordinate conversion themselves, so no need to do it in
the driver.
Signed-off-by: Peter Hutterer <peter@cs.unisa.edu.au>
|
|
|
|
If you're still not running something more recent than X11 3, then you have
other issues anyway.
|
|
|
|
Followup to 72408c2404246b9cb4698bd45be439be8ced3c23.
This patch isn't perfect yet, usage of O_NONBLOCK would be preferred and some
better error handling for incomplete packages, but at least it's less broken
than without this patch.
Signed-off-by: Peter Hutterer <peter@cs.unisa.edu.au>
X.Org Bug 14109 <http://bugs.freedesktop.org/show_bug.cgi?id=14109>
|
|
|
|
X.Org Bug 14902 <http://bugs.freedesktop.org/show_bug.cgi?id=14902>
|
|
There may be more than one packet waiting for us at a time, so loop until we
don't get one anymore.
This patch is untested for lack of a device, but should fix
X.Org Bug 14109 <http:/bugs.freedesktop.org/show_bug.cgi?id=14109>
|
|
xf86_ansic.h is no longer part of the xorg hence this causes
compilations failures on some architectures
Found by: Tinderbox
|
|
If we specify a min/max value for our valuators, the X server will clip any
coordinates at these values.
This fixes the second issue reported in #13248
X.Org Bugzilla #13248 <https://bugs.freedesktop.org/show_bug.cgi?id=13248>
|
|
cur_x and cur_y are the packets we get from the deviec, but after scaling the
actual coordinates we need to only deal with x and y.
This fixes one issue reported Bug #13248.
X.Org Bugzilla #13248 <https://bugs.freedesktop.org/attachment.cgi?id=13248>
|
|
A bad configuration can result in height or width being zero. This potentially
causes a divide-by-zero error in xf86EloConvert().
Detect the bad configuration early on and produce a meaningful error message.
X.Org Bugzilla #9803 <https://bugs.freedesktop.org/show_bug.cgi?id=9803>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
loadable build, and the same symbol can't be both static and _X_EXPORT.
Pointed out by Alan Coopersmith.
|