On Tue, 12 May 2015, Guennadi Liakhovetski wrote: > Hi Robert, > > On Tue, 12 May 2015, Robert Jarzmik wrote: > > > Hi Guennadi, > > > > Today I noticed the mioa701 build is broken on v4.1-rcX series. It was working > > in v4.0. > > > > The build error I get is : > > LINK vmlinux > > LD vmlinux.o > > MODPOST vmlinux.o > > GEN .version > > CHK include/generated/compile.h > > UPD include/generated/compile.h > > CC init/version.o > > LD init/built-in.o > > drivers/built-in.o: In function `v4l2_clk_set_rate': > > /home/rj/mio_linux/kernel/drivers/media/v4l2-core/v4l2-clk.c:196: undefined reference to `clk_round_rate' > > Makefile:932: recipe for target 'vmlinux' failed > > make: *** [vmlinux] Error 1 > > make: Target '_all' not remade because of errors. > > Not good:( > > > I have no idea what changed. Do you have a clue ? > > I've seen some patches on ALKML for PXA CCF, is it in the mainline now? > Could that have been the reason? Is CONFIG_COMMON_CLK defined in your > .config? Although, no, it's not PXA CCF, it's most probably this > > commit 4f528afcfbcac540c8690b41307cac5c22088ff1 > Author: Guennadi Liakhovetski <g.liakhovetski@xxxxxx> > Date: Sun Feb 1 08:12:33 2015 -0300 > > [media] V4L: add CCF support to the v4l2_clk API > > :( But I don't understand how this can happen. V4L is certainly not the > only driver in your build, that uses clk ops! They are exported from > drivers/clk/clk.c for GPL, but v4l2-dev.c defines the GPL licence, so, > should be ok. V4L is built as a module in your configuration, right? Can > you try building it into the image? I think I know how this is possible. PXA uses arch/arm/mach-pxa/clock.c for clk ops, and clk_round_rate() isn't defined there... Can we add a dummy for PXA? It won't be used anyway as long as PXA doesn't support CCF. Thanks Guennadi > > > > Cheers. > > > > -- > > Robert > > > > PS: A small extract of my .config > > rj@belgarion:~/mio_linux/kernel$ grep CLK .config > > CONFIG_HAVE_CLK=y > > CONFIG_PM_CLK=y > > # CONFIG_MMC_CLKGATE is not set > > CONFIG_CLKDEV_LOOKUP=y > > CONFIG_CLKSRC_OF=y > > CONFIG_CLKSRC_MMIO=y > > CONFIG_CLKSRC_PXA=y > > rj@belgarion:~/mio_linux/kernel$ grep V4L .config > > CONFIG_VIDEO_V4L2=y > > CONFIG_V4L_PLATFORM_DRIVERS=y > > # CONFIG_V4L_MEM2MEM_DRIVERS is not set > > # CONFIG_V4L_TEST_DRIVERS is not set > > CONFIG_DVB_AU8522_V4L=m > > > -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html