Iterate through all CEA data blocks across all CEA extensions and
DisplayID data blocks.

Signed-off-by: Jani Nikula <jani.nik...@intel.com>
---
 drivers/gpu/drm/drm_edid.c | 25 +++++++++++++------------
 1 file changed, 13 insertions(+), 12 deletions(-)

diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
index 77763d94dd93..d4bf9ef09c3c 100644
--- a/drivers/gpu/drm/drm_edid.c
+++ b/drivers/gpu/drm/drm_edid.c
@@ -5342,8 +5342,9 @@ static void drm_parse_cea_ext(struct drm_connector 
*connector,
                              const struct edid *edid)
 {
        struct drm_display_info *info = &connector->display_info;
+       const struct cea_db *db;
+       struct cea_db_iter iter;
        const u8 *edid_ext;
-       int i, start, end;
 
        edid_ext = drm_find_cea_extension(edid);
        if (!edid_ext)
@@ -5358,25 +5359,25 @@ static void drm_parse_cea_ext(struct drm_connector 
*connector,
        if (edid_ext[3] & EDID_CEA_YCRCB422)
                info->color_formats |= DRM_COLOR_FORMAT_YCBCR422;
 
-       if (cea_db_offsets(edid_ext, &start, &end))
-               return;
-
-       for_each_cea_db(edid_ext, i, start, end) {
-               const u8 *db = &edid_ext[i];
+       cea_db_iter_edid_begin(edid, &iter);
+       cea_db_iter_for_each(db, &iter) {
+               /* FIXME: convert parsers to use struct cea_db */
+               const u8 *data = (const u8 *)db;
 
                if (cea_db_is_hdmi_vsdb(db))
-                       drm_parse_hdmi_vsdb_video(connector, db);
+                       drm_parse_hdmi_vsdb_video(connector, data);
                if (cea_db_is_hdmi_forum_vsdb(db))
-                       drm_parse_hdmi_forum_vsdb(connector, db);
+                       drm_parse_hdmi_forum_vsdb(connector, data);
                if (cea_db_is_microsoft_vsdb(db))
-                       drm_parse_microsoft_vsdb(connector, db);
+                       drm_parse_microsoft_vsdb(connector, data);
                if (cea_db_is_y420cmdb(db))
-                       drm_parse_y420cmdb_bitmap(connector, db);
+                       drm_parse_y420cmdb_bitmap(connector, data);
                if (cea_db_is_vcdb(db))
-                       drm_parse_vcdb(connector, db);
+                       drm_parse_vcdb(connector, data);
                if (cea_db_is_hdmi_hdr_metadata_block(db))
-                       drm_parse_hdr_metadata_block(connector, db);
+                       drm_parse_hdr_metadata_block(connector, data);
        }
+       cea_db_iter_end(&iter);
 }
 
 static
-- 
2.30.2

Reply via email to