Messages in this thread | | | Date | Sat, 28 May 2011 13:33:05 +0300 | From | Mika Westerberg <> | Subject | Re: [PATCH 1/5] dmaengine: add ep93xx DMA support |
| |
On Thu, May 26, 2011 at 08:38:41PM -0500, H Hartley Sweeten wrote: > > > > +config EP93XX_DMA > > + bool "Cirrus Logic EP93xx DMA support" > > + depends on ARCH_EP93XX > > + select DMA_ENGINE > > + help > > + Enable support for the Cirrus Logic EP93xx M2P/M2M DMA controller. > > + > > config DMA_ENGINE > > bool > > The old dma-m2p support was always builtin. Should ARCH_EP93XX select > EP93XX_DMA to keep this, or should it be added to the ep93xx_defconfig?
There are no users for the DMA in ep93xx_defconfig so I think that it shouldn't be selected by default. I was planning to change platform code to be conditionally compiled, like:
arch/arm/mach-ep93xx/Makefile:
+obj-$(CONFIG_EP93XX_DMA) += dma.o
Any objections?
| |