Hello Simon, > From: Simon Horman <horms@xxxxxxxxxxxx> > Sent: 11 February 2019 09:50 > Subject: Re: [PATCH 08/12] arm64: dts: renesas: r8a774c0: Add OPPs table for cpu devices > > On Fri, Feb 08, 2019 at 03:26:49PM +0000, Fabrizio Castro wrote: > > Hello Sergei, > > > > Thank you for your feedback! > > > > > -----Original Message----- > > > From: Sergei Shtylyov <sergei.shtylyov@xxxxxxxxxxxxxxxxxx> > > > Sent: 08 February 2019 12:53 > > > To: Simon Horman <horms+renesas@xxxxxxxxxxxx>; linux-renesas-soc@xxxxxxxxxxxxxxx > > > Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; Magnus Damm <magnus.damm@xxxxxxxxx>; Fabrizio Castro > > > <fabrizio.castro@xxxxxxxxxxxxxx> > > > Subject: Re: [PATCH 08/12] arm64: dts: renesas: r8a774c0: Add OPPs table for cpu devices > > > > > > Hello! > > > > > > On 02/08/2019 02:13 PM, Simon Horman wrote: > > > > > > > From: Fabrizio Castro <fabrizio.castro@xxxxxxxxxxxxxx> > > > > > > > > This patch defines OOP tables for all CPUs, similarly to > > > > what done by Takeshi Kihara and Yoshihiro Kaneko for the > > > > R8A77990. > > > > > > > > Signed-off-by: Fabrizio Castro <fabrizio.castro@xxxxxxxxxxxxxx> > > > > Signed-off-by: Simon Horman <horms+renesas@xxxxxxxxxxxx> > > > [...] > > > > @@ -55,6 +76,8 @@ > > > > power-domains = <&sysc R8A774C0_PD_CA53_CPU0>; > > > > next-level-cache = <&L2_CA53>; > > > > enable-method = "psci"; > > > > +clocks =<&cpg CPG_CORE R8A774C0_CLK_Z2>; > > > > > > Need space after =... > > > > Doh! Simon, do you want me to send another version to fix both spacing issues? > > Likewise, sorry I didn't notice that earlier. > > I'd rather not re-send this pull request just to resolve whitespace issues. > So please send an incremental patch for now. Thanks, will do. Cheers, Fab Renesas Electronics Europe GmbH,Geschaeftsfuehrer/President : Michael Hannawald, Sitz der Gesellschaft/Registered office: Duesseldorf, Arcadiastrasse 10, 40472 Duesseldorf, Germany,Handelsregister/Commercial Register: Duesseldorf, HRB 3708 USt-IDNr./Tax identification no.: DE 119353406 WEEE-Reg.-Nr./WEEE reg. no.: DE 14978647