On Fri, May 27, 2016 at 01:59:34PM +0200, Wolfram Sang wrote: > On Tue, May 24, 2016 at 11:43:16AM +0900, Simon Horman wrote: > > From: Ai Kyuse <ai.kyuse.uw@xxxxxxxxxxx> > > I wonder if you shouldn't take over ownership of this and the previous > patch? You changed quite a lot. Sure, will do. [snip] > > + > > +static unsigned int sh_mobile_sdhi_init_tuning(struct tmio_mmc_host *host) > > +{ > > + if (!(host->mmc->caps & MMC_CAP_UHS_SDR104)) > > + return 0; > > Will the core call us if MMC_CAP_UHS_SDR104 was not set? Empirically, yes, that appears to be the case. For all your other suggestions: thanks, I will update things as you suggest.