> -----Original Message-----
> From: Dmitry Torokhov [mailto:[email protected]]
> Sent: Tuesday, January 12, 2010 2:07 PM
> To: Hiremath, Vaibhav
> Cc: [email protected]; [email protected]
> Subject: Re: [PATCH 2/3] Input:TouchScreen: Introduce TSC2004 driver
> support
>
> On Mon, Jan 11, 2010 at 11:54:05AM +0530, Hiremath, Vaibhav wrote:
> >
> > > -----Original Message-----
> > > From: Hiremath, Vaibhav
> > > Sent: Thursday, November 19, 2009 8:17 PM
> > > To: [email protected]
> > > Cc: [email protected]; Hiremath, Vaibhav
> > > Subject: [PATCH 2/3] Input:TouchScreen: Introduce TSC2004 driver
> > > support
> > >
> > > From: Vaibhav Hiremath <[email protected]>
> > >
> > > Signed-off-by: Vaibhav Hiremath <[email protected]>
> > > ---
> > > drivers/input/touchscreen/Kconfig | 11 +
> > > drivers/input/touchscreen/Makefile | 1 +
> > > drivers/input/touchscreen/tsc2004.c | 525
> > > +++++++++++++++++++++++++++++++++++
> > > include/linux/i2c/tsc2004.h | 17 ++
> > > 4 files changed, 554 insertions(+), 0 deletions(-)
> > > create mode 100644 drivers/input/touchscreen/tsc2004.c
> > > create mode 100644 include/linux/i2c/tsc2004.h
> > >
> > [Hiremath, Vaibhav] Dmitry,
> >
> > Any update on this? This driver support is important for
> AM3517EVM, can we merge this patch?
> >
>
> Hi Hiremath,
>
> I looked over the 2 drivers again and I still think that they are
> too
> much alike to warrant splitting them into 2. The only thing that you
> need is provide tsc2004-specific variant of tsc2007_read_values and
> add tsc2007-specific variant of tsc2004_prepare_for_reading.
>
> Once the drivers start really diverging we may revisit this issue.
[Hiremath, Vaibhav] Hi Dmitry,
Sorry for delayed response, as usual was busy with some other priority issues.
Let it be...
As I mentioned before, looking towards the functionality differences I decided
to implement/have separate driver for TSC2004. But I think you are right, since
as of now software is not leveraging all the features we can use existing
TSC2007 driver. Whenever we diverge we can slit them.
Just wanted to share/conform some thoughts on the implementation point of view,
- Probably change the driver from tsc2007 to tsc200x
- introduce initialization sequence as an array and pass it to the
i2c_device_id table.
- Driver will have to only use this array sequence, that's all.
I will try to change the driver accordingly and submit it again and also
update/inform you if I came across any issues (while doing this, which is
unlikely).
Thanks,
Vaibhav
>
> Thanks.
>
> --
> Dmitry
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
