2020-11-04 15:49:13

by William Breathitt Gray

[permalink] [raw]
Subject: Re: [PATCH 1/3] gpio: pcie-idio-24: Fix irq mask when masking

On Wed, Nov 04, 2020 at 04:24:53PM +0100, Arnaud de Turckheim wrote:
> Fix the bitwise operation to remove only the corresponding bit from the
> mask.
>
> Fixes: 585562046628 ("gpio: Add GPIO support for the ACCES PCIe-IDIO-24 family")
> Signed-off-by: Arnaud de Turckheim <[email protected]>

Reviewed-by: William Breathitt Gray <[email protected]>

> ---
> drivers/gpio/gpio-pcie-idio-24.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpio/gpio-pcie-idio-24.c b/drivers/gpio/gpio-pcie-idio-24.c
> index a68941d19ac6..5ea517416366 100644
> --- a/drivers/gpio/gpio-pcie-idio-24.c
> +++ b/drivers/gpio/gpio-pcie-idio-24.c
> @@ -339,7 +339,7 @@ static void idio_24_irq_mask(struct irq_data *data)
>
> raw_spin_lock_irqsave(&idio24gpio->lock, flags);
>
> - idio24gpio->irq_mask &= BIT(bit_offset);
> + idio24gpio->irq_mask &= ~BIT(bit_offset);
> new_irq_mask = idio24gpio->irq_mask >> bank_offset;
>
> if (!new_irq_mask) {
> --
> 2.25.1
>


Attachments:
(No filename) (1.00 kB)
signature.asc (849.00 B)
Download all attachments