rockchip: correctly set vop0 or vop1

The EDP_LCDC_SEL bit has to be set correctly to select vop0 or
vop1, but so far we have set it in both conditions, which is not
correct.

Can someone verify this is the correct way round?  vop1 -> set,
vop0 -> clear?

Signed-off-by: Patrick Wildt <patrick@blueri.se>
Reviewed-by: Kever Yang <kever.yang@rock-chips.com>

diff --git a/drivers/video/rockchip/rk_edp.c b/drivers/video/rockchip/rk_edp.c
index 92188be92757662541e1d100dfd4b97069d85b99..000bd48140884554e6d76d0e572ffd91adad6f94 100644 (file)
--- a/drivers/video/rockchip/rk_edp.c
+++ b/drivers/video/rockchip/rk_edp.c
@@ -1062,7 +1062,8 @@ static int rk_edp_probe(struct udevice *dev)
        rk_setreg(&priv->grf->soc_con12, 1 << 4);
 
        /* select epd signal from vop0 or vop1 */
-       rk_setreg(&priv->grf->soc_con6, (vop_id == 1) ? (1 << 5) : (1 << 5));
+       rk_clrsetreg(&priv->grf->soc_con6, (1 << 5),
+           (vop_id == 1) ? (1 << 5) : (0 << 5));
 
        rockchip_edp_wait_hpd(priv);