Messages in this thread | | | Date | Wed, 30 Nov 2011 10:34:18 +0800 | From | Feng Tang <> | Subject | Re: [PATCH 2/2] Input: tsc2007 - Add a z1_low_threshhold platform data parameter |
| |
On Tue, 29 Nov 2011 17:23:10 +0800 Dmitry Torokhov <dmitry.torokhov@gmail.com> wrote:
> On Tue, Nov 29, 2011 at 04:12:58PM +0800, Feng Tang wrote: > > This originates from a patch in Meego IVI kernel with the name > > linux-2.6.37-connext-0027-tsc2007.patch > > There is no author info excepte a line "From MeeGo > > <kernel@meego.com>" > > > > When integrating tsc2007 on Intel IVI platform, there are a lot of > > noise data whose z1 value is around 10 which is not a sane "z1". So > > add a "z1_low_threshhold" to filter those nosie data, to make the > > device work properly. > > Sounds like a task for userspace to ignore pressure that is too low. > Bonus points for making it configurable so user can adjust > sensitivity.
Actually, there is one more point :), without this patch, the driver won't work on our platforms.
The tsc2007_soft_irq will keep reading the input data unless there is no valid pressure data. In our case, those noise data will be seen as valid data during tsc2007_calculate_pressure(), and the tsc2007_soft_irq will run endlessly.
Thanks, Feng
> > Thanks. >
| |