[v2] edid-decode: downgrade missing display product name to warning
Commit Message
The E-EDID 1.4 spec says:
> The model name of the display product may be listed (optional
> but recommended) in the Display Product Name (ASCII) String
> Descriptor (tag #FCh).
So a missing Display Product Name descriptor shouldn't fail
conformance.
E-EDID 1.3 still requires the monitor name, so keep failing in
that case.
Signed-off-by: Simon Ser <contact@emersion.fr>
---
v2: keep this a failure for EDID 1.3
parse-base-block.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
base-commit: 582c935652b0303b87ddad4551e6f97f8bb883ac
@@ -1673,7 +1673,7 @@ void edid_state::parse_base_block(const unsigned char *x)
do_checksum("", x, EDID_PAGE_SIZE);
if (base.edid_minor >= 3) {
if (!base.has_name_descriptor)
- fail("Missing Display Product Name.\n");
+ msg(base.edid_minor >= 4, "Missing Display Product Name.\n");
if ((base.edid_minor == 3 || base.supports_continuous_freq) &&
!base.has_display_range_descriptor)
fail("Missing Display Range Limits Descriptor.\n");