staging: imx-drm: Don't put an else right after a return
authorCatalina Mocanu <catalina.mocanu@gmail.com>
Wed, 24 Sep 2014 21:27:36 +0000 (14:27 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 29 Sep 2014 02:08:13 +0000 (22:08 -0400)
This fixes the following checkpatch.pl warning:
WARNING: else is not generally useful after a break or return.

Signed-off-by: Catalina Mocanu <catalina.mocanu@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/imx-drm/imx-hdmi.c
drivers/staging/imx-drm/imx-tve.c

index 18c9ccd460b7f6a6fef3a085abe48998f3eb68ed..aaec6b2cdf562f32c86e981555eef949c9901a2e 100644 (file)
@@ -323,8 +323,7 @@ static unsigned int hdmi_compute_cts(unsigned int freq, unsigned long pixel_clk,
        }
        if (ratio == 100)
                return cts;
-       else
-               return (cts * ratio) / 100;
+       return (cts * ratio) / 100;
 }
 
 static void hdmi_set_clk_regenerator(struct imx_hdmi *hdmi,
index fa8f827a47fc058eb1851c86b21989bc17e4446b..42c651be6c2068955df736c05520d1765cad8773 100644 (file)
@@ -432,8 +432,7 @@ static long clk_tve_di_round_rate(struct clk_hw *hw, unsigned long rate,
                return *prate / 4;
        else if (div >= 2)
                return *prate / 2;
-       else
-               return *prate;
+       return *prate;
 }
 
 static int clk_tve_di_set_rate(struct clk_hw *hw, unsigned long rate,