From: "andrzej zaborowski" <balrogg@gmail.com>
To: David Brownell <david-b@pacbell.net>
Cc: Linux OMAP <linux-omap-open-source@linux.omap.com>
Subject: Re: Common code for TSC 2101 and 2102
Date: Mon, 24 Sep 2007 00:37:21 +0200 [thread overview]
Message-ID: <fb249edb0709231537p4de2f11aic934938f92e2936e@mail.gmail.com> (raw)
In-Reply-To: <200708132234.41538.david-b@pacbell.net>
Hi, sorry for late reply, I'm just catching up with email now.
On 14/08/2007, David Brownell <david-b@pacbell.net> wrote:
> Following this will be several patches, two related to
> this driver:
>
> - H4 board support, using TSC2101 (not TSC2102)
> - A biggish tsc210x cleanup patch, with some bugfixes
Thanks for this work.
>
> For boards using external VREF, that value should be in
> the tsc210x chip platform data. (And Andrzej, I'm
> curious why you settled on VREF=1.25V vs 2.50V ... not
> that it's necessarily an issue, just "why". A comment
> in the code might be good.)
Oh, I'm afraid I left it to be made configurable by the first person
who needs VREF of 2.5V, out of laziness. I settled on 1.25V
specifically because that's what PalmOS (the OS that ships with Palm
Tungsten E) uses. The device is a consumer PDA and I have no
schematics or even multimeter access to the chips, and likely I
wouldn't know how to use it if I had.
>
> Somewhat related, I think the TSC setup code should
> likely hold off until it can fetch calibration data
> from EEPROM or somesuch ... temperature readings are
> not all that useful until that data is available.
> Alternatively, do like I did in ads7846: punt it all
> to userspace, just provide the raw milliVolt readings
> and expect someone else to compute temperature.
The board setup code can defer the TSC setup by registering the
tsc210x device only after all information is available to fill in the
pdata struct.
Regards
next prev parent reply other threads:[~2007-09-23 22:37 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-06-04 13:44 Common code for TSC 2101 and 2102 andrzej zaborowski
2007-07-13 6:46 ` Trilok Soni
2007-07-13 7:10 ` Trilok Soni
2007-07-13 19:29 ` andrzej zaborowski
2007-07-13 20:04 ` David Brownell
2007-08-10 7:29 ` Tony Lindgren
2007-08-11 3:04 ` David Brownell
2007-08-13 7:05 ` Tony Lindgren
2007-08-14 5:34 ` David Brownell
2007-09-23 22:37 ` andrzej zaborowski [this message]
2007-08-14 5:38 ` [patch 2.6.23-rc2-omap1] tsc210x cleanup David Brownell
2007-08-15 10:54 ` Tony Lindgren
2007-08-14 5:40 ` [patch 2.6.23-rc2-omap1] H4 support for tsc210x David Brownell
2007-08-14 6:45 ` David Brownell
2007-08-15 10:54 ` Tony Lindgren
2007-07-13 18:38 ` Common code for TSC 2101 and 2102 David Brownell
2007-07-15 20:18 ` David Brownell
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=fb249edb0709231537p4de2f11aic934938f92e2936e@mail.gmail.com \
--to=balrogg@gmail.com \
--cc=david-b@pacbell.net \
--cc=linux-omap-open-source@linux.omap.com \
/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