Re: [PATCH] drm/amd/display: fix leak of probed modes

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!