linux-kernel - Re: dw_mipi_dsi-stm.c:(.text+0x8db9a3): undefined reference to `clk_hw

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <2f5b8627-60c0-4528-83fb-25d5af324516@foss.st.com>
Date: Tue, 10 Sep 2024 14:13:25 +0200
From: Raphael Gallais-Pou <raphael.gallais-pou@...s.st.com>
To: Maxime Ripard <mripard@...nel.org>
CC: Borislav Petkov <bp@...en8.de>, <linux-stm32@...md-mailman.stormreply.com>,
        Yannick Fertre
	<yannick.fertre@...s.st.com>,
        Philippe Cornu <philippe.cornu@...s.st.com>,
        Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
        Thomas Zimmermann
	<tzimmermann@...e.de>,
        David Airlie <airlied@...il.com>, Daniel Vetter
	<daniel@...ll.ch>,
        Maxime Coquelin <mcoquelin.stm32@...il.com>,
        Alexandre
 Torgue <alexandre.torgue@...s.st.com>,
        <dri-devel@...ts.freedesktop.org>,
        <linux-arm-kernel@...ts.infradead.org>, <linux-kernel@...r.kernel.org>
Subject: Re: dw_mipi_dsi-stm.c:(.text+0x8db9a3): undefined reference to
 `clk_hw_unregister'


On 9/10/24 13:59, Maxime Ripard wrote:
> 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@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
> 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.

Hi Maxime,

Oops I stand corrected, I effectively should have thought about sending this to
drm-misc-fixes.
Thank you for doing this.

Regards,

Raphaël

>
> Maxime