diff options
| author | Jakub Czapiga <jacz@semihalf.com> | 2020-10-23 15:54:46 +0200 |
|---|---|---|
| committer | Paul Fagerburg <pfagerburg@chromium.org> | 2020-10-26 13:28:41 +0000 |
| commit | 43439f6d49b104e0e2310ed675b487736ca8fea7 (patch) | |
| tree | 979bac4300908890fb74eb373629e135f1752383 /src/lib/edid.c | |
| parent | 11298542cd8d6107e76d2905e55fc913be017ed1 (diff) | |
lib/edid: Add missing name descriptor presence flag
EDID parser internal flag c->has_name_descriptor
was never set. It was causing decode_edid() function
to return NON_CONFORMANT instead of CONFORMANT even when
EDID frame was correct.
Signed-off-by: Jakub Czapiga <jacz@semihalf.com>
Change-Id: Ifdc723b892a0885cfca08dab1a5ef961463da289
Reviewed-on: https://review.coreboot.org/c/coreboot/+/46694
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Julius Werner <jwerner@chromium.org>
Reviewed-by: Paul Fagerburg <pfagerburg@chromium.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Diffstat (limited to 'src/lib/edid.c')
| -rw-r--r-- | src/lib/edid.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/lib/edid.c b/src/lib/edid.c index f20d23959e..cd7a47ac78 100644 --- a/src/lib/edid.c +++ b/src/lib/edid.c @@ -261,6 +261,7 @@ detailed_block(struct edid *result_edid, unsigned char *x, int in_extension, extract_string(x + 5, &c->has_valid_string_termination, EDID_ASCII_STRING_LENGTH)); + c->has_name_descriptor = 1; return 1; case 0xFD: { |