Re: [PATCH] gpio: tc3589x: Make irqchip immutable

[Marc Zyngier <maz@xxxxxxxxxx>]

On Mon, 26 Sep 2022 10:30:47 +0100,
Linus Walleij <linus.walleij@xxxxxxxxxx> wrote:
> 
> This turns the Toshiba tc3589x gpio irqchip immutable.
> 
> Cc: Marc Zyngier <maz@xxxxxxxxxx>
> Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx>
> ---
>  drivers/gpio/gpio-tc3589x.c | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpio/gpio-tc3589x.c b/drivers/gpio/gpio-tc3589x.c
> index 443fe975bf13..f9edb720710a 100644
> --- a/drivers/gpio/gpio-tc3589x.c
> +++ b/drivers/gpio/gpio-tc3589x.c
> @@ -244,13 +244,15 @@ static void tc3589x_gpio_irq_unmask(struct irq_data *d)
>  	tc3589x_gpio->regs[REG_DIRECT][regoffset] &= ~mask;
>  }
>  
> -static struct irq_chip tc3589x_gpio_irq_chip = {
> +static const struct irq_chip tc3589x_gpio_irq_chip = {
>  	.name			= "tc3589x-gpio",
>  	.irq_bus_lock		= tc3589x_gpio_irq_lock,
>  	.irq_bus_sync_unlock	= tc3589x_gpio_irq_sync_unlock,
>  	.irq_mask		= tc3589x_gpio_irq_mask,
>  	.irq_unmask		= tc3589x_gpio_irq_unmask,

Don't you need to update these two methods with the callbacks into the
gpiolib core code?

Thanks,

	M.

-- 
Without deviation from the norm, progress is not possible.