Re: [PATCH 2/2] arm64: dts: renesas: r8a774a1: Replace clock magic numbers

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

 



On Wed, Nov 7, 2018 at 4:24 PM Fabrizio Castro
<fabrizio.castro@xxxxxxxxxxxxxx> wrote:
> Now that include/dt-bindings/clock/r8a774a1-cpg-mssr.h is in Linus'
> master branch we can replace clock related magic numbers with the
> corresponding labels.
>
> Signed-off-by: Fabrizio Castro <fabrizio.castro@xxxxxxxxxxxxxx>

Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>

> --- a/arch/arm64/boot/dts/renesas/r8a774a1.dtsi
> +++ b/arch/arm64/boot/dts/renesas/r8a774a1.dtsi

> @@ -87,7 +87,7 @@
>                         power-domains = <&sysc R8A774A1_PD_CA53_CPU0>;
>                         next-level-cache = <&L2_CA53>;
>                         enable-method = "psci";
> -                       clocks =<&cpg CPG_CORE 1>;
> +                       clocks =<&cpg CPG_CORE R8A774A1_CLK_Z2>;

There are a few pre-existing whitespace issues in the CPU nodes' clocks
properties.

>                 };
>
>                 a53_1: cpu@101 {
> @@ -97,7 +97,7 @@
>                         power-domains = <&sysc R8A774A1_PD_CA53_CPU1>;
>                         next-level-cache = <&L2_CA53>;
>                         enable-method = "psci";
> -                       clocks =<&cpg CPG_CORE 1>;
> +                       clocks =<&cpg CPG_CORE R8A774A1_CLK_Z2>;
>                 };
>
>                 a53_2: cpu@102 {
> @@ -107,7 +107,7 @@
>                         power-domains = <&sysc R8A774A1_PD_CA53_CPU2>;
>                         next-level-cache = <&L2_CA53>;
>                         enable-method = "psci";
> -                       clocks =<&cpg CPG_CORE 1>;
> +                       clocks =<&cpg CPG_CORE R8A774A1_CLK_Z2>;
>                 };
>
>                 a53_3: cpu@103 {
> @@ -117,7 +117,7 @@
>                         power-domains = <&sysc R8A774A1_PD_CA53_CPU3>;
>                         next-level-cache = <&L2_CA53>;
>                         enable-method = "psci";
> -                       clocks =<&cpg CPG_CORE 1>;
> +                       clocks =<&cpg CPG_CORE R8A774A1_CLK_Z2>;

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux