[PATCH 1/3] drm/edid: parse multiple CEA extension block

List overview All Threads
Download

newer

older

[PATCH] drm: rcar-du: Fix CRTC...

[PATCH] dma-buf: check the return...

Lee Shawn C

22 Feb 2022 22 Feb '22

6:38 a.m.

Try to find and parse more CEA ext blocks if edid->extensions is greater than one.

Cc: Jani Nikula jani.nikula@linux.intel.com Cc: Ville Syrjala ville.syrjala@linux.intel.com Cc: Ankit Nautiyal ankit.k.nautiyal@intel.com Signed-off-by: Lee Shawn C shawn.c.lee@intel.com --- drivers/gpu/drm/drm_edid.c | 75 +++++++++++++++++++++++--------------- 1 file changed, 45 insertions(+), 30 deletions(-)

diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index 12893e7be89b..3d5dbbeca7f9 100644 --- a/drivers/gpu/drm/drm_edid.c +++ b/drivers/gpu/drm/drm_edid.c @@ -4313,43 +4313,58 @@ add_cea_modes(struct drm_connector *connector, struct edid *edid) const u8 *cea = drm_find_cea_extension(edid); const u8 *db, *hdmi = NULL, *video = NULL; u8 dbl, hdmi_len, video_len = 0; - int modes = 0; + int modes = 0, j;

- if (cea && cea_revision(cea) >= 3) { - int i, start, end; + if (!cea) + return 0;

- if (cea_db_offsets(cea, &start, &end)) - return 0; + for (j = (cea - (u8 *)edid) / EDID_LENGTH; j <= edid->extensions;) { + if (cea && cea_revision(cea) >= 3) { + int i, start, end;

- for_each_cea_db(cea, i, start, end) { - db = &cea[i]; - dbl = cea_db_payload_len(db); + if (cea_db_offsets(cea, &start, &end)) + continue;

- if (cea_db_tag(db) == VIDEO_BLOCK) { - video = db + 1; - video_len = dbl; - modes += do_cea_modes(connector, video, dbl); - } else if (cea_db_is_hdmi_vsdb(db)) { - hdmi = db; - hdmi_len = dbl; - } else if (cea_db_is_y420vdb(db)) { - const u8 *vdb420 = &db[2]; - - /* Add 4:2:0(only) modes present in EDID */ - modes += do_y420vdb_modes(connector, - vdb420, - dbl - 1); + for_each_cea_db(cea, i, start, end) { + db = &cea[i]; + dbl = cea_db_payload_len(db); + + if (cea_db_tag(db) == VIDEO_BLOCK) { + video = db + 1; + video_len = dbl; + modes += do_cea_modes(connector, video, dbl); + } else if (cea_db_is_hdmi_vsdb(db)) { + hdmi = db; + hdmi_len = dbl; + } else if (cea_db_is_y420vdb(db)) { + const u8 *vdb420 = &db[2]; + + /* Add 4:2:0(only) modes present in EDID */ + modes += do_y420vdb_modes(connector, + vdb420, + dbl - 1); + } } } - }

- /* - * We parse the HDMI VSDB after having added the cea modes as we will - * be patching their flags when the sink supports stereo 3D. - */ - if (hdmi) - modes += do_hdmi_vsdb_modes(connector, hdmi, hdmi_len, video, - video_len); + /* + * We parse the HDMI VSDB after having added the cea modes as we will + * be patching their flags when the sink supports stereo 3D. + */ + if (hdmi) { + modes += do_hdmi_vsdb_modes(connector, hdmi, hdmi_len, video, + video_len); + hdmi = NULL; + video = NULL; + hdmi_len = 0; + video_len = 0; + } + + /* move to next CEA extension block */ + cea = drm_find_edid_extension(edid, CEA_EXT, &j); + if (!cea) + break; + }

return modes; }

-- 2.17.1

Show replies by date

Lee Shawn C

22 Feb 22 Feb

6:38 a.m.

New subject: [PATCH 2/3] drm/edid: read HF-EEODB ext block

Support to read HF_EEODB block that request by HDMI 2.1 specification.

Cc: Jani Nikula jani.nikula@linux.intel.com Cc: Ville Syrjala ville.syrjala@linux.intel.com Cc: Ankit Nautiyal ankit.k.nautiyal@intel.com Signed-off-by: Lee Shawn C shawn.c.lee@intel.com --- drivers/gpu/drm/drm_connector.c | 5 ++- drivers/gpu/drm/drm_edid.c | 76 ++++++++++++++++++++++++++++++--- include/drm/drm_edid.h | 2 + 3 files changed, 77 insertions(+), 6 deletions(-)

diff --git a/drivers/gpu/drm/drm_connector.c b/drivers/gpu/drm/drm_connector.c index a50c82bc2b2f..0f9e3ef00be7 100644 --- a/drivers/gpu/drm/drm_connector.c +++ b/drivers/gpu/drm/drm_connector.c @@ -2137,8 +2137,11 @@ int drm_connector_update_edid_property(struct drm_connector *connector, if (connector->override_edid) return 0;

- if (edid) + if (edid) { size = EDID_LENGTH * (1 + edid->extensions); + if (drm_edid_is_hf_eeodb_blk_available(edid)) + size = EDID_LENGTH * (1 + drm_edid_read_hf_eeodb_blk_size(edid)); + }

/* Set the display info, using edid if available, otherwise * resetting the values to defaults. This duplicates the work diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index 3d5dbbeca7f9..a7391e427d69 100644 --- a/drivers/gpu/drm/drm_edid.c +++ b/drivers/gpu/drm/drm_edid.c @@ -1991,7 +1991,7 @@ struct edid *drm_do_get_edid(struct drm_connector *connector, void *data) { int i, j = 0, valid_extensions = 0; - u8 *edid, *new; + u8 *edid, *new, ext_eeodb_blk_size; struct edid *override;

override = drm_get_override_edid(connector); @@ -2051,7 +2051,40 @@ struct edid *drm_do_get_edid(struct drm_connector *connector, }

kfree(edid); + return (struct edid *)new; + } + + if (drm_edid_is_hf_eeodb_blk_available((struct edid *)edid)) { + ext_eeodb_blk_size = drm_edid_read_hf_eeodb_blk_size((struct edid *)edid); + + // no more ext blk wait for read + if (ext_eeodb_blk_size <= 1) + return (struct edid *)edid; + + new = krealloc(edid, (ext_eeodb_blk_size + 1) * EDID_LENGTH, GFP_KERNEL); + if (!new) + goto out; edid = new; + + valid_extensions = ext_eeodb_blk_size - 1; + for (j = 2; j <= ext_eeodb_blk_size; j++) { + u8 *block = edid + j * EDID_LENGTH; + + for (i = 0; i < 4; i++) { + if (get_edid_block(data, block, j, EDID_LENGTH)) + goto out; + if (drm_edid_block_valid(block, j, false, NULL)) + break; + } + + if (i == 4) + valid_extensions--; + } + + if (valid_extensions != ext_eeodb_blk_size - 1) { + DRM_ERROR("Not able to retrieve proper EDID contain HF-EEODB data.\n"); + goto out; + } }

return (struct edid *)edid; @@ -3315,15 +3348,17 @@ add_detailed_modes(struct drm_connector *connector, struct edid *edid, #define VIDEO_BLOCK 0x02 #define VENDOR_BLOCK 0x03 #define SPEAKER_BLOCK 0x04 -#define HDR_STATIC_METADATA_BLOCK 0x6 -#define USE_EXTENDED_TAG 0x07 -#define EXT_VIDEO_CAPABILITY_BLOCK 0x00 +#define EXT_VIDEO_CAPABILITY_BLOCK 0x00 +#define HDR_STATIC_METADATA_BLOCK 0x06 +#define USE_EXTENDED_TAG 0x07 #define EXT_VIDEO_DATA_BLOCK_420 0x0E -#define EXT_VIDEO_CAP_BLOCK_Y420CMDB 0x0F +#define EXT_VIDEO_CAP_BLOCK_Y420CMDB 0x0F +#define EXT_VIDEO_HF_EEODB_DATA_BLOCK 0x78 #define EDID_BASIC_AUDIO (1 << 6) #define EDID_CEA_YCRCB444 (1 << 5) #define EDID_CEA_YCRCB422 (1 << 4) #define EDID_CEA_VCDB_QS (1 << 6) +#define HF_EEODB_LENGTH 2

/* * Search EDID for CEA extension block. @@ -4222,6 +4257,20 @@ static bool cea_db_is_hdmi_forum_vsdb(const u8 *db) return oui(db[3], db[2], db[1]) == HDMI_FORUM_IEEE_OUI; }

+static bool cea_db_is_hdmi_forum_eeodb(const u8 *db) +{ + if (cea_db_tag(db) != USE_EXTENDED_TAG) + return false; + + if (cea_db_payload_len(db) != HF_EEODB_LENGTH) + return false; + + if (cea_db_extended_tag(db) != EXT_VIDEO_HF_EEODB_DATA_BLOCK) + return false; + + return true; +} + static bool cea_db_is_vcdb(const u8 *db) { if (cea_db_tag(db) != USE_EXTENDED_TAG) @@ -4264,6 +4313,23 @@ static bool cea_db_is_y420vdb(const u8 *db) return true; }

+bool drm_edid_is_hf_eeodb_blk_available(const struct edid *edid) +{ + const u8 *eeodb_header = (u8 *)edid + EDID_LENGTH + 4; + + if (!edid->extensions) + return false; + + return cea_db_is_hdmi_forum_eeodb(eeodb_header); +} +EXPORT_SYMBOL_GPL(drm_edid_is_hf_eeodb_blk_available); + +u8 drm_edid_read_hf_eeodb_blk_size(const struct edid *edid) +{ + return ((u8 *)edid)[EDID_LENGTH + 6]; +} +EXPORT_SYMBOL_GPL(drm_edid_read_hf_eeodb_blk_size); + #define for_each_cea_db(cea, i, start, end) \ for ((i) = (start); (i) < (end) && (i) + cea_db_payload_len(&(cea)[(i)]) < (end); (i) += cea_db_payload_len(&(cea)[(i)]) + 1)

diff --git a/include/drm/drm_edid.h b/include/drm/drm_edid.h index 18f6c700f6d0..ba2812432ead 100644 --- a/include/drm/drm_edid.h +++ b/include/drm/drm_edid.h @@ -593,5 +593,7 @@ drm_display_mode_from_cea_vic(struct drm_device *dev, const u8 *drm_find_edid_extension(const struct edid *edid, int ext_id, int *ext_index);

+bool drm_edid_is_hf_eeodb_blk_available(const struct edid *edid); +u8 drm_edid_read_hf_eeodb_blk_size(const struct edid *edid);

#endif /* __DRM_EDID_H__ */

-- 2.17.1

Lee Shawn C

6:38 a.m.

New subject: [PATCH 3/3] drm/edid: parse HF-EEODB CEA extension block

While adding CEA modes, try to get available EEODB block number. Then based on it to parse numbers of ext blocks, retrieve CEA information and add more CEA modes.

Cc: Jani Nikula jani.nikula@linux.intel.com Cc: Ville Syrjala ville.syrjala@linux.intel.com Cc: Ankit Nautiyal ankit.k.nautiyal@intel.com Signed-off-by: Lee Shawn C shawn.c.lee@intel.com --- drivers/gpu/drm/drm_displayid.c | 2 +- drivers/gpu/drm/drm_edid.c | 19 +++++++++++-------- include/drm/drm_edid.h | 2 +- 3 files changed, 13 insertions(+), 10 deletions(-)

diff --git a/drivers/gpu/drm/drm_displayid.c b/drivers/gpu/drm/drm_displayid.c index 32da557b960f..ef0dfc9fa6f9 100644 --- a/drivers/gpu/drm/drm_displayid.c +++ b/drivers/gpu/drm/drm_displayid.c @@ -37,7 +37,7 @@ static const u8 *drm_find_displayid_extension(const struct edid *edid, int *length, int *idx, int *ext_index) { - const u8 *displayid = drm_find_edid_extension(edid, DISPLAYID_EXT, ext_index); + const u8 *displayid = drm_find_edid_extension(edid, edid->extensions, DISPLAYID_EXT, ext_index); const struct displayid_header *base; int ret;

diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index a7391e427d69..9a987c77f203 100644 --- a/drivers/gpu/drm/drm_edid.c +++ b/drivers/gpu/drm/drm_edid.c @@ -3364,23 +3364,23 @@ add_detailed_modes(struct drm_connector *connector, struct edid *edid, * Search EDID for CEA extension block. */ const u8 *drm_find_edid_extension(const struct edid *edid, - int ext_id, int *ext_index) + int num_ext_blk, int ext_id, int *ext_index) { const u8 *edid_ext = NULL; int i;

/* No EDID or EDID extensions */ - if (edid == NULL || edid->extensions == 0) + if (edid == NULL || edid->extensions == 0 || *ext_index >= num_ext_blk) return NULL;

/* Find CEA extension */ - for (i = *ext_index; i < edid->extensions; i++) { + for (i = *ext_index; i < num_ext_blk; i++) { edid_ext = (const u8 *)edid + EDID_LENGTH * (i + 1); if (edid_ext[0] == ext_id) break; }

- if (i >= edid->extensions) + if (i >= num_ext_blk) return NULL;

*ext_index = i + 1; @@ -3397,7 +3397,7 @@ static const u8 *drm_find_cea_extension(const struct edid *edid)

/* Look for a top level CEA extension block */ /* FIXME: make callers iterate through multiple CEA ext blocks? */ - cea = drm_find_edid_extension(edid, CEA_EXT, &ext_index); + cea = drm_find_edid_extension(edid, edid->extensions, CEA_EXT, &ext_index); if (cea) return cea;

@@ -4378,13 +4378,16 @@ add_cea_modes(struct drm_connector *connector, struct edid *edid) { const u8 *cea = drm_find_cea_extension(edid); const u8 *db, *hdmi = NULL, *video = NULL; - u8 dbl, hdmi_len, video_len = 0; + u8 dbl, hdmi_len, video_len = 0, num_ext_blk = edid->extensions; int modes = 0, j;

if (!cea) return 0;

- for (j = (cea - (u8 *)edid) / EDID_LENGTH; j <= edid->extensions;) { + if (num_ext_blk && drm_edid_is_hf_eeodb_blk_available(edid)) + num_ext_blk = drm_edid_read_hf_eeodb_blk_size(edid); + + for (j = (cea - (u8 *)edid) / EDID_LENGTH; j <= num_ext_blk;) { if (cea && cea_revision(cea) >= 3) { int i, start, end;

@@ -4427,7 +4430,7 @@ add_cea_modes(struct drm_connector *connector, struct edid *edid) }

/* move to next CEA extension block */ - cea = drm_find_edid_extension(edid, CEA_EXT, &j); + cea = drm_find_edid_extension(edid, num_ext_blk, CEA_EXT, &j); if (!cea) break; } diff --git a/include/drm/drm_edid.h b/include/drm/drm_edid.h index ba2812432ead..a9c2708b63a1 100644 --- a/include/drm/drm_edid.h +++ b/include/drm/drm_edid.h @@ -591,7 +591,7 @@ struct drm_display_mode * drm_display_mode_from_cea_vic(struct drm_device *dev, u8 video_code); const u8 *drm_find_edid_extension(const struct edid *edid, - int ext_id, int *ext_index); + int num_ext_blk, int ext_id, int *ext_index);

bool drm_edid_is_hf_eeodb_blk_available(const struct edid *edid); u8 drm_edid_read_hf_eeodb_blk_size(const struct edid *edid);

-- 2.17.1

Ville Syrjälä

7:28 a.m.

On Tue, Feb 22, 2022 at 02:38:17PM +0800, Lee Shawn C wrote:

...

Try to find and parse more CEA ext blocks if edid->extensions is greater than one.

Cc: Jani Nikula jani.nikula@linux.intel.com Cc: Ville Syrjala ville.syrjala@linux.intel.com Cc: Ankit Nautiyal ankit.k.nautiyal@intel.com Signed-off-by: Lee Shawn C shawn.c.lee@intel.com

drivers/gpu/drm/drm_edid.c | 75 +++++++++++++++++++++++--------------- 1 file changed, 45 insertions(+), 30 deletions(-)

diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index 12893e7be89b..3d5dbbeca7f9 100644 --- a/drivers/gpu/drm/drm_edid.c +++ b/drivers/gpu/drm/drm_edid.c @@ -4313,43 +4313,58 @@ add_cea_modes(struct drm_connector *connector, struct edid *edid) const u8 *cea = drm_find_cea_extension(edid); const u8 *db, *hdmi = NULL, *video = NULL; u8 dbl, hdmi_len, video_len = 0;

int modes = 0;

int modes = 0, j;
if (cea && cea_revision(cea) >= 3) {
int i, start, end;
if (!cea)
return 0;
if (cea_db_offsets(cea, &start, &end))
	return 0;
for (j = (cea - (u8 *)edid) / EDID_LENGTH; j <= edid->extensions;) {

That looks rather illegible. I think we want a drm_find_cea_extension(const struct edid *edid, int *ext_index) and then just loop until it stops giving us stuff.

There are also several other callers of drm_find_cea_extension(). Why don't they require the same treatment?

...

```
if (cea && cea_revision(cea) >= 3) {
```
```
	int i, start, end;
```

```
for_each_cea_db(cea, i, start, end) {
```
```
	db = &cea[i];
```
```
	dbl = cea_db_payload_len(db);
```

	if (cea_db_offsets(cea, &start, &end))

```
		continue;
```

```
	if (cea_db_tag(db) == VIDEO_BLOCK) {
```
```
		video = db + 1;
```
```
		video_len = dbl;
```

		modes += do_cea_modes(connector, video, dbl);

```
	} else if (cea_db_is_hdmi_vsdb(db)) {
```
```
		hdmi = db;
```
```
		hdmi_len = dbl;
```
```
	} else if (cea_db_is_y420vdb(db)) {
```
```
		const u8 *vdb420 = &db[2];
```

		/* Add 4:2:0(only) modes present in EDID */

```
		modes += do_y420vdb_modes(connector,
```
```
					  vdb420,
```
```
					  dbl - 1);
```

```
	for_each_cea_db(cea, i, start, end) {
```
```
		db = &cea[i];
```
```
		dbl = cea_db_payload_len(db);
```
```
		if (cea_db_tag(db) == VIDEO_BLOCK) {
```
```
			video = db + 1;
```
```
			video_len = dbl;
```

			modes += do_cea_modes(connector, video, dbl);

		} else if (cea_db_is_hdmi_vsdb(db)) {

```
			hdmi = db;
```
```
			hdmi_len = dbl;
```
```
		} else if (cea_db_is_y420vdb(db)) {
```
```
			const u8 *vdb420 = &db[2];
```

			/* Add 4:2:0(only) modes present in EDID */

			modes += do_y420vdb_modes(connector,

```
						  vdb420,
```
```
						  dbl - 1);
```
```
		}
}
```
}

}
/*

* We parse the HDMI VSDB after having added the cea modes as we will

* be patching their flags when the sink supports stereo 3D.

```
*/
```
if (hdmi)

modes += do_hdmi_vsdb_modes(connector, hdmi, hdmi_len, video,

```
			    video_len);
```

```
/*
```

 * We parse the HDMI VSDB after having added the cea modes as we will

 * be patching their flags when the sink supports stereo 3D.

```
 */
```
```
if (hdmi) {
```

	modes += do_hdmi_vsdb_modes(connector, hdmi, hdmi_len, video,

```
				    video_len);
```
```
	hdmi  = NULL;
```
```
	video = NULL;
```
```
	hdmi_len = 0;
```
```
	video_len = 0;
```
```
}
```
```
/* move to next CEA extension block */
```

cea = drm_find_edid_extension(edid, CEA_EXT, &j);

```
if (!cea)
```
```
	break;
```
}

return modes;

}

2.17.1

-- Ville Syrjälä Intel

Lee, Shawn C

8:49 a.m.

On Tue, Feb 22, 2022 at 03:28:17PM +0800, Ville Syrjälä ville.syrjala@linux.intel.com wrote:

...

On Tue, Feb 22, 2022 at 02:38:17PM +0800, Lee Shawn C wrote:

...
Try to find and parse more CEA ext blocks if edid->extensions is greater than one.

Cc: Jani Nikula jani.nikula@linux.intel.com Cc: Ville Syrjala ville.syrjala@linux.intel.com Cc: Ankit Nautiyal ankit.k.nautiyal@intel.com Signed-off-by: Lee Shawn C shawn.c.lee@intel.com

drivers/gpu/drm/drm_edid.c | 75 +++++++++++++++++++++++--------------- 1 file changed, 45 insertions(+), 30 deletions(-)

diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index 12893e7be89b..3d5dbbeca7f9 100644 --- a/drivers/gpu/drm/drm_edid.c +++ b/drivers/gpu/drm/drm_edid.c @@ -4313,43 +4313,58 @@ add_cea_modes(struct drm_connector *connector, struct edid *edid) const u8 *cea = drm_find_cea_extension(edid); const u8 *db, *hdmi = NULL, *video = NULL; u8 dbl, hdmi_len, video_len = 0;

int modes = 0;

int modes = 0, j;
if (cea && cea_revision(cea) >= 3) {
int i, start, end;
if (!cea)
return 0;
if (cea_db_offsets(cea, &start, &end))
	return 0;
for (j = (cea - (u8 *)edid) / EDID_LENGTH; j <= edid->extensions;) {
That looks rather illegible. I think we want a drm_find_cea_extension(const struct edid *edid, int *ext_index) and then just loop until it stops giving us stuff.

I will modify drm_find_cea_extension() to find out next CEA from *ext_index.

...

There are also several other callers of drm_find_cea_extension(). Why don't they require the same treatment?

My suspicion is the original design judge edid->extension would be zero or one. And only one extension block would available.

Best regards, Shawn

...

```
if (cea && cea_revision(cea) >= 3) {
```
```
	int i, start, end;
```

```
for_each_cea_db(cea, i, start, end) {
```
```
	db = &cea[i];
```
```
	dbl = cea_db_payload_len(db);
```

	if (cea_db_offsets(cea, &start, &end))

```
		continue;
```

```
	if (cea_db_tag(db) == VIDEO_BLOCK) {
```
```
		video = db + 1;
```
```
		video_len = dbl;
```

		modes += do_cea_modes(connector, video, dbl);

```
	} else if (cea_db_is_hdmi_vsdb(db)) {
```
```
		hdmi = db;
```
```
		hdmi_len = dbl;
```
```
	} else if (cea_db_is_y420vdb(db)) {
```
```
		const u8 *vdb420 = &db[2];
```

		/* Add 4:2:0(only) modes present in EDID */

```
		modes += do_y420vdb_modes(connector,
```
```
					  vdb420,
```
```
					  dbl - 1);
```

```
	for_each_cea_db(cea, i, start, end) {
```
```
		db = &cea[i];
```
```
		dbl = cea_db_payload_len(db);
```
```
		if (cea_db_tag(db) == VIDEO_BLOCK) {
```
```
			video = db + 1;
```
```
			video_len = dbl;
```

			modes += do_cea_modes(connector, video, dbl);

		} else if (cea_db_is_hdmi_vsdb(db)) {

```
			hdmi = db;
```
```
			hdmi_len = dbl;
```
```
		} else if (cea_db_is_y420vdb(db)) {
```
```
			const u8 *vdb420 = &db[2];
```

			/* Add 4:2:0(only) modes present in EDID */

			modes += do_y420vdb_modes(connector,

```
						  vdb420,
```
```
						  dbl - 1);
```
```
		}
}
```
}

}
/*

* We parse the HDMI VSDB after having added the cea modes as we will

* be patching their flags when the sink supports stereo 3D.

```
*/
```
if (hdmi)

modes += do_hdmi_vsdb_modes(connector, hdmi, hdmi_len, video,

```
			    video_len);
```

```
/*
```

 * We parse the HDMI VSDB after having added the cea modes as we will

 * be patching their flags when the sink supports stereo 3D.

```
 */
```
```
if (hdmi) {
```

	modes += do_hdmi_vsdb_modes(connector, hdmi, hdmi_len, video,

```
				    video_len);
```
```
	hdmi  = NULL;
```
```
	video = NULL;
```
```
	hdmi_len = 0;
```
```
	video_len = 0;
```
```
}
```
```
/* move to next CEA extension block */
```

cea = drm_find_edid_extension(edid, CEA_EXT, &j);

```
if (!cea)
```
```
	break;
```
}

return modes;

}

2.17.1

-- Ville Syrjälä Intel

Jani Nikula

9:19 a.m.

On Tue, 22 Feb 2022, Ville Syrjälä ville.syrjala@linux.intel.com wrote:

...

On Tue, Feb 22, 2022 at 02:38:17PM +0800, Lee Shawn C wrote:

...
Try to find and parse more CEA ext blocks if edid->extensions is greater than one.

Cc: Jani Nikula jani.nikula@linux.intel.com Cc: Ville Syrjala ville.syrjala@linux.intel.com Cc: Ankit Nautiyal ankit.k.nautiyal@intel.com Signed-off-by: Lee Shawn C shawn.c.lee@intel.com

drivers/gpu/drm/drm_edid.c | 75 +++++++++++++++++++++++--------------- 1 file changed, 45 insertions(+), 30 deletions(-)

diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index 12893e7be89b..3d5dbbeca7f9 100644 --- a/drivers/gpu/drm/drm_edid.c +++ b/drivers/gpu/drm/drm_edid.c @@ -4313,43 +4313,58 @@ add_cea_modes(struct drm_connector *connector, struct edid *edid) const u8 *cea = drm_find_cea_extension(edid); const u8 *db, *hdmi = NULL, *video = NULL; u8 dbl, hdmi_len, video_len = 0;

int modes = 0;

int modes = 0, j;
if (cea && cea_revision(cea) >= 3) {
int i, start, end;
if (!cea)
return 0;
if (cea_db_offsets(cea, &start, &end))
	return 0;
for (j = (cea - (u8 *)edid) / EDID_LENGTH; j <= edid->extensions;) {
That looks rather illegible. I think we want a drm_find_cea_extension(const struct edid *edid, int *ext_index) and then just loop until it stops giving us stuff.

Neither approach takes multiple CEA blocks within DisplayID extension into account. Or some blocks outside and some inside DisplayID extension.

I think we're going to need abstracted EDID iteration similar to what I've done for DisplayID iteration. We can't have all places reimplementing the iteration like we have now.

BR, Jani.

...

There are also several other callers of drm_find_cea_extension(). Why don't they require the same treatment?

...

```
if (cea && cea_revision(cea) >= 3) {
```
```
	int i, start, end;
```

```
for_each_cea_db(cea, i, start, end) {
```
```
	db = &cea[i];
```
```
	dbl = cea_db_payload_len(db);
```

	if (cea_db_offsets(cea, &start, &end))

```
		continue;
```

```
	if (cea_db_tag(db) == VIDEO_BLOCK) {
```
```
		video = db + 1;
```
```
		video_len = dbl;
```

		modes += do_cea_modes(connector, video, dbl);

```
	} else if (cea_db_is_hdmi_vsdb(db)) {
```
```
		hdmi = db;
```
```
		hdmi_len = dbl;
```
```
	} else if (cea_db_is_y420vdb(db)) {
```
```
		const u8 *vdb420 = &db[2];
```

		/* Add 4:2:0(only) modes present in EDID */

```
		modes += do_y420vdb_modes(connector,
```
```
					  vdb420,
```
```
					  dbl - 1);
```

```
	for_each_cea_db(cea, i, start, end) {
```
```
		db = &cea[i];
```
```
		dbl = cea_db_payload_len(db);
```
```
		if (cea_db_tag(db) == VIDEO_BLOCK) {
```
```
			video = db + 1;
```
```
			video_len = dbl;
```

			modes += do_cea_modes(connector, video, dbl);

		} else if (cea_db_is_hdmi_vsdb(db)) {

```
			hdmi = db;
```
```
			hdmi_len = dbl;
```
```
		} else if (cea_db_is_y420vdb(db)) {
```
```
			const u8 *vdb420 = &db[2];
```

			/* Add 4:2:0(only) modes present in EDID */

			modes += do_y420vdb_modes(connector,

```
						  vdb420,
```
```
						  dbl - 1);
```
```
		}
}
```
}

}
/*

* We parse the HDMI VSDB after having added the cea modes as we will

* be patching their flags when the sink supports stereo 3D.

```
*/
```
if (hdmi)

modes += do_hdmi_vsdb_modes(connector, hdmi, hdmi_len, video,

```
			    video_len);
```

```
/*
```

 * We parse the HDMI VSDB after having added the cea modes as we will

 * be patching their flags when the sink supports stereo 3D.

```
 */
```
```
if (hdmi) {
```

	modes += do_hdmi_vsdb_modes(connector, hdmi, hdmi_len, video,

```
				    video_len);
```
```
	hdmi  = NULL;
```
```
	video = NULL;
```
```
	hdmi_len = 0;
```
```
	video_len = 0;
```
```
}
```
```
/* move to next CEA extension block */
```

cea = drm_find_edid_extension(edid, CEA_EXT, &j);

```
if (!cea)
```
```
	break;
```
}

return modes;

}

2.17.1

-- Jani Nikula, Intel Open Source Graphics Center

Ville Syrjälä

9:43 a.m.

On Tue, Feb 22, 2022 at 11:19:15AM +0200, Jani Nikula wrote:

...

On Tue, 22 Feb 2022, Ville Syrjälä ville.syrjala@linux.intel.com wrote:

...
On Tue, Feb 22, 2022 at 02:38:17PM +0800, Lee Shawn C wrote:

...
Try to find and parse more CEA ext blocks if edid->extensions is greater than one.

Cc: Jani Nikula jani.nikula@linux.intel.com Cc: Ville Syrjala ville.syrjala@linux.intel.com Cc: Ankit Nautiyal ankit.k.nautiyal@intel.com Signed-off-by: Lee Shawn C shawn.c.lee@intel.com

drivers/gpu/drm/drm_edid.c | 75 +++++++++++++++++++++++--------------- 1 file changed, 45 insertions(+), 30 deletions(-)

diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index 12893e7be89b..3d5dbbeca7f9 100644 --- a/drivers/gpu/drm/drm_edid.c +++ b/drivers/gpu/drm/drm_edid.c @@ -4313,43 +4313,58 @@ add_cea_modes(struct drm_connector *connector, struct edid *edid) const u8 *cea = drm_find_cea_extension(edid); const u8 *db, *hdmi = NULL, *video = NULL; u8 dbl, hdmi_len, video_len = 0;

int modes = 0;

int modes = 0, j;
if (cea && cea_revision(cea) >= 3) {
int i, start, end;
if (!cea)
return 0;
if (cea_db_offsets(cea, &start, &end))
	return 0;
for (j = (cea - (u8 *)edid) / EDID_LENGTH; j <= edid->extensions;) {
That looks rather illegible. I think we want a drm_find_cea_extension(const struct edid *edid, int *ext_index) and then just loop until it stops giving us stuff.
Neither approach takes multiple CEA blocks within DisplayID extension into account. Or some blocks outside and some inside DisplayID extension.

I think we're going to need abstracted EDID iteration similar to what I've done for DisplayID iteration. We can't have all places reimplementing the iteration like we have now.

Aye. We need so many layers of iteration in various places that the whole thing is starting to resemble a Russian doll. Following a common form should probably make that a lot more manageable.

I've been already thinking about introducing an iterator for the cea db stuff. But the EDID ext blocks is definitely another target we need to look at.

And someone is going to have to figure out what are all the ways these need to nest. I suppose the high level code should only have to care about the deepest layer of stuff and the iterators should take care to iterate through all the potential containers? Eg. if the high level code wants to look at cea dbs then it just iterates those and shouldn't have to care at all where they are stored.

-- Ville Syrjälä Intel

Lee Shawn C

2:02 p.m.

New subject: [v2 1/3] drm/edid: parse multiple CEA extension block

Try to find and parse more CEA ext blocks if edid->extensions is greater than one.

v2: add ext_index and look for available CEA block from ext_index in drm_find_cea_extension().

Cc: Jani Nikula jani.nikula@linux.intel.com Cc: Ville Syrjala ville.syrjala@linux.intel.com Cc: Ankit Nautiyal ankit.k.nautiyal@intel.com Signed-off-by: Lee Shawn C shawn.c.lee@intel.com --- drivers/gpu/drm/drm_edid.c | 110 ++++++++++++++++++++----------------- 1 file changed, 60 insertions(+), 50 deletions(-)

diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index 12893e7be89b..c83ef23757bd 100644 --- a/drivers/gpu/drm/drm_edid.c +++ b/drivers/gpu/drm/drm_edid.c @@ -3353,16 +3353,14 @@ const u8 *drm_find_edid_extension(const struct edid *edid, return edid_ext; }

-static const u8 *drm_find_cea_extension(const struct edid *edid) +static const u8 *drm_find_cea_extension(const struct edid *edid, int *ext_index) { const struct displayid_block *block; struct displayid_iter iter; const u8 *cea; - int ext_index = 0;

- /* Look for a top level CEA extension block */ - /* FIXME: make callers iterate through multiple CEA ext blocks? */ - cea = drm_find_edid_extension(edid, CEA_EXT, &ext_index); + /* Look for a CEA extension block from ext_index */ + cea = drm_find_edid_extension(edid, CEA_EXT, ext_index); if (cea) return cea;

@@ -3643,10 +3641,10 @@ add_alternate_cea_modes(struct drm_connector *connector, struct edid *edid) struct drm_device *dev = connector->dev; struct drm_display_mode *mode, *tmp; LIST_HEAD(list); - int modes = 0; + int modes = 0, ext_index = 0;

/* Don't add CEA modes if the CEA extension block is missing */ - if (!drm_find_cea_extension(edid)) + if (!drm_find_cea_extension(edid, &ext_index)) return 0;

/* @@ -4310,46 +4308,58 @@ static void drm_parse_y420cmdb_bitmap(struct drm_connector *connector, static int add_cea_modes(struct drm_connector *connector, struct edid *edid) { - const u8 *cea = drm_find_cea_extension(edid); + const u8 *cea = NULL; const u8 *db, *hdmi = NULL, *video = NULL; u8 dbl, hdmi_len, video_len = 0; - int modes = 0; + int modes = 0, j = 0;

- if (cea && cea_revision(cea) >= 3) { - int i, start, end; + for (;;) { + cea = drm_find_cea_extension(edid, &j);

- if (cea_db_offsets(cea, &start, &end)) - return 0; + if (!cea) + break;

- for_each_cea_db(cea, i, start, end) { - db = &cea[i]; - dbl = cea_db_payload_len(db); + if (cea && cea_revision(cea) >= 3) { + int i, start, end; + + if (cea_db_offsets(cea, &start, &end)) + continue;

return modes; } @@ -4551,7 +4561,7 @@ static void drm_edid_to_eld(struct drm_connector *connector, struct edid *edid) uint8_t *eld = connector->eld; const u8 *cea; const u8 *db; - int total_sad_count = 0; + int total_sad_count = 0, ext_index = 0; int mnl; int dbl;

@@ -4560,7 +4570,7 @@ static void drm_edid_to_eld(struct drm_connector *connector, struct edid *edid) if (!edid) return;

- cea = drm_find_cea_extension(edid); + cea = drm_find_cea_extension(edid, &ext_index); if (!cea) { DRM_DEBUG_KMS("ELD: no CEA Extension found\n"); return; @@ -4644,11 +4654,11 @@ static void drm_edid_to_eld(struct drm_connector *connector, struct edid *edid) */ int drm_edid_to_sad(struct edid *edid, struct cea_sad **sads) { - int count = 0; + int count = 0, ext_index = 0; int i, start, end, dbl; const u8 *cea;

- cea = drm_find_cea_extension(edid); + cea = drm_find_cea_extension(edid, &ext_index); if (!cea) { DRM_DEBUG_KMS("SAD: no CEA Extension found\n"); return 0; @@ -4706,11 +4716,11 @@ EXPORT_SYMBOL(drm_edid_to_sad); */ int drm_edid_to_speaker_allocation(struct edid *edid, u8 **sadb) { - int count = 0; + int count = 0, ext_index = 0; int i, start, end, dbl; const u8 *cea;

- cea = drm_find_cea_extension(edid); + cea = drm_find_cea_extension(edid, &ext_index); if (!cea) { DRM_DEBUG_KMS("SAD: no CEA Extension found\n"); return 0; @@ -4802,10 +4812,10 @@ EXPORT_SYMBOL(drm_av_sync_delay); bool drm_detect_hdmi_monitor(struct edid *edid) { const u8 *edid_ext; - int i; + int i, ext_index = 0; int start_offset, end_offset;

- edid_ext = drm_find_cea_extension(edid); + edid_ext = drm_find_cea_extension(edid, &ext_index); if (!edid_ext) return false;

@@ -4840,11 +4850,11 @@ EXPORT_SYMBOL(drm_detect_hdmi_monitor); bool drm_detect_monitor_audio(struct edid *edid) { const u8 *edid_ext; - int i, j; + int i, j, ext_index = 0; bool has_audio = false; int start_offset, end_offset;

- edid_ext = drm_find_cea_extension(edid); + edid_ext = drm_find_cea_extension(edid, &ext_index); if (!edid_ext) goto end;

@@ -5154,9 +5164,9 @@ static void drm_parse_cea_ext(struct drm_connector *connector, { struct drm_display_info *info = &connector->display_info; const u8 *edid_ext; - int i, start, end; + int i, start, end, ext_index = 0;

- edid_ext = drm_find_cea_extension(edid); + edid_ext = drm_find_cea_extension(edid, &ext_index); if (!edid_ext) return;

-- 2.31.1

Lee Shawn C

2:02 p.m.

New subject: [v2 2/3] drm/edid: read HF-EEODB ext block

Support to read HF_EEODB block that request by HDMI 2.1 specification.

Signed-off-by: Lee Shawn C shawn.c.lee@intel.com --- drivers/gpu/drm/drm_connector.c | 5 ++- drivers/gpu/drm/drm_edid.c | 76 ++++++++++++++++++++++++++++++--- include/drm/drm_edid.h | 2 + 3 files changed, 77 insertions(+), 6 deletions(-)

- if (edid) + if (edid) { size = EDID_LENGTH * (1 + edid->extensions); + if (drm_edid_is_hf_eeodb_blk_available(edid)) + size = EDID_LENGTH * (1 + drm_edid_read_hf_eeodb_blk_size(edid)); + }

/* Set the display info, using edid if available, otherwise * resetting the values to defaults. This duplicates the work diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index c83ef23757bd..f8514783d089 100644 --- a/drivers/gpu/drm/drm_edid.c +++ b/drivers/gpu/drm/drm_edid.c @@ -1991,7 +1991,7 @@ struct edid *drm_do_get_edid(struct drm_connector *connector, void *data) { int i, j = 0, valid_extensions = 0; - u8 *edid, *new; + u8 *edid, *new, ext_eeodb_blk_size; struct edid *override;

override = drm_get_override_edid(connector); @@ -2051,7 +2051,40 @@ struct edid *drm_do_get_edid(struct drm_connector *connector, }

/* * Search EDID for CEA extension block. @@ -4220,6 +4255,20 @@ static bool cea_db_is_hdmi_forum_vsdb(const u8 *db) return oui(db[3], db[2], db[1]) == HDMI_FORUM_IEEE_OUI; }

+static bool cea_db_is_hdmi_forum_eeodb(const u8 *db) +{ + if (cea_db_tag(db) != USE_EXTENDED_TAG) + return false; + + if (cea_db_payload_len(db) != HF_EEODB_LENGTH) + return false; + + if (cea_db_extended_tag(db) != EXT_VIDEO_HF_EEODB_DATA_BLOCK) + return false; + + return true; +} + static bool cea_db_is_vcdb(const u8 *db) { if (cea_db_tag(db) != USE_EXTENDED_TAG) @@ -4262,6 +4311,23 @@ static bool cea_db_is_y420vdb(const u8 *db) return true; }

+bool drm_edid_is_hf_eeodb_blk_available(const struct edid *edid); +u8 drm_edid_read_hf_eeodb_blk_size(const struct edid *edid);

#endif /* __DRM_EDID_H__ */

-- 2.31.1

Lee Shawn C

2:02 p.m.

New subject: [v2 3/3] drm/edid: parse HF-EEODB CEA extension block

While adding CEA modes, try to get available EEODB block number. Then based on it to parse numbers of ext blocks, retrieve CEA information and add more CEA modes.

Signed-off-by: Lee Shawn C shawn.c.lee@intel.com --- drivers/gpu/drm/drm_displayid.c | 5 ++++- drivers/gpu/drm/drm_edid.c | 34 ++++++++++++++++++--------------- include/drm/drm_edid.h | 2 +- 3 files changed, 24 insertions(+), 17 deletions(-)

diff --git a/drivers/gpu/drm/drm_displayid.c b/drivers/gpu/drm/drm_displayid.c index 32da557b960f..dc649a9efaa2 100644 --- a/drivers/gpu/drm/drm_displayid.c +++ b/drivers/gpu/drm/drm_displayid.c @@ -37,7 +37,10 @@ static const u8 *drm_find_displayid_extension(const struct edid *edid, int *length, int *idx, int *ext_index) { - const u8 *displayid = drm_find_edid_extension(edid, DISPLAYID_EXT, ext_index); + const u8 *displayid = drm_find_edid_extension(edid, + DISPLAYID_EXT, + ext_index, + edid->extensions); const struct displayid_header *base; int ret;

diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index f8514783d089..bb050d171bb8 100644 --- a/drivers/gpu/drm/drm_edid.c +++ b/drivers/gpu/drm/drm_edid.c @@ -3364,23 +3364,23 @@ add_detailed_modes(struct drm_connector *connector, struct edid *edid, * Search EDID for CEA extension block. */ const u8 *drm_find_edid_extension(const struct edid *edid, - int ext_id, int *ext_index) + int ext_id, int *ext_index, int num_ext_blk) { const u8 *edid_ext = NULL; int i;

/* No EDID or EDID extensions */ - if (edid == NULL || edid->extensions == 0) + if (edid == NULL || edid->extensions == 0 || *ext_index >= num_ext_blk) return NULL;

- if (i >= edid->extensions) + if (i >= num_ext_blk) return NULL;

*ext_index = i + 1; @@ -3388,14 +3388,15 @@ const u8 *drm_find_edid_extension(const struct edid *edid, return edid_ext; }

-static const u8 *drm_find_cea_extension(const struct edid *edid, int *ext_index) +static const u8 *drm_find_cea_extension(const struct edid *edid, + int *ext_index, int num_ext_blk) { const struct displayid_block *block; struct displayid_iter iter; const u8 *cea;

/* Look for a CEA extension block from ext_index */ - cea = drm_find_edid_extension(edid, CEA_EXT, ext_index); + cea = drm_find_edid_extension(edid, CEA_EXT, ext_index, num_ext_blk); if (cea) return cea;

@@ -3679,7 +3680,7 @@ add_alternate_cea_modes(struct drm_connector *connector, struct edid *edid) int modes = 0, ext_index = 0;

/* Don't add CEA modes if the CEA extension block is missing */ - if (!drm_find_cea_extension(edid, &ext_index)) + if (!drm_find_cea_extension(edid, &ext_index, edid->extensions)) return 0;

/* @@ -4376,11 +4377,14 @@ add_cea_modes(struct drm_connector *connector, struct edid *edid) { const u8 *cea = NULL; const u8 *db, *hdmi = NULL, *video = NULL; - u8 dbl, hdmi_len, video_len = 0; + u8 dbl, hdmi_len, video_len = 0, num_ext_blk = edid->extensions; int modes = 0, j = 0;

+ if (num_ext_blk && drm_edid_is_hf_eeodb_blk_available(edid)) + num_ext_blk = drm_edid_read_hf_eeodb_blk_size(edid); + for (;;) { - cea = drm_find_cea_extension(edid, &j); + cea = drm_find_cea_extension(edid, &j, num_ext_blk);

if (!cea) break; @@ -4636,7 +4640,7 @@ static void drm_edid_to_eld(struct drm_connector *connector, struct edid *edid) if (!edid) return;

- cea = drm_find_cea_extension(edid, &ext_index); + cea = drm_find_cea_extension(edid, &ext_index, edid->extensions); if (!cea) { DRM_DEBUG_KMS("ELD: no CEA Extension found\n"); return; @@ -4724,7 +4728,7 @@ int drm_edid_to_sad(struct edid *edid, struct cea_sad **sads) int i, start, end, dbl; const u8 *cea;

- cea = drm_find_cea_extension(edid, &ext_index); + cea = drm_find_cea_extension(edid, &ext_index, edid->extensions); if (!cea) { DRM_DEBUG_KMS("SAD: no CEA Extension found\n"); return 0; @@ -4786,7 +4790,7 @@ int drm_edid_to_speaker_allocation(struct edid *edid, u8 **sadb) int i, start, end, dbl; const u8 *cea;

- cea = drm_find_cea_extension(edid, &ext_index); + cea = drm_find_cea_extension(edid, &ext_index, edid->extensions); if (!cea) { DRM_DEBUG_KMS("SAD: no CEA Extension found\n"); return 0; @@ -4881,7 +4885,7 @@ bool drm_detect_hdmi_monitor(struct edid *edid) int i, ext_index = 0; int start_offset, end_offset;

- edid_ext = drm_find_cea_extension(edid, &ext_index); + edid_ext = drm_find_cea_extension(edid, &ext_index, edid->extensions); if (!edid_ext) return false;

@@ -4920,7 +4924,7 @@ bool drm_detect_monitor_audio(struct edid *edid) bool has_audio = false; int start_offset, end_offset;

- edid_ext = drm_find_cea_extension(edid, &ext_index); + edid_ext = drm_find_cea_extension(edid, &ext_index, edid->extensions); if (!edid_ext) goto end;

@@ -5232,7 +5236,7 @@ static void drm_parse_cea_ext(struct drm_connector *connector, const u8 *edid_ext; int i, start, end, ext_index = 0;

- edid_ext = drm_find_cea_extension(edid, &ext_index); + edid_ext = drm_find_cea_extension(edid, &ext_index, edid->extensions); if (!edid_ext) return;

diff --git a/include/drm/drm_edid.h b/include/drm/drm_edid.h index ba2812432ead..4dbfbb531dfd 100644 --- a/include/drm/drm_edid.h +++ b/include/drm/drm_edid.h @@ -591,7 +591,7 @@ struct drm_display_mode * drm_display_mode_from_cea_vic(struct drm_device *dev, u8 video_code); const u8 *drm_find_edid_extension(const struct edid *edid, - int ext_id, int *ext_index); + int ext_id, int *ext_index, int num_ext_blk);

bool drm_edid_is_hf_eeodb_blk_available(const struct edid *edid); u8 drm_edid_read_hf_eeodb_blk_size(const struct edid *edid);

-- 2.31.1

1179

Age (days ago)

1179

Last active (days ago)

dri-devel@lists.freedesktop.org

9 comments

4 participants

tags (0)

participants (4)

Jani Nikula
Lee Shawn C
Lee, Shawn C
Ville Syrjälä