On Friday, September 16, 2016 7:50:35 PM CEST Vinod Koul wrote: > On Fri, Sep 16, 2016 at 09:40:55AM +0200, Arnd Bergmann wrote: > > On Thursday, September 15, 2016 11:03:39 PM CEST Vinod Koul wrote: > > > We recently had a conversation about me cribbing that some drivers being > > > hard to compile [1]. Arnd suggested [2] to add COMPILE_TEST to these so > > > right now. With this I have one ARM config which compiles all drivers. > > > > > > I used multi_v7 default config and checked the drivers which are not > > > compiling and starting fixing them. So most of them were pretty fine, few > > > got warning mostly on bad usage of print specifiers with dma_addr_t and > > > size_t being major culprits there and few sparse warnings. > > > > > > While compile testing most of the driver compiled fine on others (i have > > > used ppc, arm, x86_64, i386, mips) expect the two where we explicitly depends > > > on ARM, one is omap and second one is tegra. Jz one had issues but I moved > > > header out of arm, so it worked fine. > > > They have some API dependency, which we should resolve later.. > > > > > > Please test... > > > > > > > Very nice series. If you have a git tree with these patches, I > > can add the branch to my randconfig testing to see if I get > > any build regression on 32-bit ARM. > > I had pushed it my kernel.org tree where Fenguuang's bot found a numbe of > issues, this series is present at: > > git://git.kernel.org/pub/scm/linux/kernel/git/vkoul/slave-dma.git test/compile-fixes > > Please let me know if you find something. > Ok, merged into the test tree now. I already got one failure: warning: (MMP_TDMA) selects MMP_SRAM which has unmet direct dependencies (ARCH_MMP) 0xC18DB3AE Fri Sep 16 16:21:42 CEST 2016 failed drivers/dma/mmp_tdma.o: In function `mmp_tdma_probe': mmp_tdma.c:(.text.mmp_tdma_probe+0x178): undefined reference to `sram_get_gpool' Arnd -- 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
