On Tue, Jul 09, 2013 at 06:02:00PM -0700, Kuninori Morimoto wrote: > > Hi Simon > > > > > diff --git a/drivers/mmc/host/sh_mobile_sdhi.c b/drivers/mmc/host/sh_mobile_sdhi.c > > > > index cc4c872..b58c1a9 100644 > > > > --- a/drivers/mmc/host/sh_mobile_sdhi.c > > > > +++ b/drivers/mmc/host/sh_mobile_sdhi.c > > > > @@ -130,6 +130,9 @@ static const struct of_device_id sh_mobile_sdhi_of_match[] = { > > > > { .compatible = "renesas,shmobile-sdhi" }, > > > > { .compatible = "renesas,sh7372-sdhi" }, > > > > { .compatible = "renesas,r8a7740-sdhi", .data = &sh_mobile_sdhi_of_cfg[0], }, > > > > + { .compatible = "renesas,sh73a0-sdhi", .data = &sh_mobile_sdhi_of_cfg[0], }, > > > > + { .compatible = "renesas,r8a73a4-sdhi", .data = &sh_mobile_sdhi_of_cfg[0], }, > > > > + { .compatible = "renesas,r8a7790-sdhi", .data = &sh_mobile_sdhi_of_cfg[0], }, > > > > {}, > > > > > > According to HW people, latest Renesas chip needs "TMIO_MMC_HAS_IDLE_WAIT". > > > Could you plase add r8a7778 / r8a7779 here ? > > > > > > # or we can use common compatible name for it ? > > > > Hi Guennadi, > > > > could you please address Morimoto-san's review? > > He did it on > > Subject: [PATCH v4 5/7] mmc: SDHI: add DT compatibility strings for further SoCs > Date: Tue, 9 Jul 2013 09:43:40 +0200 (CEST) Thanks for pointing that out. Somehow I missed it. -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html