Re: [PATCH 3/3] arm64: dts: amlogic: t7: Add clock controller nodes

[Jian Hu]

On 3/11/2026 3:53 PM, Ronald Claveau wrote:
> [ EXTERNAL EMAIL ]
>
> On 3/5/26 8:43 AM, Jian Hu wrote:
> > Add the required clock controller nodes for Amlogic T7 SoC family:
> > - SCMI clock controller
> > - PLL clock controller
> > - Peripheral clock controller
> >
> > Signed-off-by: Jian Hu <jian.hu@xxxxxxxxxxx>
> > ---
> >   arch/arm64/boot/dts/amlogic/amlogic-t7.dtsi | 125 ++++++++++++++++++++
> >   1 file changed, 125 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/amlogic/amlogic-t7.dtsi b/arch/arm64/boot/dts/amlogic/amlogic-t7.dtsi
> > index 6510068bcff9..6ea1b583b13d 100644
> > --- a/arch/arm64/boot/dts/amlogic/amlogic-t7.dtsi
> > +++ b/arch/arm64/boot/dts/amlogic/amlogic-t7.dtsi
> > @@ -6,6 +6,9 @@
> >   #include <dt-bindings/interrupt-controller/arm-gic.h>
> >   #include <dt-bindings/power/amlogic,t7-pwrc.h>
> >   #include "amlogic-t7-reset.h"
> > +#include <dt-bindings/clock/amlogic,t7-scmi.h>
> > +#include <dt-bindings/clock/amlogic,t7-pll-clkc.h>
> > +#include <dt-bindings/clock/amlogic,t7-peripherals-clkc.h>
> >
> >   / {
> >        interrupt-parent = <&gic>;
> > @@ -201,6 +204,33 @@ pwrc: power-controller {
> >                };
> >        };
> >
> > +     sram@f7042000 {
> > +             compatible = "mmio-sram";
> Applying your patches shows the following errors
>
>
> [    0.019608] sram sram@f7042000: error -EINVAL: invalid resource (null)
> [    0.019622] sram sram@f7042000: could not map SRAM registers
> [    0.019627] sram sram@f7042000: probe with driver sram failed with
> error -22
>
> Adding a reg remove those errors on kernel logs
> reg = <0x0 0xf7042000 0x0 0x100>;
>
> Can you have a look on this ?


Thanks for your feedback!

The -EINVAL error is caused by the missing 'reg' property in the sram node.
The SRAM driver requires this property to get the physical address range 
for MMIO mapping.

The 'ranges' property remains correct, and the 'reg' in the sram@0 
subnode is correct.
The SCMI clock driver works fine.

I will add the 'reg' property to the sram node to fix the kernel error 
logs in v2.

> > +             #address-cells = <1>;
> > +             #size-cells = <1>;
> > +             ranges = <0 0x0 0xf7042000 0x100>;
> > +
> > +             scmi_shmem: sram@0 {
> > +                     compatible = "arm,scmi-shmem";
> > +                     reg = <0x0 0x100>;
> > +             };
> > +     };
> > +
[...]
> > --
> > Best regards,
> > Ronald