On Mon, 31 Aug 2026, Thomas Zimmermann <[email protected]> wrote:
> Absolutely.  The only reason I added this helper is that the driver 
> needs the size for setting up the display pipeline before the connector 
> exists. I looked into using the existing code without the connector, but 
> that refactoring seemed to do more harm than good. Hence the new helper.

Heh, I looked into that too while reviewing, and agreed.

> Should the new function be named drm_edid_detect_panel_size() to signal 
> the use case?

Surprisingly, no strong feelings either way, both work for me.


BR,
Jani.


-- 
Jani Nikula, Intel

Reply via email to