FROMLIST: drm: bridge: dw-hdmi: Enable CSC even for DVI
authorNeil Armstrong <narmstrong@baylibre.com>
Fri, 3 Mar 2017 17:20:00 +0000 (19:20 +0200)
committerZheng Yang <zhengyang@rock-chips.com>
Fri, 28 Apr 2017 08:20:37 +0000 (16:20 +0800)
If the input pixel format is not RGB, the CSC must be enabled in order to
provide valid pixel to DVI sinks.
This patch removes the hdmi only dependency on the CSC enabling.

Reviewed-by: Jose Abreu <joabreu@synopsys.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
Tested-by: Neil Armstrong <narmstrong@baylibre.com>
Signed-off-by: Archit Taneja <architt@codeaurora.org>
Link: http://patchwork.freedesktop.org/patch/msgid/20170303172007.26541-4-laurent.pinchart+renesas@ideasonboard.com
Change-Id: I7e9da663158790f7a84e126c6ed8b763a262bd1f
Signed-off-by: Zheng Yang <zhengyang@rock-chips.com>
(am from https://patchwork.kernel.org/patch/9603293/)

drivers/gpu/drm/bridge/dw-hdmi.c

index 3ed35b51793bbfacb91d643df3eaf0c4f629a79e..e36c4d487c14db147384411975788b31af608d01 100644 (file)
@@ -1705,8 +1705,8 @@ static void dw_hdmi_enable_video_path(struct dw_hdmi *hdmi)
                hdmi_writeb(hdmi, clkdis, HDMI_MC_CLKDIS);
        }
 
-       /* Enable color space conversion if needed (for HDMI sinks only). */
-       if (hdmi->sink_is_hdmi && is_color_space_conversion(hdmi))
+       /* Enable color space conversion if needed */
+       if (is_color_space_conversion(hdmi))
                hdmi_writeb(hdmi, HDMI_MC_FLOWCTRL_FEED_THROUGH_OFF_CSC_IN_PATH,
                            HDMI_MC_FLOWCTRL);
        else