[PATCH 4/6] Staging/iio/adc/touchscreen/MXS: add i.MX23 support to the LRADC touchscreen driver

Marek Vasut marex at denx.de
Mon Sep 16 14:19:55 UTC 2013


Dear Jürgen Beisert,

> Hi Jonathan,
> 
> On Sunday 15 September 2013 12:50:09 Jonathan Cameron wrote:
> > On 09/11/13 09:18, Juergen Beisert wrote:
> > > Distinguish i.MX23 and i.MX28 at runtime and do the same for both SoC
> > > at least for the 4 wire touchscreen.
> > > 
> > > Note: support for the remaining LRADC channels is not tested on an
> > > i.MX23 yet.
> > 
> > An ominous comment. Are you likely to test them soon?
> 
> Sorry, currently no i.MX28 hardware available. Someone out here with an
> i.MX28 based system with a touchscreen to test?

You mean 23, no? I can test both, I have the DENX EVK as well as both MX23EVK 
and MX28EVK. Just need to find some more time to do my maintainance rounds, but 
that should happen this week nonetheless. If I do forget, please ping me again.

> > Minor code layout comment inline but otherwise I'm just looking for an
> > ack from Marek.
> > 
> > > [...]
> > > 
> > >  static u32 mxs_lradc_drive_pressure(struct mxs_lradc *lradc)
> > >  {
> > > 
> > > +	if (lradc->soc == IMX23_LRADC)
> > > +		return LRADC_CTRL0_MX23_YP | LRADC_CTRL0_MX23_XM;
> > > 
> > >  	return LRADC_CTRL0_MX28_YPPSW | LRADC_CTRL0_MX28_XNNSW;
> > 
> > Whilst it obviously doesn't actually matter, having an else
> > in there would make the code more consistent so personally
> > I would prefer it to be there.
> 
> I can change it. Anyone here with objections against such a change?
> 
> Regards
> Juergen

Best regards,
Marek Vasut


More information about the devel mailing list