From mboxrd@z Thu Jan 1 00:00:00 1970 From: Geert Uytterhoeven Subject: Re: [PATCH ] mmc: renesas_sdhi: Add r8a7743/5 support Date: Mon, 14 Aug 2017 17:20:19 +0200 Message-ID: References: <1502709581-26546-1-git-send-email-biju.das@bp.renesas.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Return-path: In-Reply-To: Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Chris Paterson Cc: Biju Das , Ulf Hansson , Wolfram Sang , Rob Herring , Mark Rutland , Simon Horman , Magnus Damm , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Linux-Renesas , "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" , Linux MMC List List-Id: devicetree@vger.kernel.org Hi Chris, On Mon, Aug 14, 2017 at 5:15 PM, Chris Paterson wrote: >> From: geert.uytterhoeven-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org [mailto:geert.uytterhoeven-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org] >> On Behalf Of Geert Uytterhoeven >> Sent: 14 August 2017 15:22 >> >> On Mon, Aug 14, 2017 at 1:19 PM, Biju Das wrote: >> > Add support for r8a7743/5 SoC.Renesas RZ/G1[ME] (R8A7743/5) SDHI is >> > identical to the R-Car Gen2 family. >> > >> > Signed-off-by: Biju Das >> > --- >> > This patch is compiled and tested against mmc/next. >> > >> > drivers/mmc/host/renesas_sdhi_sys_dmac.c | 2 ++ >> > 1 file changed, 2 insertions(+) >> > >> > diff --git a/drivers/mmc/host/renesas_sdhi_sys_dmac.c >> > b/drivers/mmc/host/renesas_sdhi_sys_dmac.c >> > index 718cb8a..f709f7f 100644 >> > --- a/drivers/mmc/host/renesas_sdhi_sys_dmac.c >> > +++ b/drivers/mmc/host/renesas_sdhi_sys_dmac.c >> > @@ -95,6 +95,8 @@ >> > { .compatible = "renesas,sdhi-r7s72100", .data = &of_rz_compatible, }, >> > { .compatible = "renesas,sdhi-r8a7778", .data = >> &of_rcar_gen1_compatible, }, >> > { .compatible = "renesas,sdhi-r8a7779", .data = >> > &of_rcar_gen1_compatible, }, >> > + { .compatible = "renesas,sdhi-r8a7743", .data = >> &of_rcar_gen2_compatible, }, >> > + { .compatible = "renesas,sdhi-r8a7745", .data = >> > + &of_rcar_gen2_compatible, }, >> > { .compatible = "renesas,sdhi-r8a7790", .data = >> &of_rcar_gen2_compatible, }, >> > { .compatible = "renesas,sdhi-r8a7791", .data = >> &of_rcar_gen2_compatible, }, >> > { .compatible = "renesas,sdhi-r8a7792", .data = >> > &of_rcar_gen2_compatible, }, >> >> I guess we want generic R-Car Gen2/3 compatible values here, too? > > Yes, but I assume that should be a separate patch submitted instead of this one? Indeed. That would avoid having to add the SoC-specific compatible values to the driver for all RZ/G1 variants. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert-Td1EMuHUCqxL1ZNQvxDV9g@public.gmane.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 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html