This is an automatic generated email to let you know that the following patch were queued:
Subject: edid-decode: update Colorimetry DB: drop MD1-MD3 Author: Hans Verkuil <hverkuil-ci...@xs4all.nl> Date: Thu Apr 25 12:14:31 2024 +0200 CTA-861.7 specifies that the old MD1-MD3 bits are now reserved for future use with CTA-861. Signed-off-by: Hans Verkuil <hverkuil-ci...@xs4all.nl> parse-cta-block.cpp | 20 +++++++------------- 1 file changed, 7 insertions(+), 13 deletions(-) --- diff --git a/parse-cta-block.cpp b/parse-cta-block.cpp index 6bb7ea201a75..5f265bfd1b77 100644 --- a/parse-cta-block.cpp +++ b/parse-cta-block.cpp @@ -2255,9 +2255,9 @@ static const char *colorimetry1_map[] = { static const char *colorimetry2_map[] = { "Gamut Boundary Description Metadata Profile P0", - "Reserved Gamut Boundary Description Metadata Profile P1", - "Reserved Gamut Boundary Description Metadata Profile P2", - "Reserved Gamut Boundary Description Metadata Profile P3", + "Reserved F41", + "Reserved F42", + "Reserved F43", "Default", "sRGB", "ICtCp", @@ -2275,17 +2275,11 @@ void edid_state::cta_colorimetry_block(const unsigned char *x, unsigned length) for (i = 0; i < ARRAY_SIZE(colorimetry1_map); i++) if (x[0] & (1 << i)) printf(" %s\n", colorimetry1_map[i]); - // Bits MD0-MD3 are used to indicate which HDMI Gamut Boundary Description - // Metadata Profiles are supported. - // - // HDMI 1.3a in section 5.3.12 describes 4 possible profiles, but it marks - // P3 as 'defined in a future specification'. - // - // HDMI 1.4b, however, only specifies profile P0 in section 8.3.3. And I've - // only seen P0 in practice. My assumption is that profiles P1-P3 are never - // used, and so these bits should be 0. + // Bit MD0 is used to indicate if HDMI Gamut Boundary Description + // Metadata Profile P0 is supported. Bits F41-F43 are reserved + // and must be set to 0. if (x[1] & 0xe) - fail("Reserved bits MD1-MD3 must be 0.\n"); + fail("Reserved bits F41-F43 must be 0.\n"); for (i = 0; i < ARRAY_SIZE(colorimetry2_map); i++) if (x[1] & (1 << i)) printf(" %s\n", colorimetry2_map[i]);