[PATCH 0/6] pinctrl: Remove unneded semicolons in drivers

[PATCH 0/6] pinctrl: Remove unneded semicolons in drivers

[Javier Martinez Canillas]

Hello Linus,

This series contains trivial patches that removes unneeded
semicolons in pinctrl drivers. These are clearly typo errors
and are just creating null statements.

Best regards,
Javier


Javier Martinez Canillas (6):
  pinctrl: tz1090: Remove unneded semicolons
  pinctrl: tz1090-pdc: Remove unneded semicolons
  pinctrl: mxs: Remove unneded semicolon
  pinctrl: sunxi: Remove unneeded semicolon
  pinctrl: mediatek: Remove unneded semicolon
  pinctrl: samsung: Remove unneded semicolon

 drivers/pinctrl/freescale/pinctrl-mxs.c       | 2 +-
 drivers/pinctrl/mediatek/pinctrl-mtk-common.c | 2 +-
 drivers/pinctrl/pinctrl-tz1090-pdc.c          | 4 ++--
 drivers/pinctrl/pinctrl-tz1090.c              | 4 ++--
 drivers/pinctrl/samsung/pinctrl-exynos5440.c  | 2 +-
 drivers/pinctrl/sunxi/pinctrl-sunxi.c         | 2 +-
 6 files changed, 8 insertions(+), 8 deletions(-)

-- 
2.4.3

[PATCH 4/6] pinctrl: sunxi: Remove unneeded semicolon

[Javier Martinez Canillas]

It's not needed an is just creating a null statement, so remove it.

Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
---

 drivers/pinctrl/sunxi/pinctrl-sunxi.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/pinctrl/sunxi/pinctrl-sunxi.c b/drivers/pinctrl/sunxi/pinctrl-sunxi.c
index fb4669c0ce0e..d3428709b0a6 100644
--- a/drivers/pinctrl/sunxi/pinctrl-sunxi.c
+++ b/drivers/pinctrl/sunxi/pinctrl-sunxi.c
@@ -1031,7 +1031,7 @@ int sunxi_pinctrl_init(struct platform_device *pdev,
 		irq_set_chip_and_handler(irqno, &sunxi_pinctrl_edge_irq_chip,
 					 handle_edge_irq);
 		irq_set_chip_data(irqno, pctl);
-	};
+	}
 
 	for (i = 0; i < pctl->desc->irq_banks; i++) {
 		/* Mask and clear all IRQs before registering a handler */
-- 
2.4.3

[PATCH 4/6] pinctrl: sunxi: Remove unneeded semicolon

[Maxime Ripard]

On Wed, Sep 16, 2015 at 10:28:29AM +0200, Javier Martinez Canillas wrote:
> It's not needed an is just creating a null statement, so remove it.
> 
> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>

Acked-by: Maxime Ripard <maxime.ripard@free-electrons.com>

Thanks!
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20150916/7a091429/attachment.sig>

[PATCH 4/6] pinctrl: sunxi: Remove unneeded semicolon

[Linus Walleij]

On Wed, Sep 16, 2015 at 1:28 AM, Javier Martinez Canillas
<javier@osg.samsung.com> wrote:

> It's not needed an is just creating a null statement, so remove it.
>
> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>

Patch applied with Maxime's ACK.

Yours,
Linus Walleij

[PATCH 5/6] pinctrl: mediatek: Remove unneded semicolon

[Javier Martinez Canillas]

It's not needed an is just creating a null statement, so remove it.

Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
---

 drivers/pinctrl/mediatek/pinctrl-mtk-common.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/pinctrl/mediatek/pinctrl-mtk-common.c b/drivers/pinctrl/mediatek/pinctrl-mtk-common.c
index 7726c6caaf83..652ec125adb3 100644
--- a/drivers/pinctrl/mediatek/pinctrl-mtk-common.c
+++ b/drivers/pinctrl/mediatek/pinctrl-mtk-common.c
@@ -1436,7 +1436,7 @@ int mtk_pctrl_init(struct platform_device *pdev,
 		irq_set_chip_and_handler(virq, &mtk_pinctrl_irq_chip,
 			handle_level_irq);
 		irq_set_chip_data(virq, pctl);
-	};
+	}
 
 	irq_set_chained_handler_and_data(irq, mtk_eint_irq_handler, pctl);
 	return 0;
-- 
2.4.3

[PATCH 5/6] pinctrl: mediatek: Remove unneded semicolon

[Hongzhou Yang]

On Wed, 2015-09-16 at 10:28 +0200, Javier Martinez Canillas wrote:
> It's not needed an is just creating a null statement, so remove it.
> 
> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
> ---
> 
>  drivers/pinctrl/mediatek/pinctrl-mtk-common.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/pinctrl/mediatek/pinctrl-mtk-common.c b/drivers/pinctrl/mediatek/pinctrl-mtk-common.c
> index 7726c6caaf83..652ec125adb3 100644
> --- a/drivers/pinctrl/mediatek/pinctrl-mtk-common.c
> +++ b/drivers/pinctrl/mediatek/pinctrl-mtk-common.c
> @@ -1436,7 +1436,7 @@ int mtk_pctrl_init(struct platform_device *pdev,
>  		irq_set_chip_and_handler(virq, &mtk_pinctrl_irq_chip,
>  			handle_level_irq);
>  		irq_set_chip_data(virq, pctl);
> -	};
> +	}
>  
>  	irq_set_chained_handler_and_data(irq, mtk_eint_irq_handler, pctl);
>  	return 0;


Acked-by: Hongzhou Yang <hongzhou.yang@mediatek.com>

Thanks
Hongzhou

[PATCH 5/6] pinctrl: mediatek: Remove unneded semicolon

[Linus Walleij]

On Wed, Sep 16, 2015 at 1:28 AM, Javier Martinez Canillas
<javier@osg.samsung.com> wrote:

> It's not needed an is just creating a null statement, so remove it.
>
> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>

Patch applied with Hongzhou's ACK.

Yours,
Linus Walleij

[PATCH 6/6] pinctrl: samsung: Remove unneded semicolon

[Javier Martinez Canillas]

It's not needed an is just creating a null statement, so remove it.

Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>

---

 drivers/pinctrl/samsung/pinctrl-exynos5440.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/pinctrl/samsung/pinctrl-exynos5440.c b/drivers/pinctrl/samsung/pinctrl-exynos5440.c
index 9ce0b8619d4c..82dc109f7ed4 100644
--- a/drivers/pinctrl/samsung/pinctrl-exynos5440.c
+++ b/drivers/pinctrl/samsung/pinctrl-exynos5440.c
@@ -284,7 +284,7 @@ static void exynos5440_dt_free_map(struct pinctrl_dev *pctldev,
 			if (!idx)
 				kfree(map[idx].data.configs.group_or_pin);
 		}
-	};
+	}
 
 	kfree(map);
 }
-- 
2.4.3

[PATCH 6/6] pinctrl: samsung: Remove unneded semicolon

[Krzysztof Kozlowski]

On 16.09.2015 17:28, Javier Martinez Canillas wrote:
> It's not needed an is just creating a null statement, so remove it.
> 
> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
> 
> ---
> 
>  drivers/pinctrl/samsung/pinctrl-exynos5440.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

Quite obvious and pointed out by coccicheck, so:

Reviewed-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>

Best regards,
Krzysztof

[PATCH 6/6] pinctrl: samsung: Remove unneded semicolon

[Linus Walleij]

On Wed, Sep 16, 2015 at 1:28 AM, Javier Martinez Canillas
<javier@osg.samsung.com> wrote:

> It's not needed an is just creating a null statement, so remove it.
>
> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>

Patch applied with Krzysztof's Review-tag.

Yours,
Linus Walleij