Re: [cip-dev] [PATCH 4.19.y-cip] mmc: renesas_sdhi: abort tuning when timeout detected

[Pavel Machek <pavel@nabladev.com>]

[-- Attachment #1: Type: text/plain, Size: 1320 bytes --]

Hi!

> > From: Wolfram Sang <wsa+renesas@sang-engineering.com>
> > 
> > commit 2c9017d0b5d3fbf17e69577a42d9e610ca122810 upstream.
> > 
> > We have to bring the eMMC from sending-data state back to transfer state once
> > we detected a CRC error (timeout) during tuning. So, send a stop command via
> > mmc_abort_tuning().
> > 
> > Fixes: 4f11997773b6 ("mmc: tmio: Add tuning support") Reported-by Yoshihiro
> > Shimoda <yoshihiro.shimoda.uh@renesas.com>
> > Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
> > Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
> > Reviewed-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
> > Tested-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
> > Link:
> > https://lore.kernel.org/r/20210602073435.5955-1-wsa+renesas@sang-engine
> > ering.com
> > Cc: stable@vger.kernel.org
> > Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
> > [PL: manually applied the changes]
> > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> 
> I reviewed this series, looks good to me.
> 
> Reviewed-by: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.x90@mail.toshiba>

Patch looks okay to me, too. Adding Ulrich to the cc.

Reviewed-by: Pavel Machek <pavel@nabladev.com>

Best regards,
								Pavel

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]