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@linux.intel.com
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