On Tue, Sep 10, 2024 at 12:48:49PM GMT, Raphael Gallais-Pou wrote: > > On 9/9/24 10:45, Borislav Petkov wrote: > > On Mon, Sep 09, 2024 at 08:57:57AM +0200, Raphael Gallais-Pou wrote: > >> Arnd Bergmann sent a patch regarding this issue on x86 that I merged several > >> weeks ago. > >> > >> https://lore.kernel.org/lkml/20240719075454.3595358-1-arnd@xxxxxxxxxx/ > >> https://lore.kernel.org/all/c3d0757a-07c0-4f83-9f06-c3ad205aa1e9@xxxxxxxxxxx/ > > Thanks. > > > >> Now, I'm not familiar with PowerPC architecture and toolchains, but I think > >> this patch should fix your problem. Do you have the above fixup in your > >> tree ? If not please try it. > > Not PowerPC - it is an x86 build. I do random config builds as part of build > > testing and when a driver depends on COMPILE_TEST, it does gets built in some > > .configs. So some may fail, as in this case. > > Unless I am mistaken, the link you provided refers to a PowerPC linker error: > > [...] > > compiler: powerpc-linux-gcc (GCC) 14.1.0 > > [...] > powerpc-linux-ld: drivers/gpu/drm/stm/dw_mipi_dsi-stm.o: in function `dw_mipi_dsi_stm_remove': > dw_mipi_dsi-stm.c:(.text+0x664): undefined reference to `of_clk_del_provider' > >> powerpc-linux-ld: dw_mipi_dsi-stm.c:(.text+0x66c): undefined reference to `clk_hw_unregister' powerpc-linux-ld: drivers/gpu/drm/stm/dw_mipi_dsi-stm.o: in function `dw_mipi_dsi_stm_probe': > dw_mipi_dsi-stm.c:(.text+0x98c): undefined reference to `clk_hw_register' > > > > > So I think you should send that patch to Linus now so that such randconfig > > builds do not fail anymore. > > What do you mean by 'sending it to Linus' ? If you meant to do a pull request, > then no. This patch is already in the drm-misc tree, which means it will keep > its usual pace of merging with the rest of the drm-misc tree. > > For more information about drm-misc tree: > https://drm.pages.freedesktop.org/maintainer-tools/repositories/drm-misc.html#merge-timeline That's not entirely correct. This should have been merged in drm-misc-fixes to begin with. I've cherry-picked the patch and pushed it. Maxime
Attachment:
signature.asc
Description: PGP signature
