From: Tom Rix <t...@redhat.com>

A break is not needed if it is preceded by a return.

Signed-off-by: Tom Rix <t...@redhat.com>
---

v2: Improve commit log

---
 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

Reply via email to