[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240910-gleaming-boa-of-drama-d0a4db@houat>
Date: Tue, 10 Sep 2024 13:59:52 +0200
From: Maxime Ripard <mripard@...nel.org>
To: Raphael Gallais-Pou <raphael.gallais-pou@...s.st.com>
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 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.
Maxime
Download attachment "signature.asc" of type "application/pgp-signature" (274 bytes)
Powered by blists - more mailing lists