Bug 9690

Summary: [palmax] touchscreen logic bug prevents use in low loads
Product: xorg Reporter: Brice Goglin <brice.goglin>
Component: Input/palmaxAssignee: Xorg Project Team <xorg-team>
Status: RESOLVED WONTFIX QA Contact: Xorg Project Team <xorg-team>
Severity: normal    
Priority: high    
Version: 7.0 (2005.12)   
Hardware: x86 (IA32)   
OS: Linux (All)   
URL: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=392498
Whiteboard:
i915 platform: i915 features:
Attachments:
Description Flags
Fix processing under low load (and fix a typo) none

Description Brice Goglin 2007-01-17 11:00:19 UTC 
Copy of the Debian bug explanation:
"The palmax touchscreen driver contains a bug related to
the parsing of data from the serial port. The problem 
occurs when less than five bytes have been read after the
current package to be processed, a very frequent case
except when the system is under high load. The result is
that touchscreen data is discarded instead of parsed,
rendering the touchscreen unresponsive. 

I have found the faulty logic and am attaching a patch 
which seems to solve the problem correctly for me, though
it could probably use a second set of eyes to check for 
other thinkos. 

I am using a Palmax PD-1100, and with this patch applied
my touchscreen seems to work fine."


This bug has been reported against to the Debian BTS 2 years ago against
xserver-xfree86, and again 3 months ago against xorg 7.0 (by the same person).
The same patch fixed the bug in both releases. Upstream did not change since 7.0
got released, so I guess the bug is still present.

By the way, the following patch also fixes a typo.
Comment 1 Brice Goglin 2007-01-17 11:01:18 UTC 
Created attachment 8434 [details] [review]
Fix processing under low load (and fix a typo)
Comment 2 Daniel Stone 2007-02-27 01:35:49 UTC 
Sorry about the phenomenal bug spam, guys.  Adding xorg-team@ to the QA contact so bugs don't get lost in future.
Comment 3 Jesse Adkins 2010-11-20 11:49:03 UTC 
As of Feb 17 2009, this driver is set to fail in configure.ac because it does not have a maintainer. When a maintainer steps up for palmax, they can reopen this.

Until then, I'm closing it.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.