Re: [PATCH v2 2/4] dma: move dw_dmac driver to an own directory

From: Andy Shevchenko
Date: Wed Jun 12 2013 - 04:27:58 EST


On Wed, Jun 12, 2013 at 10:39 AM, Vinod Koul <vinod.koul@xxxxxxxxx> 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 <vinod.koul@xxxxxxxxx> 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

and then use

config DW_DMAC
select DW_DMAC_BIG_ENDIAN_IO if ARCH=AVR32

I don't know if we will have any platform in future which has such
difference in endianess between CPU and DMA IP block.

--
With Best Regards,
Andy Shevchenko
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/