On Tue, 14 Nov 2023, Jani Nikula <jani.nikula@xxxxxxxxx> wrote: > On Thu, 26 Oct 2023, Jani Nikula <jani.nikula@xxxxxxxxx> wrote: >> This is just the first two patches of a lengthy series that I'm not >> really sure how to proceed with. Basically the series converts all of >> drm/bridge to the new struct drm_edid infrastructure. It's safer than >> struct edid, because it contains meta information about the allocated >> size of the EDID, instead of relying on the size (number of extensions) >> originating from outside of the kernel. >> >> The rest is at [1]. The commit messages are lacking, and I don't really >> have the toolchain to even build test most of it. But I think this is >> where drm/bridge should go. Among all of drm, I think bridge has the >> most uses of struct edid that do not originate from the drm_get_edid() >> family of functions, which means the validity checks are somewhat >> inconsistent, and having the meta information is more crucial. >> >> Bridge maintainers, please instruct how to best proceed with this. > > Ping. Ping. > > The two patches posted here could be merged, to add something to build > the later commits on gradually. > > BR, > Jani. > >> >> >> Thanks, >> Jani. >> >> >> >> [1] https://gitlab.freedesktop.org/jani/linux/-/commits/drm-edid-bridge >> >> >> >> Jani Nikula (2): >> drm/bridge: add ->edid_read hook and drm_bridge_edid_read() >> drm/bridge: switch to drm_bridge_read_edid() >> >> drivers/gpu/drm/drm_bridge.c | 46 +++++++++++++++++++++++++- >> drivers/gpu/drm/drm_bridge_connector.c | 16 ++++----- >> include/drm/drm_bridge.h | 33 ++++++++++++++++++ >> 3 files changed, 86 insertions(+), 9 deletions(-) -- Jani Nikula, Intel
