Julia Lawall <julia.law...@lip6.fr> writes: > From: Julia Lawall <julia.law...@lip6.fr> > > Replace a comma between expression statements by a semicolon. This changes > the semantics of the code, but given the current indentation appears to be > what is intended. > > A simplified version of the semantic patch that performs this > transformation is as follows: (http://coccinelle.lip6.fr/) > > // <smpl> > @r@ > expression e1,e2,e; > type T; > identifier i; > @@ > > e1 > -, > +; > e2; > // </smpl> > > Signed-off-by: Julia Lawall <julia.law...@lip6.fr> > > --- > Not tested. The first version did a different transformation to avoid > changing the semantics. This one leaves the layout as it is, but changes > the semantics.
This is the right one, the previous semantics were wrong due to a copy/paste error. Acked-by: Kevin Hilman <khil...@linaro.org> > drivers/gpio/gpio-omap.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpio/gpio-omap.c b/drivers/gpio/gpio-omap.c > index dfeb3a3..9f19ec9 100644 > --- a/drivers/gpio/gpio-omap.c > +++ b/drivers/gpio/gpio-omap.c > @@ -1030,7 +1030,7 @@ omap_mpuio_alloc_gc(struct gpio_bank *bank, unsigned > int irq_start, > ct->chip.irq_set_type = gpio_irq_type; > > if (bank->regs->wkup_en) > - ct->chip.irq_set_wake = gpio_wake_enable, > + ct->chip.irq_set_wake = gpio_wake_enable; > > ct->regs.mask = OMAP_MPUIO_GPIO_INT / bank->stride; > irq_setup_generic_chip(gc, IRQ_MSK(num), IRQ_GC_INIT_MASK_CACHE, > -- > 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/ -- 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/
