[coreboot-gerrit] Patch set updated for coreboot: edid: add hdmi_monitor_detected in data struct edid

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Fri Nov 6 16:15:08 CET 2015


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

-gerrit

commit 0e57d940cfe418adff8c3a0fb14ad7b640d571fd
Author: Yakir Yang <ykk at rock-chips.com>
Date:   Tue Oct 27 16:17:13 2015 +0800

    edid: add hdmi_monitor_detected in data struct edid
    
    HDMI driver need to know whether the monitor is DVI
    or HDMI interface, so this commit just introduce a
    new number 'hdmi_monitor_detected' to struct edid.
    
    There were four bits to indicate the monitor interfaces,
    it's better to take use of that. But those bits only
    existed in EDID 1.4 version, but didn't persented in
    the previous EDID version, so I decided to detect the
    hdmi cea block.
    
    BRANCH=none
    BUG=chrome-os-partner:43789
    TEST=When mickey connect with HDMI monitor, see 'hdmi_monitor_detected' is 'true'.
         When mickey connect with DVI monitor, see 'hdmi_monitor_detected' is 'false'.
    
    Change-Id: I1a4f1410e1cce1474ffae858db161a18578cac3a
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 409f041805d9fdff2d49faa1a3a262cf4dc609c2
    Original-Change-Id: Ife770898b0f2b4f58b8259711101a0cab4a5e4ac
    Original-Signed-off-by: Yakir Yang <ykk at rock-chips.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/309055
    Original-Tested-by: David Hendricks <dhendrix at chromium.org>
    Original-Reviewed-by: David Hendricks <dhendrix at chromium.org>
---
 src/include/edid.h | 2 ++
 src/lib/edid.c     | 2 ++
 2 files changed, 4 insertions(+)

diff --git a/src/include/edid.h b/src/include/edid.h
index 43e6883..d3cab17 100644
--- a/src/include/edid.h
+++ b/src/include/edid.h
@@ -87,6 +87,8 @@ struct edid {
 	u32 x_resolution;
 	u32 y_resolution;
 	u32 bytes_per_line;
+
+	int hdmi_monitor_detected;
 };
 
 /* Defined in src/lib/edid.c */
diff --git a/src/lib/edid.c b/src/lib/edid.c
index d005678..0f5643d 100644
--- a/src/lib/edid.c
+++ b/src/lib/edid.c
@@ -657,6 +657,8 @@ cea_hdmi_block(struct edid *out, unsigned char *x)
 {
 	int length = x[0] & 0x1f;
 
+	out->hdmi_monitor_detected = 1;
+
 	printk(BIOS_SPEW, " (HDMI)\n");
 	printk(BIOS_SPEW,
 	       "    Source physical address %d.%d.%d.%d\n",



More information about the coreboot-gerrit mailing list