mbox series

[v3,0/2] mmc: sdhci-of-esdhc: fix up erratum A-008171 workaround

Message ID 20191009074140.3934-1-yangbo.lu@nxp.com (mailing list archive)
Headers show
Series mmc: sdhci-of-esdhc: fix up erratum A-008171 workaround | expand

Message

Yangbo Lu Oct. 9, 2019, 7:41 a.m. UTC
This patch-set is to fix up erratum A-008171 workaround. It had been
verified on LS1046A eSDHC + Phison 32G eMMC which could trigger the
erratum.
---
Changes for v2:
	- Dropped sdhci start_tuning() hook patch.
	- Implemented SW tuning starting in eSDHC writew().
Changes for v3:
	- Converted to use esdhc_flush_async_fifo() and usleep_range().
	- Added ACK of Adrian.

Yangbo Lu (2):
  mmc: sdhci-of-esdhc: poll ESDHC_FLUSH_ASYNC_FIFO bit until completion
  mmc: sdhci-of-esdhc: fix up erratum A-008171 workaround

 drivers/mmc/host/sdhci-esdhc.h    |  14 +++
 drivers/mmc/host/sdhci-of-esdhc.c | 257 ++++++++++++++++++++++++++++++++------
 2 files changed, 231 insertions(+), 40 deletions(-)

Comments

Yangbo Lu Oct. 16, 2019, 2:20 a.m. UTC | #1
Any comments, Adrian?
Thanks.

Best regards,
Yangbo Lu

> -----Original Message-----
> From: Yangbo Lu <yangbo.lu@nxp.com>
> Sent: Wednesday, October 9, 2019 3:42 PM
> To: linux-mmc@vger.kernel.org; Ulf Hansson <ulf.hansson@linaro.org>; Adrian
> Hunter <adrian.hunter@intel.com>
> Cc: Y.b. Lu <yangbo.lu@nxp.com>
> Subject: [v3, 0/2] mmc: sdhci-of-esdhc: fix up erratum A-008171 workaround
> 
> This patch-set is to fix up erratum A-008171 workaround. It had been verified
> on LS1046A eSDHC + Phison 32G eMMC which could trigger the erratum.
> ---
> Changes for v2:
> 	- Dropped sdhci start_tuning() hook patch.
> 	- Implemented SW tuning starting in eSDHC writew().
> Changes for v3:
> 	- Converted to use esdhc_flush_async_fifo() and usleep_range().
> 	- Added ACK of Adrian.
> 
> Yangbo Lu (2):
>   mmc: sdhci-of-esdhc: poll ESDHC_FLUSH_ASYNC_FIFO bit until completion
>   mmc: sdhci-of-esdhc: fix up erratum A-008171 workaround
> 
>  drivers/mmc/host/sdhci-esdhc.h    |  14 +++
>  drivers/mmc/host/sdhci-of-esdhc.c | 257
> ++++++++++++++++++++++++++++++++------
>  2 files changed, 231 insertions(+), 40 deletions(-)
> 
> --
> 2.7.4
Ulf Hansson Oct. 18, 2019, 10:58 a.m. UTC | #2
On Wed, 9 Oct 2019 at 09:40, Yangbo Lu <yangbo.lu@nxp.com> wrote:
>
> This patch-set is to fix up erratum A-008171 workaround. It had been
> verified on LS1046A eSDHC + Phison 32G eMMC which could trigger the
> erratum.
> ---
> Changes for v2:
>         - Dropped sdhci start_tuning() hook patch.
>         - Implemented SW tuning starting in eSDHC writew().
> Changes for v3:
>         - Converted to use esdhc_flush_async_fifo() and usleep_range().
>         - Added ACK of Adrian.
>
> Yangbo Lu (2):
>   mmc: sdhci-of-esdhc: poll ESDHC_FLUSH_ASYNC_FIFO bit until completion
>   mmc: sdhci-of-esdhc: fix up erratum A-008171 workaround
>
>  drivers/mmc/host/sdhci-esdhc.h    |  14 +++
>  drivers/mmc/host/sdhci-of-esdhc.c | 257 ++++++++++++++++++++++++++++++++------
>  2 files changed, 231 insertions(+), 40 deletions(-)
>
> --
> 2.7.4
>

Applied for next, thanks!

Kind regards
Uffe