On Tue, May 22, 2018 at 04:26:26PM +0200, Ulf Hansson wrote: > Move the calls to ->prepare_hs400_tuning(), from mmc_retune() into > mmc_hs400_to_hs200(), as it better belongs there, rather than being generic > to all type of cards. > > Signed-off-by: Ulf Hansson <ulf.hansson@xxxxxxxxxx> Thanks Ulf, this looks good to me. Do you want me to carry it in my hs400 for SDHI series, which would allow me to exercise it there before it is applied, or will you go ahead and apply it? In any case: Reviewed-by: Simon Horman <horms+renesas@xxxxxxxxxxxx> > --- > drivers/mmc/core/host.c | 3 --- > drivers/mmc/core/mmc.c | 4 ++++ > 2 files changed, 4 insertions(+), 3 deletions(-) > > diff --git a/drivers/mmc/core/host.c b/drivers/mmc/core/host.c > index c57ffff..abf9e88 100644 > --- a/drivers/mmc/core/host.c > +++ b/drivers/mmc/core/host.c > @@ -143,9 +143,6 @@ int mmc_retune(struct mmc_host *host) > goto out; > > return_to_hs400 = true; > - > - if (host->ops->prepare_hs400_tuning) > - host->ops->prepare_hs400_tuning(host, &host->ios); > } > > err = mmc_execute_tuning(host->card); > diff --git a/drivers/mmc/core/mmc.c b/drivers/mmc/core/mmc.c > index 57a8bd3..4466f5d 100644 > --- a/drivers/mmc/core/mmc.c > +++ b/drivers/mmc/core/mmc.c > @@ -1282,6 +1282,10 @@ int mmc_hs400_to_hs200(struct mmc_card *card) > > mmc_set_bus_speed(card); > > + /* Prepare tuning for HS400 mode. */ > + if (host->ops->prepare_hs400_tuning) > + host->ops->prepare_hs400_tuning(host, &host->ios); > + > return 0; > > out_err: > -- > 2.7.4 > -- 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
