This is an automatic generated email to let you know that the following patch 
were queued:

Subject: edid-decode: downgrade the 'out-of-range' failure to a warning
Author:  Hans Verkuil <[email protected]>
Date:    Fri Aug 4 15:32:22 2023 +0200

It is not really very clear if it is legal or not to provide
timings that are out of range of the Monitor Range Limits Descriptor
in the base block.

So I decided to change this from a failure to a warning.

Signed-off-by: Hans Verkuil <[email protected]>

 parse-base-block.cpp | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

---

diff --git a/parse-base-block.cpp b/parse-base-block.cpp
index 61468d0f531d..c8d6cebb009f 100644
--- a/parse-base-block.cpp
+++ b/parse-base-block.cpp
@@ -1733,11 +1733,7 @@ void edid_state::check_base_block(const unsigned char *x)
                if (!out_of_range)
                        err += "    Could be due to a Monitor Range off-by-one 
rounding issue\n";
 
-               /*
-                * EDID 1.4 states (in an Errata) that explicitly defined
-                * timings supersede the monitor range definition.
-                */
-               msg(!out_of_range || base.edid_minor >= 4, "%s", err.c_str());
+               warn("%s", err.c_str());
        }
 
        if ((image_width && dtd_max_hsize_mm >= 10 + image_width / 10) ||

_______________________________________________
linuxtv-commits mailing list
[email protected]
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to