Re: [PATCH v4 2/3] arm64: dts: ti: k3-am62l: add initial infrastructure

[Bryan Brattlof]

On April  7, 2025 thus sayeth Andrew Davis:
> On 4/7/25 10:34 AM, Bryan Brattlof wrote:
> > From: Vignesh Raghavendra <vigneshr@xxxxxx>
> > 
> > Add the initial infrastructure needed for the AM62L. ALl of which can be
> > found in the Technical Reference Manual (TRM) located here:
> > 
> >      https://www.ti.com/lit/pdf/sprujb4
> > 
> > Signed-off-by: Vignesh Raghavendra <vigneshr@xxxxxx>
> > Signed-off-by: Bryan Brattlof <bb@xxxxxx>
> > ---
> > Changes in v3:
> >   - Added more nodes now that the SCMI interface is ready
> > 
> > Changes in v1:
> >   - switched to non-direct links to TRM updates are automatic
> >   - fixed white space indent issues with a few nodes
> >   - separated out device tree bindings
> > ---
> >   arch/arm64/boot/dts/ti/Makefile              |   3 +
> >   arch/arm64/boot/dts/ti/k3-am62l-main.dtsi    | 672 +++++++++++++++++++++++++++
> >   arch/arm64/boot/dts/ti/k3-am62l-thermal.dtsi |  19 +
> >   arch/arm64/boot/dts/ti/k3-am62l-wakeup.dtsi  | 144 ++++++
> >   arch/arm64/boot/dts/ti/k3-am62l.dtsi         | 121 +++++
> >   arch/arm64/boot/dts/ti/k3-am62l3.dtsi        |  67 +++
> >   arch/arm64/boot/dts/ti/k3-pinctrl.h          |   2 +
> >   7 files changed, 1028 insertions(+)
> > 
> > diff --git a/arch/arm64/boot/dts/ti/Makefile b/arch/arm64/boot/dts/ti/Makefile
> > +

...

> > +	fss: bus@fc00000 {
> > +		compatible = "simple-bus";
> > +		reg = <0x00 0x0fc00000 0x00 0x70000>;
> 
> The reg node does nothing for a "simple-bus", put the used
> register ranges in the ranges node, see how we did for some
> other devices:
> 
> https://lore.kernel.org/all/20240326205920.40147-2-afd@xxxxxx/
> 

As sweet. Thanks for the link. I'll get this updated

~Bryan