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