2023-05-05 12:02:21

by Linus Walleij

[permalink] [raw]
Subject: Re: [PATCH] gpio: zynq: fix zynqmp_gpio not an immutable chip warning

Hi Manikanta,

thanks for your patch!

On Wed, Apr 12, 2023 at 4:52 PM Manikanta Guntupalli
<[email protected]> wrote:

> Make the struct irq_chip const and flag it as IRQCHIP_IMMUTABLE to fix
> "gpio gpiochip1: (zynqmp_gpio): not an immutable chip" warning.
>
> Signed-off-by: Manikanta Guntupalli <[email protected]>

Don't you need to also add
gpiochip_disable_irq() in the .irq_mask() callback and
gpiochip_enable_irq() in the .irq_unmask() callback as we
do in most other conversions?

Example:
https://lore.kernel.org/linux-gpio/[email protected]/

Yours,
Linus Walleij