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

2018-11-15 Thread Fabrizio Castro
Thank you Geert for spotting the issue, thank you Simon for fixing. Cheers, Fab > From: Simon Horman > Sent: 13 November 2018 14:46 > Subject: Re: [PATCH 2/2] arm64: dts: renesas: r8a774a1: Replace clock magic > numbers > > On Tue, Nov 13, 2018 at 09:53:55AM +0100, Geert

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

2018-11-13 Thread Simon Horman
On Tue, Nov 13, 2018 at 09:53:55AM +0100, Geert Uytterhoeven wrote: > On Wed, Nov 7, 2018 at 4:24 PM Fabrizio Castro > 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. > >

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

2018-11-13 Thread Geert Uytterhoeven
On Wed, Nov 7, 2018 at 4:24 PM Fabrizio Castro 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 Reviewed-by: Geert Uytterhoeven > ---

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

2018-11-08 Thread Simon Horman
On Wed, Nov 07, 2018 at 03:24:27PM +, Fabrizio Castro 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 Thanks, This looks fine to me

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

2018-11-07 Thread Fabrizio Castro
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 --- arch/arm64/boot/dts/renesas/r8a774a1.dtsi | 38 +++ 1 file changed, 19