Re: Kconfig dangling references (BZ 216748)
Janne Grunau <[email protected]>
| Newsgroups | gmane.linux.sound,gmane.linux.kernel,gmane.linux.ports.arm.omap,gmane.comp.hardware.openbmc,gmane.linux.ports.mips,gmane.comp.video.dri.devel,gmane.linux.ports.arm.kernel,gmane.linux.kernel.gpio,gmane.linux.ports.sh.devel |
|---|---|
| Message-ID | <[email protected]> |
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.
So the "select DRM_KMS_DMA_HELPER" lines can be removed from both files.
Janne