On Tue, May 03, 2022 at 12:23:45PM +0300, Jani Nikula wrote: > I've kind of lost track of the version numbers on some of the iterator > patches, but this is the next version (or mostly a resend) of > [1]. There's an additional rename patch for SCDS. > > BR, > Jani. > > > [1] https://patchwork.freedesktop.org/series/102703/ > > > Jani Nikula (19): > drm/edid: reset display info in drm_add_edid_modes() for NULL edid > drm/edid: rename HDMI Forum VSDB to SCDS > drm/edid: clean up CTA data block tag definitions > drm/edid: add iterator for EDID base and extension blocks > drm/edid: add iterator for CTA data blocks > drm/edid: clean up cea_db_is_*() functions > drm/edid: convert add_cea_modes() to use cea db iter > drm/edid: convert drm_edid_to_speaker_allocation() to use cea db iter > drm/edid: convert drm_edid_to_sad() to use cea db iter > drm/edid: convert drm_detect_hdmi_monitor() to use cea db iter > drm/edid: convert drm_detect_monitor_audio() to use cea db iter > drm/edid: convert drm_parse_cea_ext() to use cea db iter > drm/edid: convert drm_edid_to_eld() to use cea db iter > drm/edid: sunset the old unused cea data block iterators > drm/edid: restore some type safety to cea_db_*() functions > drm/edid: detect basic audio in all CEA extensions > drm/edid: skip CTA extension scan in drm_edid_to_eld() just for CTA > rev > drm/edid: sunset drm_find_cea_extension() > > Lee Shawn C (1): > drm/edid: check for HF-SCDB block All of the above patches look OK to me. Reviewed-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > drm/edid: detect color formats and CTA revision in all CTA extensions For this one I'm not entirely convinced the behavioural change for the no-CTA ext case is what we want. Replied to that one individually. > > drivers/gpu/drm/drm_edid.c | 799 +++++++++++++++++++++---------------- > 1 file changed, 458 insertions(+), 341 deletions(-) > > -- > 2.30.2 -- Ville Syrjälä Intel
