Re: Kconfig dangling references (BZ 216748)

Paul Kocialkowski <[email protected]> Wed, 10 Dec 2025 23:09:33 +0100

Newsgroups	gmane.linux.ports.sh.devel,gmane.linux.kernel,gmane.linux.ports.arm.omap,gmane.comp.hardware.openbmc,gmane.linux.sound,gmane.linux.ports.mips,gmane.comp.video.dri.devel,gmane.linux.ports.arm.kernel,gmane.linux.kernel.gpio
Message-ID	<aTnvnaRuJ5lF4dVv@collins>

Hi Janne,

Le Mon 08 Dec 25, 21:05, Janne Grunau a écrit :
> On Mon, Dec 08, 2025 at 07:13:50PM +0100, Paul Kocialkowski wrote:
> > Hi Randy,
> > 
> > On Sun 07 Dec 25, 18:04, Randy Dunlap wrote:
> > > from  https://bugzilla.kernel.org/show_bug.cgi?id=216748
> > > 
> > > The bugzilla entry includes a Perl script and a shell script.
> > > This is the edited result of running them (I removed some entries that were noise).
> > 
> > [...]
> > 
> > > DRM_KMS_DMA_HELPER ---
> > > drivers/gpu/drm/adp/Kconfig:9:	select DRM_KMS_DMA_HELPER
> > > drivers/gpu/drm/logicvc/Kconfig:7:	select DRM_KMS_DMA_HELPER
> > 
> > For these two, the symbol was removed in commit
> > 09717af7d13d63df141ae6e71686289989d17efd
> 
> That commit removed DRM_KMS_CMA_HELPER. Later commit 6bcfe8eaeef0
> ("drm/fb: rename FB CMA helpers to FB DMA helpers") renamed
> DRM_KMS_CMA_HELPER erroneously to DRM_KMS_DMA_HELPER.
> 
> > but these two drivers either were
> > missed by the batch rename or were introduced a bit later.
> 
> In the case of drivers/gpu/drm/adp/Kconfig it was missed much later
> during review (but iirc went through the same rename out of tree).
> 
> > Since the symbol selected DRM_GEM_CMA_HELPER (which is still needed by the
> > drivers), it should be replaced with DRM_GEM_CMA_HELPER.
> 
> That symbol doesn't exist anymore either. It's now DRM_GEM_DMA_HELPER
> which is already present in both files.

Thanks for the details! It seems that I was looking at an older tree.

> So the "select DRM_KMS_DMA_HELPER" lines can be removed from both files.

Good, then I'll craft a patch removing these two lines.

All the best,

Paul

-- 
Paul Kocialkowski,

Independent contractor - sys-base - https://www.sys-base.io/
Free software developer - https://www.paulk.fr/

Expert in multimedia, graphics and embedded hardware support with Linux.

signature.asc (application/pgp-signature, 833 B)