[PATCH 09/16] arm/mx2/mbimx27: compile fixes with CONFIG_TOUCHSCREEN_ADS7846=y

[Uwe Kleine-König]

Hi Eric,

On Mon, May 24, 2010 at 10:34:28PM +0200, Uwe Kleine-König wrote:
> On Mon, May 24, 2010 at 10:10:55PM +0200, Eric Bénard wrote:
> > Hi Uwe,
> >
> > Le 24/05/2010 21:55, Uwe Kleine-König a écrit :
> >> This fixes:
> >>
> >> 	arch/arm/mach-mx2/eukrea_mbimx27-baseboard.c:166: error: expected identifier or '(' before '||' token
> >> 	arch/arm/mach-mx2/eukrea_mbimx27-baseboard.c:204: error: variable 'eukrea_mbimx27_spi_0_data' has initializer but incomplete type
> >> 	arch/arm/mach-mx2/eukrea_mbimx27-baseboard.c:205: error: unknown field 'chipselect' specified in initializer
> >> 	arch/arm/mach-mx2/eukrea_mbimx27-baseboard.c:205: warning: excess elements in struct initializer
> >> 	arch/arm/mach-mx2/eukrea_mbimx27-baseboard.c:205: warning: (near initialization for 'eukrea_mbimx27_spi_0_data')
> >> 	arch/arm/mach-mx2/eukrea_mbimx27-baseboard.c:206: error: unknown field 'num_chipselect' specified in initializer
> >> 	arch/arm/mach-mx2/eukrea_mbimx27-baseboard.c:206: warning: excess elements in struct initializer
> >> 	arch/arm/mach-mx2/eukrea_mbimx27-baseboard.c:206: warning: (near initialization for 'eukrea_mbimx27_spi_0_data')
> >> 	arch/arm/mach-mx2/eukrea_mbimx27-baseboard.c: In function 'eukrea_mbimx27_baseboard_init':
> >> 	arch/arm/mach-mx2/eukrea_mbimx27-baseboard.c:239: error: implicit declaration of function 'ads7846_dev_init'
> >>
> >> While at it add TOUCHSCREEN_ADS7846=m to mx27_defconfig.
> >>
> > I already fixed this in my patchset for this board :
> > http://lists.infradead.org/pipermail/linux-arm-kernel/2010-May/015826.html
> 
> What do you think about adding TOUCHSCREEN_ADS7846=m to mx27_defconfig?
> 
> Other than that I'm OK with your patch, too.  I will remove this patch
> from my queue.
If we want to have ADS7846 at least compilable in .35 we should go with
my patch as your's is (IMHO) too intrusive after -rc1.

Sascha?

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |