Re: [PATCH 08/13] arm64: dts: exynos: gs101: enable cmu-peric0 clock controller

Sam Protsenko
2023-12-14T15:39:47Z

On Thu, Dec 14, 2023 at 4:52 AM Tudor Ambarus <tudor.ambarus@linaro.org> wrote:
>
> Enable the cmu-peric0 clock controller. It feeds USI and I3c.
>
> Signed-off-by: Tudor Ambarus <tudor.ambarus@linaro.org>
> ---
> arch/arm64/boot/dts/exynos/google/gs101.dtsi | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/exynos/google/gs101.dtsi b/arch/arm64/boot/dts/exynos/google/gs101.dtsi
> index 9747cb3fa03a..d0b0ad70c6ba 100644
> --- a/arch/arm64/boot/dts/exynos/google/gs101.dtsi
> +++ b/arch/arm64/boot/dts/exynos/google/gs101.dtsi
> @@ -339,6 +339,18 @@ ppi_cluster2: interrupt-partition-2 {
> };
> };
>
> + cmu_peric0: clock-controller@10800000 {
> + compatible = "google,gs101-cmu-peric0";
> + reg = <0x10800000 0x4000>;
> + #clock-cells = <1>;
> + clocks = <&ext_24_5m>,
> + <&cmu_top CLK_DOUT_CMU_PERIC0_BUS>,
> + <&cmu_top CLK_DOUT_CMU_PERIC0_IP>;
> + clock-names = "oscclk",
> + "dout_cmu_peric0_bus",

I'd pull this line to the above line. Other than that:

Reviewed-by: Sam Protsenko <semen.protsenko@linaro.org>

> + "dout_cmu_peric0_ip";
> + };
> +
> sysreg_peric0: syscon@10820000 {
> compatible = "google,gs101-peric0-sysreg", "syscon";
> reg = <0x10820000 0x10000>;
> --
> 2.43.0.472.g3155946c3a-goog
>