From: Pavel Machek <pavel@ucw.cz>
To: Paul Parsons <lost.distance@yahoo.com>,
dtor@mail.ru, dmitry.torokhov@gmail.com,
linux-input@vger.kernel.org
Cc: rpurdie@rpsys.net, lenz@cs.wisc.edu,
linux-kernel@vger.kernel.org, Dirk@Opfer-Online.de,
arminlitzel@web.de, metan@ucw.cz, thommycheck@gmail.com,
linux-arm-kernel@lists.infradead.org, dbaryshkov@gmail.com,
omegamoon@gmail.com, eric.y.miao@gmail.com, utx@penguin.cz
Subject: Re: [PATCH] hx4700: fix hx4700 touchscreen pressure values
Date: Wed, 18 Nov 2009 18:28:49 +0100 [thread overview]
Message-ID: <20091118172849.GD25150@elf.ucw.cz> (raw)
In-Reply-To: <E1NAYJo-00016g-PW@pih-relay04.plus.net>
On Wed 2009-11-18 00:22:41, Paul Parsons wrote:
> > Does pressure actually work as expected -- that is not inverted?
> > Because I need this patch on spitz... but it may break your
> > machine...
>
> You are right, the pressure values were being inverted on my hx4700.
> Pressing hard generates somewhat lower Rt values than pressing lightly.
>
> Since typical hx4700 Rt values are close to (pressure_max/2) anyway,
> the inverted and non-inverted values were not very different.
> For example:
>
> Rt ts->pressure_max-Rt
> 599 425
> 562 462
> 588 436
> 570 454
>
> So by all means submit your ads7846.c patch; it will not break hx4700.
Dmitry, those pressure values are reverted on more than one machine,
and we do not have anyone reporting it works ok for them. Can we
perhaps just.... fix it?
Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
next parent reply other threads:[~2009-11-18 17:29 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <E1NAYJo-00016g-PW@pih-relay04.plus.net>
2009-11-18 17:28 ` Pavel Machek [this message]
2009-11-18 17:39 ` [PATCH] hx4700: fix hx4700 touchscreen pressure values Dmitry Torokhov
2009-11-18 17:46 ` Dmitry Torokhov
2009-11-20 18:24 ` Pavel Machek
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20091118172849.GD25150@elf.ucw.cz \
--to=pavel@ucw.cz \
--cc=Dirk@Opfer-Online.de \
--cc=arminlitzel@web.de \
--cc=dbaryshkov@gmail.com \
--cc=dmitry.torokhov@gmail.com \
--cc=dtor@mail.ru \
--cc=eric.y.miao@gmail.com \
--cc=lenz@cs.wisc.edu \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-input@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lost.distance@yahoo.com \
--cc=metan@ucw.cz \
--cc=omegamoon@gmail.com \
--cc=rpurdie@rpsys.net \
--cc=thommycheck@gmail.com \
--cc=utx@penguin.cz \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).