Re: [PATCH v11 01/12] staging: imx-drm: imx-hdmi: make checkpatch happy

2014-11-14 Thread Daniel Kurtz
On Fri, Nov 14, 2014 at 9:38 PM, Andy Yan wrote: > CHECK: Alignment should match open parenthesis > + if ((hdmi->vic == 10) || (hdmi->vic == 11) || > + (hdmi->vic == 12) || (hdmi->vic == 13) || > > CHECK: braces {} should be used on all arms of this statement > + if

[PATCH v11 01/12] staging: imx-drm: imx-hdmi: make checkpatch happy

2014-11-14 Thread Andy Yan
CHECK: Alignment should match open parenthesis + if ((hdmi->vic == 10) || (hdmi->vic == 11) || + (hdmi->vic == 12) || (hdmi->vic == 13) || CHECK: braces {} should be used on all arms of this statement + if (hdmi->hdmi_data.video_mode.mdvi) [...] + else { [...]

[PATCH v11 01/12] staging: imx-drm: imx-hdmi: make checkpatch happy

2014-11-14 Thread Andy Yan
CHECK: Alignment should match open parenthesis + if ((hdmi-vic == 10) || (hdmi-vic == 11) || + (hdmi-vic == 12) || (hdmi-vic == 13) || CHECK: braces {} should be used on all arms of this statement + if (hdmi-hdmi_data.video_mode.mdvi) [...] + else { [...]

Re: [PATCH v11 01/12] staging: imx-drm: imx-hdmi: make checkpatch happy

2014-11-14 Thread Daniel Kurtz
On Fri, Nov 14, 2014 at 9:38 PM, Andy Yan andy@rock-chips.com wrote: CHECK: Alignment should match open parenthesis + if ((hdmi-vic == 10) || (hdmi-vic == 11) || + (hdmi-vic == 12) || (hdmi-vic == 13) || CHECK: braces {} should be used on all arms of this statement +