Re: [RFC PATCH v2 16/30] drivers/irqchip: SH7751 IRL external encoder with enable gate.

Hi Sato-san,

On Wed, Sep 13, 2023 at 11:27 AM Yoshinori Sato
<[email protected]> wrote:
> SH7751 have 15 level external interrupt.
> It is typically connected to the CPU through a priority encoder
> that can suppress requests.
> This driver provides a way to control those hardware with irqchip.
>
> Signed-off-by: Yoshinori Sato <[email protected]>

Thanks for your patch!

> --- a/drivers/irqchip/Kconfig
> +++ b/drivers/irqchip/Kconfig
> @@ -688,4 +688,11 @@ config RENESAS_SH7751_INTC
>           Support for the Renesas SH7751 On-chip interrupt controller.
>           And external interrupt encoder for some targets.
>
> +config RENESAS_SH7751IRL_INTC
> +        bool "Renesas SH7751 based target IRL encoder support."
> +       depends on RENESAS_SH7751_INTC
> +       help

Inconsistent indentation

> +         Support for External Interrupt encoder
> +         on the some Renesas SH7751 based target.
> +
>  endmenu

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds