On 21/06/2021 19:26, Dmitry Osipenko wrote: > 21.06.2021 20:13, Daniel Lezcano пишет: >> >> Hi Dmitry, >> >> I compiled the your series and got these unresolved. >> >> arm-linux-gnueabi-ld: drivers/thermal/tegra/soctherm-fuse.o: in function >> `tegra_calc_shared_calib': >> soctherm-fuse.c:(.text+0x60): undefined reference to `tegra_fuse_readl' >> arm-linux-gnueabi-ld: soctherm-fuse.c:(.text+0xf0): undefined reference >> to `tegra_fuse_readl' >> arm-linux-gnueabi-ld: drivers/thermal/tegra/soctherm-fuse.o: in function >> `tegra_calc_tsensor_calib': >> soctherm-fuse.c:(.text+0x144): undefined reference to `tegra_fuse_readl' >> arm-linux-gnueabi-ld: drivers/thermal/tegra/tegra30-tsensor.o: in >> function `tegra_tsensor_fuse_read_spare': >> tegra30-tsensor.c:(.text+0x364): undefined reference to `tegra_fuse_readl' >> arm-linux-gnueabi-ld: drivers/thermal/tegra/tegra30-tsensor.o: in >> function `tegra_tsensor_probe': >> tegra30-tsensor.c:(.text+0x874): undefined reference to `tegra_fuse_readl' >> arm-linux-gnueabi-ld: >> drivers/thermal/tegra/tegra30-tsensor.o:tegra30-tsensor.c:(.text+0x904): >> more undefined references to `tegra_fuse_readl' follow >> make[1]: *** [/home/dlezcano/Work/src/linux/Makefile:1196: vmlinux] Error 1 >> make: *** [/home/dlezcano/Work/src/linux/Makefile:215: __sub-make] Error 2 > > The missing stub was added by [1]. I guess you could take [2] for the > base since Thierry already sent out PR for 5.14, or we could defer the > "thermal/drivers/tegra: Correct compile-testing of drivers" patch till > the next kernel version. Please choose whatever is easier for you. I would like to prevent to have more patches floating around. As we are close the to the merge window, I think we can live with the missing stubs ATM. But in the future, provide an immutable branch with the fixes, so we can share it and prevent these issues. -- Daniel -- <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | <http://twitter.com/#!/linaroorg> Twitter | <http://www.linaro.org/linaro-blog/> Blog