Hi, El 07/03/16 a las 12:47, Boris Brezillon escribió: (...) >> Does SPI refer the Serial Peripheral Interface? >> >> If yes, then I would point out that current sun4i SPI driver doesn't >> actually use DMA [1] >> >> http://lists.infradead.org/pipermail/linux-arm-kernel/2016-February/411 >> 722.html > > I just moved this assignment and the associated comment in the driver, > so maybe we should ask Emilio why he thinks SPI config should be the > default one, and how he tested it... When I was working on the dmaengine driver, I needed a way to test mem<->dev and mem<->mem transfers. I used dmatest.ko for the latter, and SPI was a good fit for the former as I had a logic analyzer. That's why there's patches to support DMA on it (but, as Priit pointed out, are not in mainline yet). At first SPI was acting weird when using DMA, but the problems went away when configuring the timings with these magic values you see on the driver today. These timings turned out to also work for audio, so there was no need for a mechanism to configure them. And that's basically the story behind SUN4I_DDMA_MAGIC_SPI_PARAMETERS :) Cheers, Emilio -- 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