Hi Prabhakar, Thanks for your patch! On Mon, Feb 19, 2024 at 5:10 PM Prabhakar <prabhakar.csengg@xxxxxxxxx> wrote: > From: Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> > > Add DT binding documentation for System Controller (SYS) found on > RZ/V2H{P} ("R9A09G057") SoC's. RZ/V2H(P) > > SYS block contains the SYS_LSI_DEVID register which can be used to > retrieve SoC version information. > > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> > Reviewed-by: Fabrizio Castro <fabrizio.castro.jz@xxxxxxxxxxx> > --- /dev/null > +++ b/Documentation/devicetree/bindings/soc/renesas/renesas,r9a09g057-sys.yaml > @@ -0,0 +1,59 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/soc/renesas/renesas,r9a09g057-sys.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Renesas RZ/V2H{P} System Controller (SYS) > + > +maintainers: > + - Geert Uytterhoeven <geert+renesas@xxxxxxxxx> > + > +description: > + The RZ/V2H{P} SYS (System Controller) controls the overall RZ/V2H(P) > + configuration of the LSI and supports the following functions, > + - Trust zone control > + - Extend access by specific masters to address beyond 4GB space > + - GBETH configuration > + - Control of settings and states of SRAM/PCIe/CM33/CA55/CR8/xSPI/ADC/TSU > + - LSI version > + - WDT stop control > + - General registers > + > +properties: > + compatible: > + const: renesas,r9a09g057-sys > + > + reg: > + maxItems: 1 > + > + clocks: > + items: > + - description: Clock from external oscillator Isn't this SYS_0_PCLK inside the CPG? > + > + resets: > + items: > + - description: SYS_0_PRESETN reset signal > + > +required: > + - compatible > + - reg > + - clocks > + - resets > + > +additionalProperties: false > + > +examples: > + - | > + extal_clk: extal-clk { > + compatible = "fixed-clock"; > + #clock-cells = <0>; > + clock-frequency = <24000000>; > + }; > + > + sys: system-controller@10430000 { > + compatible = "renesas,r9a09g057-sys"; > + reg = <0x10430000 0x10000>; > + clocks = <&extal_clk>; clocks = <&cpg 1>; (I guess it will be 1 ;-) > + resets = <&cpg 1>; > + }; Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds