| Message ID | 20191115155752.2562-1-gregory.clement@bootlin.com (mailing list archive) |
|---|---|
| State | Mainlined |
| Commit | 04fb02757ae5188031eb71b2f6f189edb1caf5dc |
| Headers | show |
| Series | pinctrl: armada-37xx: Fix irq mask access in armada_37xx_irq_set_type() | expand |
On Fri, Nov 15, 2019 at 4:58 PM Gregory CLEMENT <gregory.clement@bootlin.com> wrote: > As explained in the following commit a9a1a4833613 ("pinctrl: > armada-37xx: Fix gpio interrupt setup") the armada_37xx_irq_set_type() > function can be called before the initialization of the mask field. > > That means that we can't use this field in this function and need to > workaround it using hwirq. > > Fixes: 30ac0d3b0702 ("pinctrl: armada-37xx: Add edge both type gpio irq support") > Cc: stable@vger.kernel.org > Reported-by: Russell King <rmk+kernel@armlinux.org.uk> > Signed-off-by: Gregory CLEMENT <gregory.clement@bootlin.com> Patch applied. I do not have time to send any more fixes pull requests at this point, so this will go into v5.5 in the merge window with everything else, and then to stable from there. Yours, Linus Walleij
diff --git a/drivers/pinctrl/mvebu/pinctrl-armada-37xx.c b/drivers/pinctrl/mvebu/pinctrl-armada-37xx.c index 9df4277a16be..aa9dcde0f069 100644 --- a/drivers/pinctrl/mvebu/pinctrl-armada-37xx.c +++ b/drivers/pinctrl/mvebu/pinctrl-armada-37xx.c @@ -595,10 +595,10 @@ static int armada_37xx_irq_set_type(struct irq_data *d, unsigned int type) regmap_read(info->regmap, in_reg, &in_val); /* Set initial polarity based on current input level. */ - if (in_val & d->mask) - val |= d->mask; /* falling */ + if (in_val & BIT(d->hwirq % GPIO_PER_REG)) + val |= BIT(d->hwirq % GPIO_PER_REG); /* falling */ else - val &= ~d->mask; /* rising */ + val &= ~(BIT(d->hwirq % GPIO_PER_REG)); /* rising */ break; } default:
As explained in the following commit a9a1a4833613 ("pinctrl: armada-37xx: Fix gpio interrupt setup") the armada_37xx_irq_set_type() function can be called before the initialization of the mask field. That means that we can't use this field in this function and need to workaround it using hwirq. Fixes: 30ac0d3b0702 ("pinctrl: armada-37xx: Add edge both type gpio irq support") Cc: stable@vger.kernel.org Reported-by: Russell King <rmk+kernel@armlinux.org.uk> Signed-off-by: Gregory CLEMENT <gregory.clement@bootlin.com> --- drivers/pinctrl/mvebu/pinctrl-armada-37xx.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)