On Wed, Jun 12, 2013 at 11:27:51AM +0300, Andy Shevchenko wrote: > On Wed, Jun 12, 2013 at 10:39 AM, Vinod Koul <[email protected]> wrote: > > On Wed, Jun 12, 2013 at 10:43:29AM +0300, Andy Shevchenko wrote: > >> On Wed, Jun 12, 2013 at 8:08 AM, Vinod Koul <[email protected]> wrote: > >> > On Wed, Jun 05, 2013 at 03:26:44PM +0300, Andy Shevchenko wrote: > > [] > > > Something like > > config DW_DMAC_BIG_ENDIAN_IO > > tristate > > > > config DW_DMAC > > select DW_DMAC_BIG_ENDIAN_IO > > > > I belive DW_DMAC is for AVR support and the PCI ones is different right. > > The point is that these things should be selected by the required symbols > > based > > on what ARCH we are on and not be prompted for user to configure via > > menuconfig > > DW_DMAC is a generic platform driver which supports AVR32, ARM and > Intel (ACPI case) platforms. > > We may do that option invisible to user Yup > > and then use > > config DW_DMAC > select DW_DMAC_BIG_ENDIAN_IO if ARCH=AVR32 Thats would be the right way > > I don't know if we will have any platform in future which has such > difference in endianess between CPU and DMA IP block. you never know what h/w designers would think!
-- ~Vinod -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
