[PATCH] arch/arm/mach-at91/gpio.c: Remove unecessary semicolon

Subsystems: arm port, arm/microchip (at91) soc support, the rest

STALE5056d

13 messages, 5 authors, 2012-10-08 · open the first message on its own page

[PATCH] arch/arm/mach-at91/gpio.c: Remove unecessary semicolon

From: Peter Senna Tschudin <peter.senna@gmail.com>
Date: 2012-09-18 16:36:29

Found by http://coccinelle.lip6.fr/

Signed-off-by: Peter Senna Tschudin <peter.senna@gmail.com>
---
 arch/arm/mach-at91/gpio.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/mach-at91/gpio.c b/arch/arm/mach-at91/gpio.c
index be42cf0..ea31e96 100644
--- a/arch/arm/mach-at91/gpio.c
+++ b/arch/arm/mach-at91/gpio.c
@@ -658,7 +658,7 @@ static int at91_gpio_show(struct seq_file *s, void *unused)
 	seq_printf(s, "Pin\t");
 	for (bank = 0; bank < gpio_banks; bank++) {
 		seq_printf(s, "PIO%c\t\t", 'A' + bank);
-	};
+	}
 	seq_printf(s, "\n\n");
 
 	/* print pin status */
-- 
1.7.11.4

[PATCH] arch/arm/plat-mxc/ulpi.c: Remove unecessary semicolon

From: Peter Senna Tschudin <peter.senna@gmail.com>
Date: 2012-09-18 16:36:37

Found by http://coccinelle.lip6.fr/

Signed-off-by: Peter Senna Tschudin <peter.senna@gmail.com>
---
 arch/arm/plat-mxc/ulpi.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/plat-mxc/ulpi.c b/arch/arm/plat-mxc/ulpi.c
index d296342..a799afb 100644
--- a/arch/arm/plat-mxc/ulpi.c
+++ b/arch/arm/plat-mxc/ulpi.c
@@ -51,7 +51,7 @@ static int ulpi_poll(void __iomem *view, u32 bit)
 			return 0;
 
 		cpu_relax();
-	};
+	}
 
 	printk(KERN_WARNING "timeout polling for ULPI device\n");
 
-- 
1.7.11.4

[PATCH] arch/arm/mach-omap2: Remove unecessary semicolon

From: Peter Senna Tschudin <peter.senna@gmail.com>
Date: 2012-09-18 16:36:42

Found by http://coccinelle.lip6.fr/

Signed-off-by: Peter Senna Tschudin <peter.senna@gmail.com>
---
 arch/arm/mach-omap2/board-flash.c          | 2 +-
 arch/arm/mach-omap2/clkt_clksel.c          | 2 +-
 arch/arm/mach-omap2/mux.c                  | 2 +-
 arch/arm/mach-omap2/omap_hwmod_3xxx_data.c | 6 +++---
 arch/arm/mach-omap2/pm-debug.c             | 2 +-
 5 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/arch/arm/mach-omap2/board-flash.c b/arch/arm/mach-omap2/board-flash.c
index 53c39d2..d4c5bd5 100644
--- a/arch/arm/mach-omap2/board-flash.c
+++ b/arch/arm/mach-omap2/board-flash.c
@@ -218,7 +218,7 @@ void __init board_flash_init(struct flash_partitions partition_info[],
 			if (onenandcs > GPMC_CS_NUM)
 				onenandcs = cs;
 			break;
-		};
+		}
 		cs++;
 	}
 
diff --git a/arch/arm/mach-omap2/clkt_clksel.c b/arch/arm/mach-omap2/clkt_clksel.c
index 04d551b..fdb79c5 100644
--- a/arch/arm/mach-omap2/clkt_clksel.c
+++ b/arch/arm/mach-omap2/clkt_clksel.c
@@ -365,7 +365,7 @@ void omap2_init_clksel_parent(struct clk *clk)
 						 ((clk->parent) ?
 						  clk->parent->name : "NULL"));
 					clk_reparent(clk, clks->parent);
-				};
+				}
 				found = 1;
 			}
 		}
diff --git a/arch/arm/mach-omap2/mux.c b/arch/arm/mach-omap2/mux.c
index 9fe6829..701e17c 100644
--- a/arch/arm/mach-omap2/mux.c
+++ b/arch/arm/mach-omap2/mux.c
@@ -486,7 +486,7 @@ void omap_hwmod_mux(struct omap_hwmod_mux_info *hmux, u8 state)
 		default:
 			/* Nothing to be done */
 			break;
-		};
+		}
 
 		if (val >= 0) {
 			omap_mux_write(pad->partition, val,
diff --git a/arch/arm/mach-omap2/omap_hwmod_3xxx_data.c b/arch/arm/mach-omap2/omap_hwmod_3xxx_data.c
index ce7e606..5b91454 100644
--- a/arch/arm/mach-omap2/omap_hwmod_3xxx_data.c
+++ b/arch/arm/mach-omap2/omap_hwmod_3xxx_data.c
@@ -3459,7 +3459,7 @@ int __init omap3xxx_hwmod_init(void)
 	} else {
 		WARN(1, "OMAP3 hwmod family init: unknown chip type\n");
 		return -EINVAL;
-	};
+	}
 
 	r = omap_hwmod_register_links(h);
 	if (r < 0)
@@ -3476,7 +3476,7 @@ int __init omap3xxx_hwmod_init(void)
 		   rev == OMAP3430_REV_ES3_0 || rev == OMAP3430_REV_ES3_1 ||
 		   rev == OMAP3430_REV_ES3_1_2) {
 		h = omap3430es2plus_hwmod_ocp_ifs;
-	};
+	}
 
 	if (h) {
 		r = omap_hwmod_register_links(h);
@@ -3491,7 +3491,7 @@ int __init omap3xxx_hwmod_init(void)
 	} else if (rev == OMAP3430_REV_ES3_0 || rev == OMAP3430_REV_ES3_1 ||
 		   rev == OMAP3430_REV_ES3_1_2) {
 		h = omap3430_es3plus_hwmod_ocp_ifs;
-	};
+	}
 
 	if (h)
 		r = omap_hwmod_register_links(h);
diff --git a/arch/arm/mach-omap2/pm-debug.c b/arch/arm/mach-omap2/pm-debug.c
index 814bcd9..b5acc75 100644
--- a/arch/arm/mach-omap2/pm-debug.c
+++ b/arch/arm/mach-omap2/pm-debug.c
@@ -169,7 +169,7 @@ static int pm_dbg_open(struct inode *inode, struct file *file)
 	default:
 		return single_open(file, pm_dbg_show_timers,
 			&inode->i_private);
-	};
+	}
 }
 
 static const struct file_operations debug_fops = {
-- 
1.7.11.4

[PATCH] arch/arm/plat-omap/omap-pm-noop.c: Remove unecessary semicolon

From: Peter Senna Tschudin <peter.senna@gmail.com>
Date: 2012-09-18 16:36:47

Found by http://coccinelle.lip6.fr/

Signed-off-by: Peter Senna Tschudin <peter.senna@gmail.com>
---
 arch/arm/plat-omap/omap-pm-noop.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/arch/arm/plat-omap/omap-pm-noop.c b/arch/arm/plat-omap/omap-pm-noop.c
index 5a97b4d..9e01fac 100644
--- a/arch/arm/plat-omap/omap-pm-noop.c
+++ b/arch/arm/plat-omap/omap-pm-noop.c
@@ -38,7 +38,7 @@ int omap_pm_set_max_mpu_wakeup_lat(struct device *dev, long t)
 	if (!dev || t < -1) {
 		WARN(1, "OMAP PM: %s: invalid parameter(s)", __func__);
 		return -EINVAL;
-	};
+	}
 
 	if (t == -1)
 		pr_debug("OMAP PM: remove max MPU wakeup latency constraint: "
@@ -67,7 +67,7 @@ int omap_pm_set_min_bus_tput(struct device *dev, u8 agent_id, unsigned long r)
 	    agent_id != OCP_TARGET_AGENT)) {
 		WARN(1, "OMAP PM: %s: invalid parameter(s)", __func__);
 		return -EINVAL;
-	};
+	}
 
 	if (r == 0)
 		pr_debug("OMAP PM: remove min bus tput constraint: "
@@ -94,7 +94,7 @@ int omap_pm_set_max_dev_wakeup_lat(struct device *req_dev, struct device *dev,
 	if (!req_dev || !dev || t < -1) {
 		WARN(1, "OMAP PM: %s: invalid parameter(s)", __func__);
 		return -EINVAL;
-	};
+	}
 
 	if (t == -1)
 		pr_debug("OMAP PM: remove max device latency constraint: "
@@ -124,7 +124,7 @@ int omap_pm_set_max_sdma_lat(struct device *dev, long t)
 	if (!dev || t < -1) {
 		WARN(1, "OMAP PM: %s: invalid parameter(s)", __func__);
 		return -EINVAL;
-	};
+	}
 
 	if (t == -1)
 		pr_debug("OMAP PM: remove max DMA latency constraint: "
-- 
1.7.11.4

[PATCH] arch/arm/mach-omap1/devices.c: Remove unecessary semicolon

From: Peter Senna Tschudin <peter.senna@gmail.com>
Date: 2012-09-18 16:38:17

Found by http://coccinelle.lip6.fr/

Signed-off-by: Peter Senna Tschudin <peter.senna@gmail.com>
---
 arch/arm/mach-omap1/devices.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/mach-omap1/devices.c b/arch/arm/mach-omap1/devices.c
index fa1fa4d..688b23a 100644
--- a/arch/arm/mach-omap1/devices.c
+++ b/arch/arm/mach-omap1/devices.c
@@ -232,7 +232,7 @@ void __init omap1_init_mmc(struct omap_mmc_platform_data **mmc_data,
 
 		omap_mmc_add("mmci-omap", i, base, size, irq,
 				rx_req, tx_req, mmc_data[i]);
-	};
+	}
 }
 
 #endif
-- 
1.7.11.4

[PATCH] arch/arm/mach-davinci/board-dm646x-evm.c: Remove unecessary

From: Peter Senna Tschudin <peter.senna@gmail.com>
Date: 2012-09-18 16:38:38

Found by http://coccinelle.lip6.fr/

Signed-off-by: Peter Senna Tschudin <peter.senna@gmail.com>
---
 arch/arm/mach-davinci/board-dm646x-evm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/mach-davinci/board-dm646x-evm.c b/arch/arm/mach-davinci/board-dm646x-evm.c
index 958679a..1ccaace 100644
--- a/arch/arm/mach-davinci/board-dm646x-evm.c
+++ b/arch/arm/mach-davinci/board-dm646x-evm.c
@@ -193,7 +193,7 @@ static int evm_led_setup(struct i2c_client *client, int gpio,
 	while (ngpio--) {
 		leds->gpio = gpio++;
 		leds++;
-	};
+	}
 
 	evm_led_dev = platform_device_alloc("leds-gpio", 0);
 	platform_device_add_data(evm_led_dev, &evm_led_data,
-- 
1.7.11.4

Re: [PATCH] arch/arm/mach-davinci/board-dm646x-evm.c: Remove unecessary

From: Prabhakar Lad <prabhakar.csengg@gmail.com>
Date: 2012-09-18 16:49:43

Hi Peter,

Thanks for the patch.

On Tue, Sep 18, 2012 at 10:06 PM, Peter Senna Tschudin
[off-list ref] wrote:
Found by http://coccinelle.lip6.fr/
The commit message 'Remove unecessary' doesn't explain
what are you trying to do.

Thanks,
--Prabhakar
quoted hunk
Signed-off-by: Peter Senna Tschudin <peter.senna@gmail.com>
---
 arch/arm/mach-davinci/board-dm646x-evm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/mach-davinci/board-dm646x-evm.c b/arch/arm/mach-davinci/board-dm646x-evm.c
index 958679a..1ccaace 100644
--- a/arch/arm/mach-davinci/board-dm646x-evm.c
+++ b/arch/arm/mach-davinci/board-dm646x-evm.c
@@ -193,7 +193,7 @@ static int evm_led_setup(struct i2c_client *client, int gpio,
        while (ngpio--) {
                leds->gpio = gpio++;
                leds++;
-       };
+       }

        evm_led_dev = platform_device_alloc("leds-gpio", 0);
        platform_device_add_data(evm_led_dev, &evm_led_data,
--
1.7.11.4


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel at lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Re: [PATCH] arch/arm/mach-davinci/board-dm646x-evm.c: Remove unecessary

From: Peter Senna Tschudin <peter.senna@gmail.com>
Date: 2012-09-18 16:58:22

The commit message 'Remove unecessary' doesn't explain
what are you trying to do.
Sorry for that. I'll resend it now.

-- 
Peter

Re: [PATCH] arch/arm/plat-mxc/ulpi.c: Remove unecessary semicolon

From: Uwe Kleine-König <hidden>
Date: 2012-09-18 18:27:33

Hello,

$Subject ~= s/unecessary/unnecessary/

other than that the patch looks correct.

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-K?nig            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |

Re: [PATCH] arch/arm/mach-at91/gpio.c: Remove unecessary semicolon

From: Jean-Christophe PLAGNIOL-VILLARD <hidden>
Date: 2012-09-18 21:28:00

On 18:36 Tue 18 Sep     , Peter Senna Tschudin wrote:
Found by http://coccinelle.lip6.fr/

Signed-off-by: Peter Senna Tschudin <peter.senna@gmail.com>
---
this will have to wait the pinctrl patch series

Best Regards,
J.

Re: [PATCH] arch/arm/mach-omap1/devices.c: Remove unecessary semicolon

From: Tony Lindgren <tony@atomide.com>
Date: 2012-10-08 21:19:49

* Peter Senna Tschudin [off-list ref] [120918 09:37]:
quoted hunk
Found by http://coccinelle.lip6.fr/

Signed-off-by: Peter Senna Tschudin <peter.senna@gmail.com>
---
 arch/arm/mach-omap1/devices.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/mach-omap1/devices.c b/arch/arm/mach-omap1/devices.c
index fa1fa4d..688b23a 100644
--- a/arch/arm/mach-omap1/devices.c
+++ b/arch/arm/mach-omap1/devices.c
@@ -232,7 +232,7 @@ void __init omap1_init_mmc(struct omap_mmc_platform_data **mmc_data,
 
 		omap_mmc_add("mmci-omap", i, base, size, irq,
 				rx_req, tx_req, mmc_data[i]);
-	};
+	}
 }
 
 #endif
Thanks applying into omap fixes branch.

Regards,

Tony

Re: [PATCH] arch/arm/mach-omap2: Remove unecessary semicolon

From: Tony Lindgren <tony@atomide.com>
Date: 2012-10-08 21:19:59

* Peter Senna Tschudin [off-list ref] [120918 09:37]:
Found by http://coccinelle.lip6.fr/

Signed-off-by: Peter Senna Tschudin <peter.senna@gmail.com>
---
 arch/arm/mach-omap2/board-flash.c          | 2 +-
 arch/arm/mach-omap2/clkt_clksel.c          | 2 +-
 arch/arm/mach-omap2/mux.c                  | 2 +-
 arch/arm/mach-omap2/omap_hwmod_3xxx_data.c | 6 +++---
 arch/arm/mach-omap2/pm-debug.c             | 2 +-
 5 files changed, 7 insertions(+), 7 deletions(-)
Thanks applying into omap fixes branch.

Regards,

Tony 

Re: [PATCH] arch/arm/plat-omap/omap-pm-noop.c: Remove unecessary semicolon

From: Tony Lindgren <tony@atomide.com>
Date: 2012-10-08 21:20:18

* Peter Senna Tschudin [off-list ref] [120918 09:37]:
Found by http://coccinelle.lip6.fr/

Signed-off-by: Peter Senna Tschudin <peter.senna@gmail.com>
---
 arch/arm/plat-omap/omap-pm-noop.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)
Thanks applying into omap fixes branch.

Regards,

Tony
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help