2020-10-28 20:18:56

by Tom Rix

Subject: [PATCH] clk: imx: remove unneeded semicolon

From: Tom Rix <[email protected]>

A semicolon is not needed after a switch statement.

Signed-off-by: Tom Rix <[email protected]>
---
drivers/clk/imx/clk-pll14xx.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clk/imx/clk-pll14xx.c b/drivers/clk/imx/clk-pll14xx.c
index aba36e4217d2..2b5ed86b9dbb 100644
--- a/drivers/clk/imx/clk-pll14xx.c
+++ b/drivers/clk/imx/clk-pll14xx.c
@@ -416,7 +416,7 @@ struct clk_hw *imx_dev_clk_hw_pll14xx(struct device *dev, const char *name,
__func__, name);
kfree(pll);
return ERR_PTR(-EINVAL);
- };
+ }

pll->base = base;
pll->hw.init = &init;
--
2.18.1

2020-10-28 20:37:20

by Abel Vesa

Subject: Re: [PATCH] clk: imx: remove unneeded semicolon

On 20-10-27 11:57:56, [email protected] wrote:
> From: Tom Rix <[email protected]>
>
> A semicolon is not needed after a switch statement.
>
> Signed-off-by: Tom Rix <[email protected]>

Reviewed-by: Abel Vesa <[email protected]>

> ---
> drivers/clk/imx/clk-pll14xx.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/clk/imx/clk-pll14xx.c b/drivers/clk/imx/clk-pll14xx.c
> index aba36e4217d2..2b5ed86b9dbb 100644
> --- a/drivers/clk/imx/clk-pll14xx.c
> +++ b/drivers/clk/imx/clk-pll14xx.c
> @@ -416,7 +416,7 @@ struct clk_hw *imx_dev_clk_hw_pll14xx(struct device *dev, const char *name,
> __func__, name);
> kfree(pll);
> return ERR_PTR(-EINVAL);
> - };
> + }
>
> pll->base = base;
> pll->hw.init = &init;
> --
> 2.18.1
>

2020-11-01 09:22:42

by Shawn Guo

Subject: Re: [PATCH] clk: imx: remove unneeded semicolon

On Tue, Oct 27, 2020 at 11:57:56AM -0700, [email protected] wrote:
> From: Tom Rix <[email protected]>
>
> A semicolon is not needed after a switch statement.
>
> Signed-off-by: Tom Rix <[email protected]>

Applied, thanks.