Re: [PATCH 2/5] arm64: dts: exynos: Add missing parent clocks to audio block in Exynos 5433 SoC

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Marek,

2016-11-16 22:06 GMT+09:00 Marek Szyprowski <m.szyprowski@xxxxxxxxxxx>:
> Audio pll and oscilator clocks are proper parent clocks for AUD CMU.
> They are not visible as such on first glance on Exynos 5433 SoC docs,
> but they are needed for this CMU to operate properly.
>
> Signed-off-by: Marek Szyprowski <m.szyprowski@xxxxxxxxxxx>
> ---
>  arch/arm64/boot/dts/exynos/exynos5433.dtsi | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/exynos/exynos5433.dtsi b/arch/arm64/boot/dts/exynos/exynos5433.dtsi
> index 6564875..a80eb4c 100644
> --- a/arch/arm64/boot/dts/exynos/exynos5433.dtsi
> +++ b/arch/arm64/boot/dts/exynos/exynos5433.dtsi
> @@ -374,6 +374,8 @@
>                         compatible = "samsung,exynos5433-cmu-aud";
>                         reg = <0x114c0000 0x1000>;
>                         #clock-cells = <1>;
> +                       clock-names = "oscclk", "fout_aud_pll";
> +                       clocks = <&xxti>, <&cmu_top CLK_FOUT_AUD_PLL>;
>                 };
>
>                 cmu_bus0: clock-controller@13600000 {

Reviewed-by: Chanwoo Choi <cw00.choi@xxxxxxxxxxx>

Also, you should update the binding documentation of exynos5433's clock[1].
[1] Documentation/devicetree/bindings/clock/exynos5433-clock.txt

Best Regards,
Chanwoo Choi
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux