Hi Vinod, On Thu, Jun 28, 2018 at 8:29 AM Vinod <vkoul@xxxxxxxxxx> wrote: > On 22-06-18, 11:44, Angelo Dureghello wrote: > > obj-$(CONFIG_EP93XX_DMA) += ep93xx_dma.o > > obj-$(CONFIG_FSL_DMA) += fsldma.o > > obj-$(CONFIG_FSL_EDMA) += fsl-edma.o fsl-edma-common.o > > +obj-$(CONFIG_MCF_EDMA) += mcf-edma.o fsl-edma-common.o > > that makes kernel have two copies of common.o one in thsi driver and one Does it? It's a common pattern in several Makefiles (e.g. drivers/net/ethernet/8390/Makefile and drivers/scsi/Makefile) > in previous one why not do: > > CONFIG_FSL_COMMON += fsl-edma-common.o > CONFIG_FSL_EDMA += fsl-edma.o > CONFIG_MCF_EDMA += mcf-edma.o > > and you select CONFIG_FSL_COMMON in both FSL and MCF Kconfig? That's what he had originally, until I suggested to do the above, as nothing else needed the CONFIG_FSL_COMMON symbol. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds -- To unsubscribe from this list: send the line "unsubscribe dmaengine" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html