leandrof/linux
1
0
Fork 0
forked from Minki/linux
Code Issues Pull Requests Packages Projects Releases Wiki Activity

imx-drm: imx-hdmi: Remove parentheses from returns

Browse Source 
Fix the following checkpatch warnings:

ERROR: return is not a function, parentheses are not required
#462: FILE: drivers/staging/imx-drm/imx-hdmi.c:462:
+       return (hdmi->hdmi_data.enc_in_format !=

ERROR: return is not a function, parentheses are not required
#468: FILE: drivers/staging/imx-drm/imx-hdmi.c:468:
+       return ((hdmi->hdmi_data.enc_out_format == YCBCR422_8BITS) &&

ERROR: return is not a function, parentheses are not required
#475: FILE: drivers/staging/imx-drm/imx-hdmi.c:475:
+       return ((hdmi->hdmi_data.enc_in_format == YCBCR422_8BITS) &&

While at it, broke the return code from is_color_space_decimation() and
is_color_space_interpolation() for better readability

Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Fabio Estevam 2014-02-06 10:12:03 -02:00 committed by Greg Kroah-Hartman
parent 2ec1cb7605
commit ba92b22515
1 changed files with 13 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
drivers/staging/imx-drm/imx-hdmi.c
View File

@ -463,22 +463,27 @@ static void hdmi_video_sample(struct imx_hdmi *hdmi)
static int is_color_space_conversion(struct imx_hdmi *hdmi) static int is_color_space_conversion(struct imx_hdmi *hdmi)
{ {
return (hdmi->hdmi_data.enc_in_format != return hdmi->hdmi_data.enc_in_format != hdmi->hdmi_data.enc_out_format;
hdmi->hdmi_data.enc_out_format);
} }
static int is_color_space_decimation(struct imx_hdmi *hdmi) static int is_color_space_decimation(struct imx_hdmi *hdmi)
{ {
return ((hdmi->hdmi_data.enc_out_format == YCBCR422_8BITS) && if (hdmi->hdmi_data.enc_out_format != YCBCR422_8BITS)
(hdmi->hdmi_data.enc_in_format == RGB || return 0;
hdmi->hdmi_data.enc_in_format == YCBCR444)); if (hdmi->hdmi_data.enc_in_format == RGB ||
hdmi->hdmi_data.enc_in_format == YCBCR444)
return 1;
return 0;
} }
static int is_color_space_interpolation(struct imx_hdmi *hdmi) static int is_color_space_interpolation(struct imx_hdmi *hdmi)
{ {
return ((hdmi->hdmi_data.enc_in_format == YCBCR422_8BITS) && if (hdmi->hdmi_data.enc_in_format != YCBCR422_8BITS)
(hdmi->hdmi_data.enc_out_format == RGB || return 0;
hdmi->hdmi_data.enc_out_format == YCBCR444)); if (hdmi->hdmi_data.enc_out_format == RGB ||
hdmi->hdmi_data.enc_out_format == YCBCR444)
return 1;
return 0;
} }
static void imx_hdmi_update_csc_coeffs(struct imx_hdmi *hdmi) static void imx_hdmi_update_csc_coeffs(struct imx_hdmi *hdmi)