On Tue, 14 Nov 2023, Thomas Zimmermann <tzimmermann@xxxxxxx> wrote: > If I'm not mistaken, the correct pattern is to read the EDID block in > the detect callback and only parse it for modes in get_modes. If so, you > might also inline this helper into its only caller in mgag200. I'll > later split it up into detect and get_modes. Never merged this, so I sent your suggested alternative instead [1]. BR, Jani. [1] https://patchwork.freedesktop.org/series/128269/ -- Jani Nikula, Intel