Hi Krzysztof Kozlowski,
Thanks for the feedback.
> -----Original Message-----
> From: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
> Sent: 11 November 2022 08:35
> To: Biju Das <biju.das.jz@xxxxxxxxxxxxxx>; Rob Herring <robh+dt@xxxxxxxxxx>;
> Krzysztof Kozlowski <krzysztof.kozlowski+dt@xxxxxxxxxx>
> Cc: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>; Magnus Damm
> <magnus.damm@xxxxxxxxx>; linux-renesas-soc@xxxxxxxxxxxxxxx;
> devicetree@xxxxxxxxxxxxxxx; Chris Paterson <Chris.Paterson2@xxxxxxxxxxx>;
> Fabrizio Castro <fabrizio.castro.jz@xxxxxxxxxxx>
> Subject: Re: [PATCH v2 3/3] arm64: dts: renesas: r9a09g011: Add system
> configuration node
>
> On 10/11/2022 17:21, Biju Das wrote:
> > Add system configuration node to RZ/V2M SoC dtsi.
> >
> > Signed-off-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx>
> > ---
> > v2:
> > * New patch
> > ---
> > arch/arm64/boot/dts/renesas/r9a09g011.dtsi | 6 ++++++
> > 1 file changed, 6 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/renesas/r9a09g011.dtsi
> b/arch/arm64/boot/dts/renesas/r9a09g011.dtsi
> > index 7b949e40745a..07164d9e4a0f 100644
> > --- a/arch/arm64/boot/dts/renesas/r9a09g011.dtsi
> > +++ b/arch/arm64/boot/dts/renesas/r9a09g011.dtsi
> > @@ -130,6 +130,12 @@ cpg: clock-controller@a3500000 {
> > #power-domain-cells = <0>;
> > };
> >
> > + sysc: system-configuration@a3f03000 {
> > + compatible = "renesas,r9a09g011-sys";
> > + reg = <0 0xa3f03000 0 0x400>;
> > + status = "disabled";
>
> Why disabled? You do not have any other resources needed. This is odd.
OK, will enable by default. Currently the driver compatible is used for getting SoC
Major and Minor versions. But later will enhance to support more features.
Cheers,
Biju
