S3C64XX & S3C24XX TS/ADC support in mainline

S3C64XX & S3C24XX TS/ADC support in mainline

[ch.naveen]

From: Naveen Krishna CH <ch.naveen@samsung.com>

Hi Ben,

This is Naveen from Samsung, India. 

I'm working on adding support for TS/ADC for the following boards
6410, 6440 and V210 on 2.6.34-rc2 kernel. 

1)
I did some initial work and found that the current driver 
(s3c24xx_ts.c) exists for ARCH_S3C24XX alone 
and which requires ADC for S3C24XX.  

Though. there is no board under (mach-s3c24XX) implementing the 
touchscreen driver.

I would like to know, on what all boards is this driver tested  & 
how much compatible it is with S3C64XX and S5P series SoCs.

2)
I also have a design issue with 
a.  There are some GPIO config specific to 2410 in the current driver.
b.  It uses only 1 IRQ (PenDown) and 
c.  The ISR for this IRQ uses the ADC APIs.

I have a reference driver, which uses 2 IRQs 
One for PENDOWN and another Stylus Action.
And this driver does not use the ADC driver.

TS on S3C64XX and S5P series does not seem to need
ADC driver as such. 

I need your suggestions on this. 


-- 
Thank You,
(: Naveen Krishna Ch :)

Re: S3C64XX & S3C24XX TS/ADC support in mainline

[Ben Dooks]

On Tue, Apr 06, 2010 at 10:35:10AM +0530, ch.naveen@samsung.com wrote:
> From: Naveen Krishna CH <ch.naveen@samsung.com>
> 
> Hi Ben,
> 
> This is Naveen from Samsung, India. 
> 
> I'm working on adding support for TS/ADC for the following boards
> 6410, 6440 and V210 on 2.6.34-rc2 kernel. 
> 
> 1)
> I did some initial work and found that the current driver 
> (s3c24xx_ts.c) exists for ARCH_S3C24XX alone 
> and which requires ADC for S3C24XX.  
> 
> Though. there is no board under (mach-s3c24XX) implementing the 
> touchscreen driver.
> 
> I would like to know, on what all boards is this driver tested  & 
> how much compatible it is with S3C64XX and S5P series SoCs.

The h1940 should be using the touchscreen, since this is what it was
originally written for.
 
> 2)
> I also have a design issue with 
> a.  There are some GPIO config specific to 2410 in the current driver.

yes, although this could be removed and given to the relevant machine.

> b.  It uses only 1 IRQ (PenDown) and 

yes.

> c.  The ISR for this IRQ uses the ADC APIs.
> 
> I have a reference driver, which uses 2 IRQs 
> One for PENDOWN and another Stylus Action.
> And this driver does not use the ADC driver.

either update the existing driver to use the StylusAction IRQ
or do without it.
 
> TS on S3C64XX and S5P series does not seem to need
> ADC driver as such. 

please dont' assume you know bettert than any of your potentuial
customers... the ADC API is there for a reason, there are more than
just the TS ADC inputs and it is entirly possible something else
will need acces to them.

On some of our boards, they provide input voltage monitoring and
feedback from sensors.

-- 
Ben

Q:      What's a light-year?
A:      One-third less calories than a regular year.