Hi Ovidiu,
On Tue, 20 Jan 2026 at 16:06, Ovidiu Panait
<ovidiu.panait.rb@renesas.com> wrote:
> Add versa3 clock generator node. It provides clocks for the RTC, PCIe
> and audio devices.
>
> Signed-off-by: Ovidiu Panait <ovidiu.panait.rb@renesas.com>
Thanks for your patch!
> --- a/arch/arm64/boot/dts/renesas/r9a09g057h44-rzv2h-evk.dts
> +++ b/arch/arm64/boot/dts/renesas/r9a09g057h44-rzv2h-evk.dts
> @@ -108,6 +108,12 @@ vqmmc_sdhi1: regulator-vccq-sdhi1 {
> states = <3300000 0>, <1800000 1>;
> };
>
> + x1: x1-clock {
> + compatible = "fixed-clock";
> + #clock-cells = <0>;
> + clock-frequency = <24000000>;
> + };
> +
> /* 32.768kHz crystal */
> x6: x6-clock {
> compatible = "fixed-clock";
> @@ -277,6 +283,25 @@ raa215300: pmic@12 {
> clocks = <&x6>;
> clock-names = "xin";
> };
> +
> + versa3: clock-generator@69 {
> + compatible = "renesas,5l35023";
> + reg = <0x69>;
> + clocks = <&x1>;
> + #clock-cells = <1>;
> + assigned-clocks = <&versa3 0>,
> + <&versa3 1>,
> + <&versa3 2>,
> + <&versa3 3>,
> + <&versa3 4>,
> + <&versa3 5>;
> + assigned-clock-rates = <24000000>,
Shouldn't (at least) qextal_clk and
> + <24576000>,
> + <32768>,
rtxin_clk be overridden to point to the corresponding versa3 outputs?
Same comment for [PATCH v2 5/5].
> + <22579200>,
> + <100000000>,
> + <100000000>;
> + };
> };
>
> &mdio0 {
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
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