Hi Linus, On 03/11/2011 12:27, Linus Walleij wrote: > 2011/11/3 Maxime Ripard <maxime.ripard@xxxxxxxxxxxxxxxxxx>: > >> Cc: Nicolas Ferre <nicolas.ferre@xxxxxxxxx> >> Cc: Patrice Vilchez <patrice.vilchez@xxxxxxxxx> >> Signed-off-by: Maxime Ripard <maxime.ripard@xxxxxxxxxxxxxxxxxx> >> --- >> arch/arm/mach-at91/include/mach/board.h | 22 ++++++++++++++++++++++ >> 1 files changed, 22 insertions(+), 0 deletions(-) > > We're not supposed to have platform data dependent to stuff in > staging under arch/arm or anyplace else in the main kernel tree. > > Please move this to > drivers/staging/iio/adc/at91adc-board.h > or so. Won't moving this part to staging prevent from using this structure in board files ? If so, how will I be able to declare a new board that is using this ADC (or add the support for the ADC to a new one) ? > As for calling the at91_add_device_adc() function (which I guess > you want to do at some point) the pattern I followed for other > drivers is to declare a dummy function in arch/arm/mach-* > with __weak and let the staging driver override that. This way > the staging driver can go away without any compilation trouble > happening. I don't really see why my changes will break the compilation if the driver is no longer present in staging. At worst, the structure will be filled but used by no one, right ? Regards, -- Maxime Ripard, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html