Re: [PATCH 1/7] ARM: dts: at91: sam9x60: Add AIC node

[Eugen Hristev]

On 1/27/25 12:09, Manikandan Muralidharan wrote:
> Add Advanced Interrupt Controller node and define it as interrupt
> parent in sam9x60 SoC DT.
> 
> Signed-off-by: Manikandan Muralidharan <manikanda...@microchip.com>
> ---
>  arch/arm/dts/sam9x60.dtsi | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/arch/arm/dts/sam9x60.dtsi b/arch/arm/dts/sam9x60.dtsi
> index 3b684fc63d5..fdc3ad0fdcb 100644
> --- a/arch/arm/dts/sam9x60.dtsi
> +++ b/arch/arm/dts/sam9x60.dtsi
> @@ -17,6 +17,7 @@
>  /{
>       model = "Microchip SAM9X60 SoC";
>       compatible = "microchip,sam9x60";
> +     interrupt-parent = <&aic>;
>  
>       aliases {
>               serial0 = &dbgu;
> @@ -199,6 +200,14 @@
>                               reg = <0xffffea00 0x100>;
>                       };
>  
> +                     aic: interrupt-controller@fffff100 {
> +                             compatible = "microchip,sam9x60-aic";
> +                             reg = <0xfffff100 0x100>;

Reg property is placed below interrupt-controller in Linux, can you make
it identical please ?

> +                             #interrupt-cells = <3>;
> +                             interrupt-controller;
> +                             atmel,external-irqs = <31>;
> +                     };
> +
>                       dbgu: serial@fffff200 {
>                               compatible = "atmel,at91sam9260-dbgu", 
> "atmel,at91sam9260-usart";
>                               reg = <0xfffff200 0x200>;