On 17/06/2024 12:22, Kamlesh Gurudasani wrote: > Add the node for sa3ul crypto accelerator. > > Signed-off-by: Jayesh Choudhary <j-choudhary@xxxxxx> > Signed-off-by: Kamlesh Gurudasani <kamlesh@xxxxxx> > --- > arch/arm64/boot/dts/ti/k3-am62a-main.dtsi | 12 ++++++++++++ > 1 file changed, 12 insertions(+) > > diff --git a/arch/arm64/boot/dts/ti/k3-am62a-main.dtsi b/arch/arm64/boot/dts/ti/k3-am62a-main.dtsi > index ce4a2f105630..298d0c91fc0a 100644 > --- a/arch/arm64/boot/dts/ti/k3-am62a-main.dtsi > +++ b/arch/arm64/boot/dts/ti/k3-am62a-main.dtsi > @@ -216,6 +216,18 @@ k3_reset: reset-controller { > }; > }; > > + crypto: crypto@40900000 { > + compatible = "ti,am62-sa3ul"; > + reg = <0x00 0x40900000 0x00 0x1200>; > + #address-cells = <2>; > + #size-cells = <2>; > + ranges = <0x00 0x40900000 0x00 0x40900000 0x00 0x30000>; Why do you need ranges? No children here. Anyway, keep the order as in DTS coding style. Best regards, Krzysztof