[coreboot-gerrit] New patch to review for coreboot: parade/ps8640: Initialize edid_size

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Sat May 7 08:08:19 CEST 2016


Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/14698

-gerrit

commit 60564e4ff71d0c7363332f015450613c0be48416
Author: Patrick Georgi <pgeorgi at chromium.org>
Date:   Sat May 7 08:06:05 2016 +0200

    parade/ps8640: Initialize edid_size
    
    decode_edid either gets EDID_LENGTH bytes or (in the extended case),
    2*EDID_LENGTH.
    See that this is reflected in its size argument.
    
    Change-Id: If6c76358db4e9ee01c2bd2dbdd5948c61b7aa5bc
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
---
 src/drivers/parade/ps8640/ps8640.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/drivers/parade/ps8640/ps8640.c b/src/drivers/parade/ps8640/ps8640.c
index dea7be3..884b46a 100644
--- a/src/drivers/parade/ps8640/ps8640.c
+++ b/src/drivers/parade/ps8640/ps8640.c
@@ -25,7 +25,7 @@ int ps8640_get_edid(uint8_t bus, uint8_t chip, struct edid *out)
 {
 	int ret;
 	u8 edid[EDID_LENGTH * 2];
-	int edid_size;
+	int edid_size = EDID_LENGTH;
 
 	i2c_writeb(bus, chip + 2, PAGE2_I2C_BYPASS,
 		   EDID_I2C_ADDR | I2C_BYPASS_EN);



More information about the coreboot-gerrit mailing list