Hi Wolfram,
Thanks for your work.
On 2021-04-13 10:31:37 +0200, Wolfram Sang wrote:
> Currently, only SDHI on R-Car Gen2+ reinitializes the irq register
> during reset but it should be done on all instances. We can move it from
> the SDHI driver to the TMIO core, because we now have the
> 'sd_irq_mask_all' variable which carries the proper value to use. That
> also means we can remove the initialization from tmio_mmc_probe()
> because it calls tmio_mmc_reset(), too. We only move that
> tmio_mmc_reset() call there a little to ensure 'sd_irq_mask_all' is
> properly set.
>
> Signed-off-by: Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx>
I like patches like this.
Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx>
> ---
>
> Changes since v1:
>
> * use direct register write to initialize irq register instead of
> masking bits. Also initialize the cache variable directly.
>
> drivers/mmc/host/renesas_sdhi_core.c | 2 --
> drivers/mmc/host/tmio_mmc_core.c | 11 ++++++-----
> 2 files changed, 6 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/mmc/host/renesas_sdhi_core.c b/drivers/mmc/host/renesas_sdhi_core.c
> index d36181b6f687..635bf31a6735 100644
> --- a/drivers/mmc/host/renesas_sdhi_core.c
> +++ b/drivers/mmc/host/renesas_sdhi_core.c
> @@ -588,8 +588,6 @@ static void renesas_sdhi_reset(struct tmio_mmc_host *host)
> renesas_sdhi_scc_reset(host, priv);
> }
>
> - sd_ctrl_write32_as_16_and_16(host, CTL_IRQ_MASK, TMIO_MASK_ALL_RCAR2);
> -
> if (sd_ctrl_read16(host, CTL_VERSION) >= SDHI_VER_GEN3_SD) {
> val = sd_ctrl_read16(host, CTL_SD_MEM_CARD_OPT);
> val |= CARD_OPT_EXTOP;
> diff --git a/drivers/mmc/host/tmio_mmc_core.c b/drivers/mmc/host/tmio_mmc_core.c
> index 0c474d78b186..7dfc26f48c18 100644
> --- a/drivers/mmc/host/tmio_mmc_core.c
> +++ b/drivers/mmc/host/tmio_mmc_core.c
> @@ -192,6 +192,9 @@ static void tmio_mmc_reset(struct tmio_mmc_host *host)
> if (host->reset)
> host->reset(host);
>
> + sd_ctrl_write32_as_16_and_16(host, CTL_IRQ_MASK, host->sdcard_irq_mask_all);
> + host->sdcard_irq_mask = host->sdcard_irq_mask_all;
> +
> tmio_mmc_set_bus_width(host, host->mmc->ios.bus_width);
>
> if (host->pdata->flags & TMIO_MMC_SDIO_IRQ) {
> @@ -1176,13 +1179,11 @@ int tmio_mmc_host_probe(struct tmio_mmc_host *_host)
> if (pdata->flags & TMIO_MMC_SDIO_IRQ)
> _host->sdio_irq_mask = TMIO_SDIO_MASK_ALL;
>
> - _host->set_clock(_host, 0);
> - tmio_mmc_reset(_host);
> -
> - _host->sdcard_irq_mask = sd_ctrl_read16_and_16_as_32(_host, CTL_IRQ_MASK);
> if (!_host->sdcard_irq_mask_all)
> _host->sdcard_irq_mask_all = TMIO_MASK_ALL;
> - tmio_mmc_disable_mmc_irqs(_host, _host->sdcard_irq_mask_all);
> +
> + _host->set_clock(_host, 0);
> + tmio_mmc_reset(_host);
>
> if (_host->native_hotplug)
> tmio_mmc_enable_mmc_irqs(_host,
> --
> 2.30.0
>
--
Regards,
Niklas Söderlund
