| Message ID | 1543319795-48325-6-git-send-email-biju.das@bp.renesas.com (mailing list archive) |
|---|---|
| State | Superseded |
| Delegated to: | Simon Horman |
| Headers | show |
| Series | Add more support to RZ/G1N | expand |
On Tue, Nov 27, 2018 at 11:56:18AM +0000, Biju Das wrote: > Add eMMC support for iWave RZ/G1N Qseven System On Module. > > Signed-off-by: Biju Das <biju.das@bp.renesas.com> Thanks, This looks fine to me but I will wait to see if there are other reviews before applying. Reviewed-by: Simon Horman <horms+renesas@verge.net.au>
On Tue, Nov 27, 2018 at 1:05 PM Biju Das <biju.das@bp.renesas.com> wrote: > Add eMMC support for iWave RZ/G1N Qseven System On Module. > > Signed-off-by: Biju Das <biju.das@bp.renesas.com> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> 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)? Gr{oetje,eeting}s, Geert
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@bp.renesas.com> wrote: > > Add eMMC support for iWave RZ/G1N Qseven System On Module. > > > > Signed-off-by: Biju Das <biju.das@bp.renesas.com> > > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> > > 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.
Hi Biju, On Fri, Nov 30, 2018 at 11:43 AM Biju Das <biju.das@bp.renesas.com> wrote: > > 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@bp.renesas.com> wrote: > > > Add eMMC support for iWave RZ/G1N Qseven System On Module. > > > > > > Signed-off-by: Biju Das <biju.das@bp.renesas.com> > > > > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> > > > > 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? OK, then it's fine for me. Gr{oetje,eeting}s, Geert
diff --git a/arch/arm/boot/dts/r8a7744-iwg20m.dtsi b/arch/arm/boot/dts/r8a7744-iwg20m.dtsi index 6166ae0..1e57b1f 100644 --- a/arch/arm/boot/dts/r8a7744-iwg20m.dtsi +++ b/arch/arm/boot/dts/r8a7744-iwg20m.dtsi @@ -29,3 +29,20 @@ &extal_clk { clock-frequency = <20000000>; }; + +&pfc { + mmcif0_pins: mmc { + groups = "mmc_data8_b", "mmc_ctrl"; + function = "mmc"; + }; +}; + +&mmcif0 { + pinctrl-0 = <&mmcif0_pins>; + pinctrl-names = "default"; + + vmmc-supply = <®_3p3v>; + bus-width = <8>; + non-removable; + status = "okay"; +};
Add eMMC support for iWave RZ/G1N Qseven System On Module. Signed-off-by: Biju Das <biju.das@bp.renesas.com> --- arch/arm/boot/dts/r8a7744-iwg20m.dtsi | 17 +++++++++++++++++ 1 file changed, 17 insertions(+)