Thread (2 messages) 2 messages, 2 authors, 2020-10-19

Re: [PATCH] pinctrl: remove unneeded break

From: Andy Shevchenko <hidden>
Date: 2020-10-19 19:21:22
Also in: linux-gpio, linux-samsung-soc, lkml

On Mon, Oct 19, 2020 at 10:04 PM [off-list ref] wrote:
From: Tom Rix <trix@redhat.com>

A break is not needed if it is preceded by a return
When it's a single driver, use the proper prefix in the Subject line.

Also use proper English punctuation, i.e. period at the end of commit message.
quoted hunk ↗ jump to hunk
Signed-off-by: Tom Rix <trix@redhat.com>
---
 drivers/pinctrl/samsung/pinctrl-s3c24xx.c | 5 -----
 1 file changed, 5 deletions(-)
diff --git a/drivers/pinctrl/samsung/pinctrl-s3c24xx.c b/drivers/pinctrl/samsung/pinctrl-s3c24xx.c
index 5e24838a582f..2223ead5bd72 100644
--- a/drivers/pinctrl/samsung/pinctrl-s3c24xx.c
+++ b/drivers/pinctrl/samsung/pinctrl-s3c24xx.c
@@ -108,19 +108,14 @@ static int s3c24xx_eint_get_trigger(unsigned int type)
        switch (type) {
        case IRQ_TYPE_EDGE_RISING:
                return EINT_EDGE_RISING;
-               break;
        case IRQ_TYPE_EDGE_FALLING:
                return EINT_EDGE_FALLING;
-               break;
        case IRQ_TYPE_EDGE_BOTH:
                return EINT_EDGE_BOTH;
-               break;
        case IRQ_TYPE_LEVEL_HIGH:
                return EINT_LEVEL_HIGH;
-               break;
        case IRQ_TYPE_LEVEL_LOW:
                return EINT_LEVEL_LOW;
-               break;
        default:
                return -EINVAL;
        }
--
2.18.1

-- 
With Best Regards,
Andy Shevchenko

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help