Hello.

On 11/11/2016 08:07 PM, Ulrich Hecht wrote:

> Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas at gmail.com>
> ---
>  arch/arm64/boot/dts/renesas/r8a7795.dtsi | 34 
> ++++++++++++++++++++++++++++++--
>  1 file changed, 32 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/renesas/r8a7795.dtsi 
> b/arch/arm64/boot/dts/renesas/r8a7795.dtsi
> index 811c32e..d9e5706 100644
> --- a/arch/arm64/boot/dts/renesas/r8a7795.dtsi
> +++ b/arch/arm64/boot/dts/renesas/r8a7795.dtsi
> @@ -147,6 +147,30 @@
>               clock-frequency = <0>;
>       };
>
> +     programmable_clk0: clock_out0 {
> +             compatible = "fixed-clock";
> +             #clock-cells = <0>;
> +             clock-frequency = <0>;
> +     };
> +
> +     x21_clk: x21-clock {

    Please name it just x21 to math the documentation -- this will be 
reflected in the output clock name.

> +             compatible = "fixed-clock";
> +             #clock-cells = <0>;
> +             clock-frequency = <0>;
> +     };
> +
> +     x22_clk: x22-clock {

    Same here.

> +             compatible = "fixed-clock";
> +             #clock-cells = <0>;
> +             clock-frequency = <0>;
> +     };
> +
> +     programmable_clk1: clock_out1 {
> +             compatible = "fixed-clock";
> +             #clock-cells = <0>;
> +             clock-frequency = <0>;
> +     };
> +
>       soc {
>               compatible = "simple-bus";
>               interrupt-parent = <&gic>;
[...]

MBR, Sergei

Reply via email to