[PATCH] drm/amd/display: fix leak of probed modes
Fedor Pchelkin
pchelkin at ispras.ru
Tue Aug 19 16:45:04 UTC 2025
On Mon, 18. Aug 21:17, Limonciello, Mario wrote:
> On 8/17/25 4:43 AM, Fedor Pchelkin wrote:
> > For what the patch does there exists a drm_mode_remove() helper but it's
> > static at drivers/gpu/drm/drm_connector.c and requires to be exported
> > first. This probably looks like a subject for an independent for-next
> > patch, if needed.
>
> So you're saying this change will take two iterations of patches to ping
> pong the code?
>
> Why not just send this as a two patch series?
>
> 1) Export the symbol drm_mode_remove()
> 2) This patch, but use the symbol.
>
Initially I wasn't sure if the exporting patch was worth moving the code
around and in the end decided to make the current patch with a minimum of
prerequisites.
But giving this a second glance, I see exporting the symbol would be
technically better. I'll send out v2.
Thanks!
More information about the amd-gfx
mailing list