From mboxrd@z Thu Jan 1 00:00:00 1970 From: Pavel Machek Subject: Re: ads7846 touchscreen: fix pressure Date: Fri, 9 Oct 2009 22:22:51 +0200 Message-ID: <20091009202250.GA2040@elf.ucw.cz> References: <20091006195413.GA1469@ucw.cz> <5d5443650910072209t63e3fa9dvff7ab2fee7bb2f88@mail.gmail.com> <20091008092433.GA26837@elf.ucw.cz> <20091009163506.GE1092@core.coreip.homeip.net> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Content-Disposition: inline In-Reply-To: <20091009163506.GE1092@core.coreip.homeip.net> Sender: linux-kernel-owner@vger.kernel.org To: Dmitry Torokhov , mroth@nessie.de, bn@niasdigital.com, eric.miao@marvell.com Cc: Trilok Soni , linux-input@vger.kernel.org, kernel list , Andrew Morton List-Id: linux-input@vger.kernel.org On Fri 2009-10-09 09:35:07, Dmitry Torokhov wrote: > On Thu, Oct 08, 2009 at 11:24:33AM +0200, Pavel Machek wrote: > > On Thu 2009-10-08 10:39:39, Trilok Soni wrote: > > > Hi Pavel, > > >=20 > > > On Wed, Oct 7, 2009 at 1:24 AM, Pavel Machek wrote= : > > > > > > > > On Zaurus, pressure is reported inverted -- the lighter the pre= ssure, > > > > the bigger numerical value. This should fix it. > > > > > > > > Signed-off-by: Pavel Machek > > > > > > > > --- linux-rc/drivers/input.ofic/touchscreen/ads7846.c =A0 2009-= 10-06 13:49:56.000000000 +0200 > > > > +++ linux-rc/drivers/input/touchscreen/ads7846.c =A0 =A0 =A0 =A0= 2009-10-06 21:18:25.000000000 +0200 > > > > @@ -608,12 +609,12 @@ > > > > > > > > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0input_report_abs(input, ABS_X, x= ); > > > > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0input_report_abs(input, ABS_Y, y= ); > > > > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 input_report_abs(input, ABS_PRESS= URE, Rt); > > > > + =A0 =A0 =A0 =A0 =A0 =A0 =A0 input_report_abs(input, ABS_PRESS= URE, ts->pressure_max-Rt); > > >=20 > > > but same behavior is seen by others using this chip on different > > > boards? If not, this has to be done through platform data. > >=20 > > Well, I'd expect the chip to behave in similar way on all > > platforms. Of course, it would be good if someone with different hw > > tested it, but I do not have that hw... > >=20 > > So I guess the best way is to make it go in early and see how it do= es > > in testing.... >=20 > No, this is not how it works. Please do >=20 > git log -- drivers/input/touchscreen/ads7846.c >=20 > and try to check with people who were working on this driver. I someh= ow > doubt that they'd overlook such obvious issue as inverted pressure > readings. Well, it was certainly not obvious to the one who switched zaurus to ads7846, thus causing regression. Anyway, people, can you please test ads7846, to see if it has normal or inverted pressure readings? I could only find 3 people contributing to ads7846 in last year or so, and Eric was one of them...=20 Pavel --=20 (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses= /blog.html