diff options
author | Hung-Te Lin <hungte@chromium.org> | 2014-04-03 18:35:58 +0800 |
---|---|---|
committer | Isaac Christensen <isaac.christensen@se-eng.com> | 2014-10-01 17:24:43 +0200 |
commit | 79445817aedacb5b1edbd6db2e3b6edab97eb5e0 (patch) | |
tree | fd0ccd9882c02943125f1aab334777338e0d535f /util/ifdfake | |
parent | fc0d244b39ec640ba8040b64e09e2fb5c931f95f (diff) | |
download | coreboot-79445817aedacb5b1edbd6db2e3b6edab97eb5e0.tar.xz |
edid: Fix extension parsing when EDID blob does not have any extensions.
When parsing "extensions", we should skip the first EDID (main) block and start
from offset 128 (EDID may have only main block, so an EDID without any
extension is fine) because the header format for main block and extensions are
different.
Without this we will see "Unknown extension block" on all EDIDs, and seeing an
error (1) return value for EDIDs without extension.
Also, after the first "unknown" error is fixed, we can now collect all return
values from parse_extension, and return an error when any of the extensions are
wrong (not just last one).
Change-Id: I0ee029ac8ec6800687cd7749e23989399e721109
Signed-off-by: Hung-Te Lin <hungte@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/193011
(cherry picked from commit fdf0cc2e9573c19b550fa2b5e4e06337b114f864)
Signed-off-by: Isaac Christensen <isaac.christensen@se-eng.com>
Reviewed-on: http://review.coreboot.org/6995
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Tested-by: build bot (Jenkins)
Reviewed-by: Ronald G. Minnich <rminnich@gmail.com>
Diffstat (limited to 'util/ifdfake')
0 files changed, 0 insertions, 0 deletions