Re: dw_mipi_dsi-stm.c:(.text+0x8db9a3): undefined reference to `clk_hw_unregister'

[Raphael Gallais-Pou <raphael.gallais-pou@foss.st.com>]

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@kernel.org/
>> https://lore.kernel.org/all/c3d0757a-07c0-4f83-9f06-c3ad205aa1e9@foss.st.com/
> 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

>
> Unfortunately, I cannot test it right now because I've removed the triggering
> randconfigs but when I hit it again, I'll make sure to apply yours.
Thank you for this. Do not hesitate to reach if you encounter any problems with
this patch.

Regards,
Raphaël

>
> Thx.
>