Hi Geert, Thanks for the feedback. > Subject: Re: [PATCH 05/22] ARM: dts: r8a7744-iwg20m: Add eMMC support > > On Tue, Nov 27, 2018 at 1:05 PM Biju Das <biju.das@xxxxxxxxxxxxxx> wrote: > > Add eMMC support for iWave RZ/G1N Qseven System On Module. > > > > Signed-off-by: Biju Das <biju.das@xxxxxxxxxxxxxx> > > Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> > > I assume the PCB is the same for RZ/G1M and RZ/G1N, so perhaps this can > be factored out in a common .dtsi file, like we did for Salvator-X(S)? SoM is different between RZ/G1M and RZ/G1N in terms of memory and PCie/SATA. The only common part in SoM is eMMC,SDHI0 and QSPI flash. So it doesn't make sense to factored it out. What do you think? Regards, Biju Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709.