On Fri, Sep 28, 2012 at 8:35 PM, Tony Lindgren <tony@xxxxxxxxxxx> wrote: > * Shilimkar, Santosh <santosh.shilimkar@xxxxxx> [120928 08:02]: >> On Fri, Sep 28, 2012 at 8:25 PM, Tony Lindgren <tony@xxxxxxxxxxx> wrote: >> > >> > * Lokesh Vutla <lokeshvutla@xxxxxx> [120928 06:41]: >> > > Move plat/dma.h header to platform_data/dma-omap.h as >> > > part of the single zImage work. >> > >> > Hmm there's no platform data in this header, just >> > exported things for drivers to use. So it should not >> > be placed into platform_data. >> > >> > Maybe it should be #include <asm/mach/dma-omap.h> for now? >> > >> I wasn't sure either when the file was placed under platform-data. >> I agree for now we can keep it mach layer but than means OMAP1 and >> OMAP2+ DMA header and source code needs to be split. That >> is not so straight forward. > > No need for that, the path I'm suggesting is located under > arch/arm/include/asm/mach, it's not same as include <mach/dma-omap.h>. > That sound ok. >> With DMA engine conversion hopefully, we might get rid of the >> header eventually, but for now not sure whether we should >> go ahead and follow the splitting part. >> >> Thoughts ? > > No need for splitting anything :) > > The other possible location would be just include <linux/dma-omap.h>, > but as we all know that will be going away, <asm/mach/dma-omap.h> > is probably better. > Agree. I missed the "arch/arm/include/asm/mach" path. Thanks. Regards Santosh -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
