On 3/9/22 11:22 AM, Clément Léger wrote:
When using interrupts property, a global interrupt controller needs to be added to avoid warnings when compiling device-tree:
arch/arm/dts/at91-sama5d2_xplained.dtb: Warning (interrupts_property): /ahb/apb/timer@f800c000: Missing interrupt-parent
Add AIC node as the sama5d2 global interrupt controller.
Signed-off-by: Clément Léger clement.leger@bootlin.com
arch/arm/dts/sama5d2.dtsi | 10 ++++++++++ 1 file changed, 10 insertions(+)
diff --git a/arch/arm/dts/sama5d2.dtsi b/arch/arm/dts/sama5d2.dtsi index 038cd73c03..1904f16eb6 100644 --- a/arch/arm/dts/sama5d2.dtsi +++ b/arch/arm/dts/sama5d2.dtsi @@ -3,6 +3,7 @@ / { model = "Atmel SAMA5D2 family SoC"; compatible = "atmel,sama5d2";
interrupt-parent = <&aic>; aliases { spi0 = &spi0;@@ -762,6 +763,15 @@ status = "disabled"; };
aic: interrupt-controller@fc020000 {compatible = "atmel,sama5d2-aic";reg = <0xfc020000 0x200>;#interrupt-cells = <3>;interrupt-controller;atmel,external-irqs = <49>;status = "disabled";};
Hello Clement,
Any reason to not use the exact node from Linux ?
aic: interrupt-controller@fc020000 { #interrupt-cells = <3>;
compatible = "atmel,sama5d2-aic";
interrupt-controller;
reg = <0xfc020000 0x200>;
atmel,external-irqs = <49>;
};
I would like to avoid any difference between Uboot and Linux, unless really required.
Don't worry if the node is enabled, it's no problem with that. It won't be probed due to several reasons: no driver and no need for it, since Uboot is based on lazy probing.
Eugen
i2c1: i2c@fc028000 { compatible = "atmel,sama5d2-i2c"; reg = <0xfc028000 0x100>;-- 2.34.1