Messages in this thread |  | | Date | Sun, 30 Sep 2012 10:45:16 +0200 | | From | Andrew Lunn <> | | Subject | Re: [PATCH] gpio: mvebu: Add missing breaks in mvebu_gpio_irq_set_type |
| |
On Sun, Sep 30, 2012 at 04:23:27PM +0800, Axel Lin wrote: > Signed-off-by: Axel Lin <axel.lin@ingics.com> > --- > drivers/gpio/gpio-mvebu.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/gpio/gpio-mvebu.c b/drivers/gpio/gpio-mvebu.c > index 902af43..7a874129 100644 > --- a/drivers/gpio/gpio-mvebu.c > +++ b/drivers/gpio/gpio-mvebu.c > @@ -381,11 +381,13 @@ static int mvebu_gpio_irq_set_type(struct irq_data *d, unsigned int type) > u = readl_relaxed(mvebu_gpioreg_in_pol(mvchip)); > u &= ~(1 << pin); > writel_relaxed(u, mvebu_gpioreg_in_pol(mvchip)); > + break; > case IRQ_TYPE_EDGE_FALLING: > case IRQ_TYPE_LEVEL_LOW: > u = readl_relaxed(mvebu_gpioreg_in_pol(mvchip)); > u |= 1 << pin; > writel_relaxed(u, mvebu_gpioreg_in_pol(mvchip)); > + break; > case IRQ_TYPE_EDGE_BOTH: { > u32 v; > > @@ -401,6 +403,7 @@ static int mvebu_gpio_irq_set_type(struct irq_data *d, unsigned int type) > else > u &= ~(1 << pin); /* rising */ > writel_relaxed(u, mvebu_gpioreg_in_pol(mvchip)); > + break; > } > } > return 0;
Hi Axel
Good catch, thanks.
Acked-by: Andrew Lunn <andrew@lunn.ch>
Andrew
|  |