New patch to review for coreboot: rockchip/rk3288: hdmi: configure display output mode with EDID information

[Patrick Georgi]

Patrick Georgi ([e-mail]) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/12346

-gerrit

commit 9813b23ef2169136b8e9fc1b5e31cc4ee8e6dd87
Author: Yakir Yang 
Date:   Tue Oct 27 16:40:08 2015 +0800

rockchip/rk3288: hdmi: configure display output mode with EDID information

'edid->hdmi_monitor_detected' would indicate whether the monitor
    interface is HDMI or DVI.

BRANCH=none
    BUG=chrome-os-partner:43789
    TEST=Previously, my LG monitor couldn't show dev screen. But now I can see
         dev screen have been posted normally.

Change-Id: Id71f051b2cd792712e52bee7a763db383c1962a8
    Signed-off-by: Patrick Georgi 
    Original-Commit-Id: 88101589a22d06f0bc25e0750b2862cf66b55391
    Original-Change-Id: I157861d327926b834e1e8606b0b676f413491c70
    Original-Signed-off-by: Yakir Yang 
    Original-Reviewed-on: https://chromium-review.googlesource.com/309056
    Original-Tested-by: David Hendricks 
    Original-Reviewed-by: David Hendricks 
---
 src/soc/rockchip/rk3288/hdmi.c | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/src/soc/rockchip/rk3288/hdmi.c b/src/soc/rockchip/rk3288/hdmi.c
index 104d80f..805b98a 100644
--- a/src/soc/rockchip/rk3288/hdmi.c
+++ b/src/soc/rockchip/rk3288/hdmi.c
@@ -488,7 +488,6 @@ static int hdmi_phy_init(u32 mpixelclock)
 static void hdmi_av_composer(const struct edid *edid)
 {
    u8 mdataenablepolarity = 1;
-	u8 mdvi = 0;
    u8 inv_val;

/* set up hdmi_fc_invidconf */
@@ -506,9 +505,9 @@ static void hdmi_av_composer(const struct edid *edid)
    	   HDMI_FC_INVIDCONF_DE_IN_POLARITY_ACTIVE_HIGH :
    	   HDMI_FC_INVIDCONF_DE_IN_POLARITY_ACTIVE_LOW);

-	inv_val |= (mdvi ?
-		   HDMI_FC_INVIDCONF_DVI_MODEZ_DVI_MODE :
-		   HDMI_FC_INVIDCONF_DVI_MODEZ_HDMI_MODE);
+	inv_val |= (edid->hdmi_monitor_detected ?
+		   HDMI_FC_INVIDCONF_DVI_MODEZ_HDMI_MODE :
+		   HDMI_FC_INVIDCONF_DVI_MODEZ_DVI_MODE);

inv_val |= HDMI_FC_INVIDCONF_R_V_BLANK_IN_OSC_ACTIVE_LOW;