Quoting Kamil Konieczny (2018-11-29 07:51:32)
> Document imem clock bindings for SSS (Security SubSystem) and SlimSSS IPs.
>
> Signed-off-by: Kamil Konieczny <k.konieczny@xxxxxxxxxxxxxxxxxxx>
> ---
Fix the subject of this patch please. It's not arm64: or dts:
> .../bindings/clock/exynos5433-clock.txt | 23 +++++++++++++++++++
> 1 file changed, 23 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/clock/exynos5433-clock.txt b/Documentation/devicetree/bindings/clock/exynos5433-clock.txt
> index 50d5897c9849..4e4352bf5a0b 100644
> --- a/Documentation/devicetree/bindings/clock/exynos5433-clock.txt
> +++ b/Documentation/devicetree/bindings/clock/exynos5433-clock.txt
> @@ -264,6 +272,21 @@ Example 2: Examples of clock controller nodes are listed below.
> <&cmu_top CLK_SCLK_USBDRD30_FSYS>;
> };
>
> + cmu_imem: clock-controller@11060000 {
> + compatible = "samsung,exynos5433-cmu-imem";
> + reg = <0x11060000 0x1000>;
> + #clock-cells = <1>;
> +
> + clock-names = "oscclk",
How about just osc? clk seems redundant.
> + "aclk_imem_sssx_266",
> + "aclk_imem_266",
> + "aclk_imem_200";
And what is 'aclk'? Also redundant?
> + clocks = <&xxti>,
> + <&cmu_top CLK_DIV_ACLK_IMEM_SSSX_266>,
> + <&cmu_top CLK_DIV_ACLK_IMEM_266>,
> + <&cmu_top CLK_DIV_ACLK_IMEM_200>;
> + };
> +
> cmu_g2d: clock-controller@12460000 {
> compatible = "samsung,exynos5433-cmu-g2d";
> reg = <0x12460000 0x0b08>;
> --
> 2.19.1
>
