Re: [PATCH 04/16] ARM: imx: move platform_data definitions

2012-09-13 Thread Vinod Koul
On Tue, 2012-09-11 at 14:59 +0200, Arnd Bergmann wrote: > Platform data for device drivers should be defined in > include/linux/platform_data/*.h, not in the architecture > and platform specific directories. > > This moves such data out of the imx include directories For these: > drivers/dma/imx

Re: [PATCH 04/16] ARM: imx: move platform_data definitions

2012-09-13 Thread Shawn Guo
On Wed, Sep 12, 2012 at 03:52:25PM +, Arnd Bergmann wrote: > On Tuesday 11 September 2012, Guennadi Liakhovetski wrote: > > ipu.h is used by the dmaengine and IRQ driver under drivers/dma/ipu/, and > > by its users drivers/media/platform/soc_camera/mx3_camera.c and > > drivers/video/mx3fb.c.

Re: [PATCH 04/16] ARM: imx: move platform_data definitions

2012-09-12 Thread Arnd Bergmann
On Tuesday 11 September 2012, Guennadi Liakhovetski wrote: > ipu.h is used by the dmaengine and IRQ driver under drivers/dma/ipu/, and > by its users drivers/media/platform/soc_camera/mx3_camera.c and > drivers/video/mx3fb.c. It has been placed under arch/arm/... because it's > arch-specific, bu

Re: [PATCH 04/16] ARM: imx: move platform_data definitions

2012-09-11 Thread Guennadi Liakhovetski
On Tue, 11 Sep 2012, Arnd Bergmann wrote: > On Tuesday 11 September 2012, Guennadi Liakhovetski wrote: > > Hi Arnd > > > > It probably wouldn't be very difficult to also include > > arch/arm/plat-mxc/include/mach/ipu.h in this your patch, but you can also > > leave it for later. Just to make su

Re: [PATCH 04/16] ARM: imx: move platform_data definitions

2012-09-11 Thread Arnd Bergmann
On Tuesday 11 September 2012, Guennadi Liakhovetski wrote: > Hi Arnd > > It probably wouldn't be very difficult to also include > arch/arm/plat-mxc/include/mach/ipu.h in this your patch, but you can also > leave it for later. Just to make sure it's not something that would make > your future pa

Re: [PATCH 04/16] ARM: imx: move platform_data definitions

2012-09-11 Thread Sascha Hauer
On Tue, Sep 11, 2012 at 02:59:38PM +0200, Arnd Bergmann wrote: > Platform data for device drivers should be defined in > include/linux/platform_data/*.h, not in the architecture > and platform specific directories. > > This moves such data out of the imx include directories > > Signed-off-by: Arn

Re: [PATCH 04/16] ARM: imx: move platform_data definitions

2012-09-11 Thread Guennadi Liakhovetski
Hi Arnd It probably wouldn't be very difficult to also include arch/arm/plat-mxc/include/mach/ipu.h in this your patch, but you can also leave it for later. Just to make sure it's not something that would make your future patching more complicated. Thanks Guennadi --- Guennadi Liakhovetski, Ph

[PATCH 04/16] ARM: imx: move platform_data definitions

2012-09-11 Thread Arnd Bergmann
Platform data for device drivers should be defined in include/linux/platform_data/*.h, not in the architecture and platform specific directories. This moves such data out of the imx include directories Signed-off-by: Arnd Bergmann Cc: Sascha Hauer Cc: Russell King Cc: Vinod Koul Cc: Dan Willi