mbox series

[v3,0/2] mmc: renesas_sdhi: avoid bad TAPs for Gen3

Message ID 20200423130432.9990-1-wsa+renesas@sang-engineering.com (mailing list archive)
Headers show
Series mmc: renesas_sdhi: avoid bad TAPs for Gen3 | expand

Message

Wolfram Sang April 23, 2020, 1:04 p.m. UTC
From: Wolfram Sang <wsa@the-dreams.de>

Here is a series which avoids using TAPs on R-Car Gen3 SoCs which are
bad to use according to HW engineers. The patch was originally for the
BSP (Thanks, Saito-san) and has been upported for the upstream driver.
Before we can apply this, we need to add some mising information for
R-Car M3-W ES1.[23].

Patches have been tested on Renesas Salvator-XS boards with R-Car H3
ES2.0 (changing from TAP3 to TAP4 here) and R-Car M3-N (changing from
TAP7 to TAP0 here).

A branch can be found here:

git://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux.git renesas/sdhi/new_manual_calib

It is based on current mmc/next + "[PATCH] mmc: renesas_sdhi: shorten
types after refactorization"

One implementation detail: For now, I kept using a generic naming for
the quirks, e.g. "sdhi_quirks_bad_taps2367". It is possible that we need
to rename/add ones later which are SoC bound, e.g. "sdhi_quirks_m3w_30".
Especially when we add manual calibration somewhen. But for now, the
generic approach seems better to me, so I sticked to it.

Looking forward to further comments and/or testing.

Happy hacking,

   Wolfram

Takeshi Saito (1):
  mmc: renesas_sdhi: Avoid bad TAP in HS400

Wolfram Sang (1):
  mmc: renesas_sdhi: handle M3-W ES1.2 and 1.3 revisions

 drivers/mmc/host/renesas_sdhi.h      |  1 +
 drivers/mmc/host/renesas_sdhi_core.c | 61 ++++++++++++++++++++++++----
 2 files changed, 54 insertions(+), 8 deletions(-)

Comments

Ulf Hansson May 5, 2020, 11:32 a.m. UTC | #1
On Thu, 23 Apr 2020 at 15:04, Wolfram Sang
<wsa+renesas@sang-engineering.com> wrote:
>
> From: Wolfram Sang <wsa@the-dreams.de>
>
> Here is a series which avoids using TAPs on R-Car Gen3 SoCs which are
> bad to use according to HW engineers. The patch was originally for the
> BSP (Thanks, Saito-san) and has been upported for the upstream driver.
> Before we can apply this, we need to add some mising information for
> R-Car M3-W ES1.[23].
>
> Patches have been tested on Renesas Salvator-XS boards with R-Car H3
> ES2.0 (changing from TAP3 to TAP4 here) and R-Car M3-N (changing from
> TAP7 to TAP0 here).
>
> A branch can be found here:
>
> git://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux.git renesas/sdhi/new_manual_calib
>
> It is based on current mmc/next + "[PATCH] mmc: renesas_sdhi: shorten
> types after refactorization"
>
> One implementation detail: For now, I kept using a generic naming for
> the quirks, e.g. "sdhi_quirks_bad_taps2367". It is possible that we need
> to rename/add ones later which are SoC bound, e.g. "sdhi_quirks_m3w_30".
> Especially when we add manual calibration somewhen. But for now, the
> generic approach seems better to me, so I sticked to it.
>
> Looking forward to further comments and/or testing.
>
> Happy hacking,
>
>    Wolfram
>
> Takeshi Saito (1):
>   mmc: renesas_sdhi: Avoid bad TAP in HS400
>
> Wolfram Sang (1):
>   mmc: renesas_sdhi: handle M3-W ES1.2 and 1.3 revisions
>
>  drivers/mmc/host/renesas_sdhi.h      |  1 +
>  drivers/mmc/host/renesas_sdhi_core.c | 61 ++++++++++++++++++++++++----
>  2 files changed, 54 insertions(+), 8 deletions(-)
>
> --
> 2.20.1
>

Applied for next, thanks!

Kind regards
Uffe