Continue is not needed at the bottom of a loop. The Coccinelle semantic patch implementing this change is:
@@ @@ for (...;...;...) { ... if (...) { ... - continue; } } Signed-off-by: Himangi Saraogi <himangi...@gmail.com> Acked-by: Julia Lawall <julia.law...@lip6.fr> --- arch/arm/mach-integrator/impd1.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/arch/arm/mach-integrator/impd1.c b/arch/arm/mach-integrator/impd1.c index 3ce8807..c4eb8c9 100644 --- a/arch/arm/mach-integrator/impd1.c +++ b/arch/arm/mach-integrator/impd1.c @@ -376,10 +376,8 @@ static int __init_refok impd1_probe(struct lm_device *dev) irq1, irq2, idev->platform_data, idev->id, &dev->resource); - if (IS_ERR(d)) { + if (IS_ERR(d)) dev_err(&dev->dev, "unable to register device: %ld\n", PTR_ERR(d)); - continue; - } } return 0; -- 1.9.1 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/