Re: [PATCH 0/2] drm/bridge: start moving towards struct drm_edid

[Jani Nikula <jani.nikula@intel.com>]

On Thu, 26 Oct 2023, Jani Nikula <jani.nikula@intel.com> 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.

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