Hi, On 24/11/16 04:16, Chen-Yu Tsai wrote: > Hi, > > On Thu, Nov 24, 2016 at 9:17 AM, Andre Przywara <andre.przywara@xxxxxxx> wrote: >> The sun6i DMA driver is used in the Allwinner A64 and H5 SoC, which >> have arm64 capable cores. Add the generic sunxi config symbol to allow >> the driver to be selected by arm64 Kconfigs, which don't feature >> SoC specific MACH_xxxx configs. >> >> Signed-off-by: Andre Przywara <andre.przywara@xxxxxxx> >> --- >> drivers/dma/Kconfig | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/dma/Kconfig b/drivers/dma/Kconfig >> index af63a6b..003c284 100644 >> --- a/drivers/dma/Kconfig >> +++ b/drivers/dma/Kconfig >> @@ -157,7 +157,7 @@ config DMA_SUN4I >> >> config DMA_SUN6I >> tristate "Allwinner A31 SoCs DMA support" >> - depends on MACH_SUN6I || MACH_SUN8I || COMPILE_TEST >> + depends on MACH_SUN6I || MACH_SUN8I || COMPILE_TEST || ARCH_SUNXI > > AFAIK ARCH_SUNXI encompasses/supersedes MACH_SUN*I. > (And I don't have to add MACH_SUN9I later :) ) Sure, admittedly it was just a quick hack to get things going. Actually I don't know why we had a *depend* on those MACH_s before. I think technically it does not depend on a certain SoC (having the COMPILE_TEST in there hints on that). So what about: depends on ARCH_SUNXI || COMPILE_TEST and maybe: default y if MACH_SUN6I || MACH_SUN8I Though I see that both multi_v7_defconfig and sunxi_defconfig explicitly set this, so this wouldn't be needed? Cheers, Andre. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
