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
