Message ID | 20191025092233.25108-1-yuehaibing@huawei.com (mailing list archive) |
---|---|
State | Mainlined |
Commit | d5d3594db9f0de02c9073f1b7c7bcd7f805be02a |
Headers | show |
Series | [-next] pinctrl: armada-37xx: remove unneeded semicolon | expand |
YueHaibing <yuehaibing@huawei.com> writes: > Remove unneeded semicolon. > This is detected by coccinelle. > > Signed-off-by: YueHaibing <yuehaibing@huawei.com> I am against this patch, I don't see any value of this change. It will only make backporting future fix more difficult and will obscurate the git blame. I could see the interest of scripts/coccinelle/misc/semicolon.cocci when submitting new code, but not for existing code. Gregory > --- > drivers/pinctrl/mvebu/pinctrl-armada-37xx.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/pinctrl/mvebu/pinctrl-armada-37xx.c b/drivers/pinctrl/mvebu/pinctrl-armada-37xx.c > index 9df4277..fd32989 100644 > --- a/drivers/pinctrl/mvebu/pinctrl-armada-37xx.c > +++ b/drivers/pinctrl/mvebu/pinctrl-armada-37xx.c > @@ -733,7 +733,7 @@ static int armada_37xx_irqchip_register(struct platform_device *pdev, > ret = 0; > break; > } > - }; > + } > if (ret) { > dev_err(dev, "no gpio-controller child node\n"); > return ret; > @@ -800,7 +800,7 @@ static int armada_37xx_gpiochip_register(struct platform_device *pdev, > ret = 0; > break; > } > - }; > + } > if (ret) > return ret; > > -- > 2.7.4 > >
diff --git a/drivers/pinctrl/mvebu/pinctrl-armada-37xx.c b/drivers/pinctrl/mvebu/pinctrl-armada-37xx.c index 9df4277..fd32989 100644 --- a/drivers/pinctrl/mvebu/pinctrl-armada-37xx.c +++ b/drivers/pinctrl/mvebu/pinctrl-armada-37xx.c @@ -733,7 +733,7 @@ static int armada_37xx_irqchip_register(struct platform_device *pdev, ret = 0; break; } - }; + } if (ret) { dev_err(dev, "no gpio-controller child node\n"); return ret; @@ -800,7 +800,7 @@ static int armada_37xx_gpiochip_register(struct platform_device *pdev, ret = 0; break; } - }; + } if (ret) return ret;
Remove unneeded semicolon. This is detected by coccinelle. Signed-off-by: YueHaibing <yuehaibing@huawei.com> --- drivers/pinctrl/mvebu/pinctrl-armada-37xx.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)