Messages in this thread | | | Date | Wed, 27 Jun 2012 16:26:18 -0300 | Subject | Re: [PATCH 1/4] mtd: lpc32xx_slc: Select AMBA_PL08X in Kconfig | From | Alexandre Pereira da Silva <> |
| |
On Wed, Jun 27, 2012 at 4:15 PM, Russell King - ARM Linux <linux@arm.linux.org.uk> wrote: > On Wed, Jun 27, 2012 at 05:51:12PM +0200, Roland Stigge wrote: >> From: Alexandre Pereira da Silva <aletes.xgr@gmail.com> >> >> Since this driver depends on the amba pl08x dma driver, select it in Kconfig. >> >> Signed-off-by: Alexandre Pereira da Silva <aletes.xgr@gmail.com> >> Signed-off-by: Roland Stigge <stigge@antcom.de> >> --- >> drivers/mtd/nand/Kconfig | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/drivers/mtd/nand/Kconfig b/drivers/mtd/nand/Kconfig >> index de69978..a9e8b73 100644 >> --- a/drivers/mtd/nand/Kconfig >> +++ b/drivers/mtd/nand/Kconfig >> @@ -417,6 +417,7 @@ config MTD_NAND_PXA3xx >> config MTD_NAND_SLC_LPC32XX >> tristate "NXP LPC32xx SLC Controller" >> depends on ARCH_LPC32XX >> + select AMBA_PL08X >> help >> Enables support for NXP's LPC32XX SLC (i.e. for Single Level Cell >> chips) NAND controller. This is the default for the PHYTEC 3250 > > Surely not? Drivers using DMA engine really should not depend on any > particular DMA engine implementation.
Should the DMA code in this driver be ifdef'd like in spi-pl022.c?
Or just depend on DMA_ENGINE instead? -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |