Re: [PATCH] arm64: dts: renesas: r8a774c0: Fix cpu nodes style

2019-03-04 Thread Simon Horman
On Fri, Mar 01, 2019 at 12:00:39PM +, Fabrizio Castro wrote:
> Hello Simon,
> 
> Are you happy with this patch?
> 
> Thanks,
> Fab
> 
> > From: Fabrizio Castro 
> > Sent: 15 February 2019 12:26
> > Subject: [PATCH] arm64: dts: renesas: r8a774c0: Fix cpu nodes style
> > 
> > We usually leave a space between "=" and the value of device
> > tree properties, but unfortunately that was overlooked for the
> > "clocks" property of cpu@0 and cpu@1.
> > This patch fixes the spacing with the "clocks" property of
> > cpu@0 and cpu@1.
> > 
> > Signed-off-by: Fabrizio Castro 

Thanks, and sorry for the delay.

Applied for v5.2.

> > ---
> >  arch/arm64/boot/dts/renesas/r8a774c0.dtsi | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/arch/arm64/boot/dts/renesas/r8a774c0.dtsi 
> > b/arch/arm64/boot/dts/renesas/r8a774c0.dtsi
> > index 61a0afb..0bbcaf1 100644
> > --- a/arch/arm64/boot/dts/renesas/r8a774c0.dtsi
> > +++ b/arch/arm64/boot/dts/renesas/r8a774c0.dtsi
> > @@ -76,7 +76,7 @@
> > power-domains = <&sysc R8A774C0_PD_CA53_CPU0>;
> > next-level-cache = <&L2_CA53>;
> > enable-method = "psci";
> > -   clocks =<&cpg CPG_CORE R8A774C0_CLK_Z2>;
> > +   clocks = <&cpg CPG_CORE R8A774C0_CLK_Z2>;
> > operating-points-v2 = <&cluster1_opp>;
> > };
> > 
> > @@ -87,7 +87,7 @@
> > power-domains = <&sysc R8A774C0_PD_CA53_CPU1>;
> > next-level-cache = <&L2_CA53>;
> > enable-method = "psci";
> > -   clocks =<&cpg CPG_CORE R8A774C0_CLK_Z2>;
> > +   clocks = <&cpg CPG_CORE R8A774C0_CLK_Z2>;
> > operating-points-v2 = <&cluster1_opp>;
> > };
> > 
> > --
> > 2.7.4
> 


RE: [PATCH] arm64: dts: renesas: r8a774c0: Fix cpu nodes style

2019-03-01 Thread Fabrizio Castro
Hello Simon,

Are you happy with this patch?

Thanks,
Fab

> From: Fabrizio Castro 
> Sent: 15 February 2019 12:26
> Subject: [PATCH] arm64: dts: renesas: r8a774c0: Fix cpu nodes style
> 
> We usually leave a space between "=" and the value of device
> tree properties, but unfortunately that was overlooked for the
> "clocks" property of cpu@0 and cpu@1.
> This patch fixes the spacing with the "clocks" property of
> cpu@0 and cpu@1.
> 
> Signed-off-by: Fabrizio Castro 
> ---
>  arch/arm64/boot/dts/renesas/r8a774c0.dtsi | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/renesas/r8a774c0.dtsi 
> b/arch/arm64/boot/dts/renesas/r8a774c0.dtsi
> index 61a0afb..0bbcaf1 100644
> --- a/arch/arm64/boot/dts/renesas/r8a774c0.dtsi
> +++ b/arch/arm64/boot/dts/renesas/r8a774c0.dtsi
> @@ -76,7 +76,7 @@
>   power-domains = <&sysc R8A774C0_PD_CA53_CPU0>;
>   next-level-cache = <&L2_CA53>;
>   enable-method = "psci";
> - clocks =<&cpg CPG_CORE R8A774C0_CLK_Z2>;
> + clocks = <&cpg CPG_CORE R8A774C0_CLK_Z2>;
>   operating-points-v2 = <&cluster1_opp>;
>   };
> 
> @@ -87,7 +87,7 @@
>   power-domains = <&sysc R8A774C0_PD_CA53_CPU1>;
>   next-level-cache = <&L2_CA53>;
>   enable-method = "psci";
> - clocks =<&cpg CPG_CORE R8A774C0_CLK_Z2>;
> + clocks = <&cpg CPG_CORE R8A774C0_CLK_Z2>;
>   operating-points-v2 = <&cluster1_opp>;
>   };
> 
> --
> 2.7.4



Re: [PATCH] arm64: dts: renesas: r8a774c0: Fix cpu nodes style

2019-02-15 Thread Geert Uytterhoeven
On Fri, Feb 15, 2019 at 1:26 PM Fabrizio Castro
 wrote:
> We usually leave a space between "=" and the value of device
> tree properties, but unfortunately that was overlooked for the
> "clocks" property of cpu@0 and cpu@1.
> This patch fixes the spacing with the "clocks" property of
> cpu@0 and cpu@1.
>
> Signed-off-by: Fabrizio Castro 

Reviewed-by: Geert Uytterhoeven 

Gr{oetje,eeting}s,

Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

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