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

[Vinod Koul]

On Wed, Jun 12, 2013 at 02:23:39PM +0530, Viresh Kumar wrote:
> On 12 June 2013 13:42, Vinod Koul <vinod.koul@xxxxxxxxx> wrote:
> 
> > Never mind, am applying your patches and below fix, let me if you guys are okay
> 
> I am not :)
> 
> > diff --git a/drivers/dma/dw/Kconfig b/drivers/dma/dw/Kconfig
> > index db2b41f..7be1cf8 100644
> > --- a/drivers/dma/dw/Kconfig
> > +++ b/drivers/dma/dw/Kconfig
> > @@ -10,6 +10,7 @@ config DW_DMAC_CORE
> >  config DW_DMAC
> >         tristate "Synopsys DesignWare AHB DMA platform driver"
> >         select DW_DMAC_CORE
> > +       select DW_DMAC_BIG_ENDIAN_IO if ARCH=AVR32
> >         default y if CPU_AT32AP7000
> >         help
> >           Support the Synopsys DesignWare AHB DMA controller. This
> > @@ -25,12 +26,4 @@ config DW_DMAC_PCI
> >           Intel Medfield has integrated this GPDMA controller.
> >
> >  config DW_DMAC_BIG_ENDIAN_IO
> > -       bool "Use big endian I/O register access"
> > -       default y if AVR32
> > -       depends on DW_DMAC_CORE
> > -       help
> > -         Say yes here to use big endian I/O access when reading and writing
> > -         to the DMA controller registers. This is needed on some platforms,
> > -         like the Atmel AVR32 architecture.
> 
> This information was useful to understand this config option and must
> be kept in some way.
Well now the config option is not selected by users, so they dont need to know.
It was useful in knowing what to select earlier

For documenting, we can add this in commit log or better in driver code where
you use this

--
~Vinod
-- 
--
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/