Re: [PATCH 06/22] ARM: dts: r8a7744-iwg20m: Enable SDHI10 controller
On Tue, Nov 27, 2018 at 1:05 PM Biju Das wrote:
> Enable the SDHI0 controller on iWave RZ/G1N Qseven System On Module.
>
> Signed-off-by: Biju Das
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
Re: [PATCH 06/22] ARM: dts: r8a7744-iwg20m: Enable SDHI10 controller
On Wed, Nov 28, 2018 at 02:15:55PM +, Biju Das wrote: > > > -Original Message- > > From: linux-renesas-soc-ow...@vger.kernel.org > ow...@vger.kernel.org> On Behalf Of Simon Horman > > Sent: 28 November 2018 13:24 > > To: Biju Das > > Cc: Sergei Shtylyov ; Rob Herring > > ; Mark Rutland ; Magnus > > Damm ; linux-renesas-soc@vger.kernel.org; > > devicet...@vger.kernel.org; Geert Uytterhoeven > > ; Chris Paterson > > ; Fabrizio Castro > > > > Subject: Re: [PATCH 06/22] ARM: dts: r8a7744-iwg20m: Enable SDHI10 > > > > > > > > On 11/27/2018 02:56 PM, Biju Das wrote: > > > > > > > > > Enable the SDHI0 controller on iWave RZ/G1N Qseven System On > > Module. > > > > > > > >Why your subject has SDHI10? > > > > > > It is a typo. > > > > > > Simon, Will you fix this while applying or do you want me to send V2? > > > Please let me know. > > > > No need to repost for this. > > Thanks Simon. Thanks again, I have now reviewed this patch and it looks fine to me but I will wait to see if there are other reviews before applying. Reviewed-by: Simon Horman
RE: [PATCH 06/22] ARM: dts: r8a7744-iwg20m: Enable SDHI10 controller
> -Original Message- > From: linux-renesas-soc-ow...@vger.kernel.org ow...@vger.kernel.org> On Behalf Of Simon Horman > Sent: 28 November 2018 13:24 > To: Biju Das > Cc: Sergei Shtylyov ; Rob Herring > ; Mark Rutland ; Magnus > Damm ; linux-renesas-soc@vger.kernel.org; > devicet...@vger.kernel.org; Geert Uytterhoeven > ; Chris Paterson > ; Fabrizio Castro > > Subject: Re: [PATCH 06/22] ARM: dts: r8a7744-iwg20m: Enable SDHI10 > > > > > > On 11/27/2018 02:56 PM, Biju Das wrote: > > > > > > > Enable the SDHI0 controller on iWave RZ/G1N Qseven System On > Module. > > > > > >Why your subject has SDHI10? > > > > It is a typo. > > > > Simon, Will you fix this while applying or do you want me to send V2? > > Please let me know. > > No need to repost for this. Thanks Simon. Regards, Biju Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709.
Re: [PATCH 06/22] ARM: dts: r8a7744-iwg20m: Enable SDHI10 controller
On Tue, Nov 27, 2018 at 02:22:12PM +, Biju Das wrote: > Hello Sergei, > > Thanks for the feedback. > > > -Original Message- > > From: Sergei Shtylyov > > Sent: 27 November 2018 14:17 > > To: Biju Das ; Rob Herring > > ; Mark Rutland > > Cc: Simon Horman ; Magnus Damm > > ; linux-renesas-soc@vger.kernel.org; > > devicet...@vger.kernel.org; Geert Uytterhoeven > > ; Chris Paterson > > ; Fabrizio Castro > > > > Subject: Re: [PATCH 06/22] ARM: dts: r8a7744-iwg20m: Enable SDHI10 > > controller > > > > Hello! > > > > On 11/27/2018 02:56 PM, Biju Das wrote: > > > > > Enable the SDHI0 controller on iWave RZ/G1N Qseven System On Module. > > > >Why your subject has SDHI10? > > It is a typo. > > Simon, Will you fix this while applying or do you want me to send V2? > Please let me know. No need to repost for this. > > > > Signed-off-by: Biju Das > > [...] > > > > MBR, Sergei > > > > Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, > Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered > No. 04586709.
RE: [PATCH 06/22] ARM: dts: r8a7744-iwg20m: Enable SDHI10 controller
Hello Sergei, Thanks for the feedback. > -Original Message- > From: Sergei Shtylyov > Sent: 27 November 2018 14:17 > To: Biju Das ; Rob Herring > ; Mark Rutland > Cc: Simon Horman ; Magnus Damm > ; linux-renesas-soc@vger.kernel.org; > devicet...@vger.kernel.org; Geert Uytterhoeven > ; Chris Paterson > ; Fabrizio Castro > > Subject: Re: [PATCH 06/22] ARM: dts: r8a7744-iwg20m: Enable SDHI10 > controller > > Hello! > > On 11/27/2018 02:56 PM, Biju Das wrote: > > > Enable the SDHI0 controller on iWave RZ/G1N Qseven System On Module. > >Why your subject has SDHI10? It is a typo. Simon, Will you fix this while applying or do you want me to send V2? Please let me know. > > Signed-off-by: Biju Das > [...] > > MBR, Sergei Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709.
Re: [PATCH 06/22] ARM: dts: r8a7744-iwg20m: Enable SDHI10 controller
Hello! On 11/27/2018 02:56 PM, Biju Das wrote: > Enable the SDHI0 controller on iWave RZ/G1N Qseven System On Module. Why your subject has SDHI10? > Signed-off-by: Biju Das [...] MBR, Sergei
