<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
Hi ZhengYang,<br>
<br>
Thanks for your comments ;)<br>
<br>
<div class="moz-cite-prefix">On 01/29/2016 02:17 PM, 郑阳 wrote:<br>
</div>
<blockquote cite="mid:56AB03E6.6020107@rock-chips.com" type="cite">
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
在 2016年01月07日 18:03, Yakir Yang 写道:<br>
<blockquote
cite="mid:1452161024-27198-1-git-send-email-ykk@rock-chips.com"
type="cite">
<pre wrap="">diff --git a/drivers/gpu/drm/rockchip/inno_hdmi.c b/drivers/gpu/drm/rockchip/inno_hdmi.c
new file mode 100644
index 0000000..9327617
--- /dev/null
+++ b/drivers/gpu/drm/rockchip/inno_hdmi.c
+static int inno_hdmi_config_video_avi(struct inno_hdmi *hdmi)
+{
+
+ <font color="#ff0000">if (hdmi->hdmi_data.enc_out_format == HDMI_COLORSPACE_RGB)
+ avi_color_mode = AVI_COLOR_MODE_RGB</font>;
+ else if (hdmi->hdmi_data.enc_out_format == HDMI_COLORSPACE_YUV444)
+ avi_color_mode = AVI_COLOR_MODE_YCBCR444;
+ else if (hdmi->hdmi_data.enc_out_format == HDMI_COLORSPACE_YUV422)
+ avi_color_mode = AVI_COLOR_MODE_YCBCR422;
+ else
+ avi_color_mode = AVI_COLOR_MODE_RGB;
+</pre>
</blockquote>
<font color="#000099">Fist if is no need, its result is
same to "else".</font><br>
</blockquote>
Got it<br>
<blockquote cite="mid:56AB03E6.6020107@rock-chips.com" type="cite">
<blockquote
cite="mid:1452161024-27198-1-git-send-email-ykk@rock-chips.com"
type="cite">
<pre wrap="">+static int inno_hdmi_config_video_csc(struct inno_hdmi *hdmi)
+{
+ <font color="#ff0000">if (data->enc_out_format == data->enc_out_format)</font> {
+ if ((data->enc_in_format == HDMI_COLORSPACE_RGB) ||
+ (data->enc_in_format >= HDMI_COLORSPACE_YUV444)) {
+ value = v_SOF_DISABLE | v_COLOR_DEPTH_NOT_INDICATED(1);
+ hdmi_writeb(hdmi, HDMI_VIDEO_CONTRL3, value);
+
+ hdmi_modb(hdmi, HDMI_VIDEO_CONTRL,
+ m_VIDEO_AUTO_CSC | m_VIDEO_C0_C2_SWAP,
+ v_VIDEO_AUTO_CSC(AUTO_CSC_DISABLE) |
+ v_VIDEO_C0_C2_SWAP(C0_C2_CHANGE_DISABLE));
+ return 0;
+ }
+ }
</pre>
</blockquote>
<font color="#000099">It seems should be if
(data->enc_in_format == data->enc_out_format).<br>
</font><br>
</blockquote>
Ah, good catch, thanks<br>
<br>
- Yakir<br>
</body>
</html>