Re: [PATCH 09/16] irqchip: irq-xtensa-mx: include header for missing prototype

[Marc Zyngier]

On Wed, 20 Sep 2023 06:21:32 +0100,
Randy Dunlap <rdunlap@xxxxxxxxxxxxx> wrote:
> 
> Add <linux/irqchips/xtensa-mx.h> to provide the function prototype to
> prevent a build warning:
> 
> drivers/irqchip/irq-xtensa-mx.c:166:12: warning: no previous prototype for 'xtensa_mx_init_legacy' [-Wmissing-prototypes]
>   166 | int __init xtensa_mx_init_legacy(struct device_node *interrupt_parent)
> 
> Signed-off-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
> Cc: Chris Zankel <chris@xxxxxxxxxx>
> Cc: Max Filippov <jcmvbkbc@xxxxxxxxx>
> Cc: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> Cc: Marc Zyngier <maz@xxxxxxxxxx>
> ---
>  drivers/irqchip/irq-xtensa-mx.c |    1 +
>  1 file changed, 1 insertion(+)
> 
> diff -- a/drivers/irqchip/irq-xtensa-mx.c b/drivers/irqchip/irq-xtensa-mx.c
> --- a/drivers/irqchip/irq-xtensa-mx.c
> +++ b/drivers/irqchip/irq-xtensa-mx.c
> @@ -12,6 +12,7 @@
>  #include <linux/irqdomain.h>
>  #include <linux/irq.h>
>  #include <linux/irqchip.h>
> +#include <linux/irqchip/xtensa-mx.h>
>  #include <linux/of.h>
>  
>  #include <asm/mxregs.h>
> 

Acked-by: Marc Zyngier <maz@xxxxxxxxxx>

It also begs the question of the continuing support for non-DT driven
systems.

Max, Chris: are we any closer to being DT-only on the xtensa front?

Thanks,

	M.

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