On Sun, Feb 18, 2024 at 10:52 AM Chen Wang <unicornxw@xxxxxxxxx> wrote: > > From: Chen Wang <unicorn_wang@xxxxxxxxxxx> > > Add clock generator node to device tree for SG2042, and enable clock for > uart. > > Signed-off-by: Chen Wang <unicorn_wang@xxxxxxxxxxx> > --- > .../boot/dts/sophgo/sg2042-milkv-pioneer.dts | 12 +++++ > arch/riscv/boot/dts/sophgo/sg2042.dtsi | 48 +++++++++++++++++++ > 2 files changed, 60 insertions(+) > > diff --git a/arch/riscv/boot/dts/sophgo/sg2042-milkv-pioneer.dts b/arch/riscv/boot/dts/sophgo/sg2042-milkv-pioneer.dts > index 49b4b9c2c101..80cb017974d8 100644 > --- a/arch/riscv/boot/dts/sophgo/sg2042-milkv-pioneer.dts > +++ b/arch/riscv/boot/dts/sophgo/sg2042-milkv-pioneer.dts > @@ -14,6 +14,18 @@ chosen { > }; > }; > > +&cgi_main { > + clock-frequency = <25000000>; > +}; > + > +&cgi_dpll0 { > + clock-frequency = <25000000>; > +}; > + > +&cgi_dpll1 { > + clock-frequency = <25000000>; > +}; > + If all are fixed, why do we separate them into three pieces? To show real internal hardware? > &uart0 { > status = "okay"; > }; > diff --git a/arch/riscv/boot/dts/sophgo/sg2042.dtsi b/arch/riscv/boot/dts/sophgo/sg2042.dtsi > index ead1cc35d88b..e70c43e2ccbe 100644 > --- a/arch/riscv/boot/dts/sophgo/sg2042.dtsi > +++ b/arch/riscv/boot/dts/sophgo/sg2042.dtsi > @@ -5,6 +5,9 @@ > > /dts-v1/; > #include <dt-bindings/interrupt-controller/irq.h> > +#include <dt-bindings/clock/sophgo,sg2042-pll.h> > +#include <dt-bindings/clock/sophgo,sg2042-rpgate.h> > +#include <dt-bindings/clock/sophgo,sg2042-clkgen.h> > > #include "sg2042-cpus.dtsi" > > @@ -18,12 +21,54 @@ aliases { > serial0 = &uart0; > }; > > + cgi_main: oscillator0 { > + compatible = "fixed-clock"; > + clock-output-names = "cgi_main"; > + #clock-cells = <0>; > + }; > + > + cgi_dpll0: oscillator1 { > + compatible = "fixed-clock"; > + clock-output-names = "cgi_dpll0"; > + #clock-cells = <0>; > + }; > + > + cgi_dpll1: oscillator2 { > + compatible = "fixed-clock"; > + clock-output-names = "cgi_dpll1"; > + #clock-cells = <0>; > + }; > + > soc: soc { > compatible = "simple-bus"; > #address-cells = <2>; > #size-cells = <2>; > ranges; > > + pllclk: clock-controller@70300100c0 { > + compatible = "sophgo,sg2042-pll"; > + reg = <0x70 0x300100c0 0x0 0x40>; > + clocks = <&cgi_main>, <&cgi_dpll0>, <&cgi_dpll1>; > + #clock-cells = <1>; > + }; > + > + rpgate: clock-controller@7030010368 { > + compatible = "sophgo,sg2042-rpgate"; > + reg = <0x70 0x30010368 0x0 0x98>; > + clocks = <&clkgen GATE_CLK_RP_CPU_NORMAL>; > + #clock-cells = <1>; > + }; > + > + clkgen: clock-controller@7030012000 { > + compatible = "sophgo,sg2042-clkgen"; > + reg = <0x70 0x30012000 0x0 0x1000>; > + clocks = <&pllclk MPLL_CLK>, > + <&pllclk FPLL_CLK>, > + <&pllclk DPLL0_CLK>, > + <&pllclk DPLL1_CLK>; > + #clock-cells = <1>; > + }; > + > clint_mswi: interrupt-controller@7094000000 { > compatible = "sophgo,sg2042-aclint-mswi", "thead,c900-aclint-mswi"; > reg = <0x00000070 0x94000000 0x00000000 0x00004000>; > @@ -333,6 +378,9 @@ uart0: serial@7040000000 { > interrupt-parent = <&intc>; > interrupts = <112 IRQ_TYPE_LEVEL_HIGH>; > clock-frequency = <500000000>; > + clocks = <&clkgen GATE_CLK_UART_500M>, > + <&clkgen GATE_CLK_APB_UART>; > + clock-names = "baudclk", "apb_pclk"; > reg-shift = <2>; > reg-io-width = <4>; > status = "disabled"; > -- > 2.25.1 > Others LGTM. Reviewed-by: Guo Ren <guoren@xxxxxxxxxx> -- Best Regards Guo Ren