[coreboot-gerrit] New patch to review for coreboot: 90f3a2e edid: initialize has_valid_detailed_blocks as 1

Marc Jones (marc.jones@se-eng.com) gerrit at coreboot.org
Wed Dec 17 00:06:55 CET 2014


Marc Jones (marc.jones at se-eng.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/7835

-gerrit

commit 90f3a2e3f98c13436ffc3342b0a162dc1a741469
Author: Vince Hsu <vinceh at nvidia.com>
Date:   Mon Apr 21 17:07:57 2014 +0800

    edid: initialize has_valid_detailed_blocks as 1
    
    In last clean-up commit, the detailed_blocks parsing has been merged to one
    for-loop and combining return values in each iteration instead of assignment.
    As a result, has_valid_detailed_blocks should now be initialized as 1.
    
    BRANCH=none
    BUG=none
    TEST=Tested AUO 1080p and InnoLux 720p panels on nyan_big
    
    Original-Change-Id: Ie4b6e25de63c0e216ae5de9bde20eed1fe3e59a6
    Original-Signed-off-by: Vince Hsu <vinceh at nvidia.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/195803
    Original-Reviewed-by: Hung-Te Lin <hungte at chromium.org>
    Original-Commit-Queue: Hung-Te Lin <hungte at chromium.org>
    Original-Tested-by: Hung-Te Lin <hungte at chromium.org>
    (cherry picked from commit 21ac533d17c892c39532c263cc6ec15e4507ed3e)
    Signed-off-by: Marc Jones <marc.jones at se-eng.com>
    
    Change-Id: I23111efb902c7e5994a4dbdfe77242c13ef5a70e
---
 src/lib/edid.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/lib/edid.c b/src/lib/edid.c
index 89755e0..07850ee 100644
--- a/src/lib/edid.c
+++ b/src/lib/edid.c
@@ -968,6 +968,7 @@ int decode_edid(unsigned char *edid, int size, struct edid *out)
 	    .has_valid_cvt = 1,
 	    .has_valid_dummy_block = 1,
 	    .has_valid_descriptor_ordering = 1,
+	    .has_valid_detailed_blocks = 1,
 	    .has_valid_descriptor_pad = 1,
 	    .has_valid_range_descriptor = 1,
 	    .has_valid_max_dotclock = 1,



More information about the coreboot-gerrit mailing list