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]);

Reply via email to