[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAPDyKFohKt0E7zjnq0c9MkOLEEKZPtonWeDfz4w4Pw23=+uNtQ@mail.gmail.com>
Date: Wed, 28 Aug 2024 17:23:24 +0200
From: Ulf Hansson <ulf.hansson@...aro.org>
To: Liming Sun <limings@...dia.com>
Cc: Adrian Hunter <adrian.hunter@...el.com>, David Thompson <davthompson@...dia.com>,
linux-mmc@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2] mmc: sdhci-of-dwcmshc: Add hw_reset() support for
BlueField-3 SoC
On Tue, 27 Aug 2024 at 18:40, Liming Sun <limings@...dia.com> wrote:
>
> The eMMC RST_N register is implemented as secure register on
> the BlueField-3 SoC and controlled by TF-A. This commit adds the
> hw_reset() support which sends an SMC call to TF-A for the eMMC
> HW reset.
>
> Reviewed-by: David Thompson <davthompson@...dia.com>
> Signed-off-by: Liming Sun <limings@...dia.com>
Applied for next, thanks!
Kind regards
Uffe
> ---
> v1->v2:
> Fixed a typo reported by test robot.
> v1: Initial version.
> ---
> drivers/mmc/host/sdhci-of-dwcmshc.c | 29 ++++++++++++++++++++++++++++-
> 1 file changed, 28 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/mmc/host/sdhci-of-dwcmshc.c b/drivers/mmc/host/sdhci-of-dwcmshc.c
> index ba8960d8b2d4..8999b97263af 100644
> --- a/drivers/mmc/host/sdhci-of-dwcmshc.c
> +++ b/drivers/mmc/host/sdhci-of-dwcmshc.c
> @@ -8,6 +8,7 @@
> */
>
> #include <linux/acpi.h>
> +#include <linux/arm-smccc.h>
> #include <linux/bitfield.h>
> #include <linux/clk.h>
> #include <linux/dma-mapping.h>
> @@ -201,6 +202,9 @@
> SDHCI_TRNS_BLK_CNT_EN | \
> SDHCI_TRNS_DMA)
>
> +/* SMC call for BlueField-3 eMMC RST_N */
> +#define BLUEFIELD_SMC_SET_EMMC_RST_N 0x82000007
> +
> enum dwcmshc_rk_type {
> DWCMSHC_RK3568,
> DWCMSHC_RK3588,
> @@ -1111,6 +1115,29 @@ static const struct sdhci_ops sdhci_dwcmshc_ops = {
> .irq = dwcmshc_cqe_irq_handler,
> };
>
> +#ifdef CONFIG_ACPI
> +static void dwcmshc_bf3_hw_reset(struct sdhci_host *host)
> +{
> + struct arm_smccc_res res = { 0 };
> +
> + arm_smccc_smc(BLUEFIELD_SMC_SET_EMMC_RST_N, 0, 0, 0, 0, 0, 0, 0, &res);
> +
> + if (res.a0)
> + pr_err("%s: RST_N failed.\n", mmc_hostname(host->mmc));
> +}
> +
> +static const struct sdhci_ops sdhci_dwcmshc_bf3_ops = {
> + .set_clock = sdhci_set_clock,
> + .set_bus_width = sdhci_set_bus_width,
> + .set_uhs_signaling = dwcmshc_set_uhs_signaling,
> + .get_max_clock = dwcmshc_get_max_clock,
> + .reset = sdhci_reset,
> + .adma_write_desc = dwcmshc_adma_write_desc,
> + .irq = dwcmshc_cqe_irq_handler,
> + .hw_reset = dwcmshc_bf3_hw_reset,
> +};
> +#endif
> +
> static const struct sdhci_ops sdhci_dwcmshc_rk35xx_ops = {
> .set_clock = dwcmshc_rk3568_set_clock,
> .set_bus_width = sdhci_set_bus_width,
> @@ -1163,7 +1190,7 @@ static const struct dwcmshc_pltfm_data sdhci_dwcmshc_pdata = {
> #ifdef CONFIG_ACPI
> static const struct dwcmshc_pltfm_data sdhci_dwcmshc_bf3_pdata = {
> .pdata = {
> - .ops = &sdhci_dwcmshc_ops,
> + .ops = &sdhci_dwcmshc_bf3_ops,
> .quirks = SDHCI_QUIRK_CAP_CLOCK_BASE_BROKEN,
> .quirks2 = SDHCI_QUIRK2_PRESET_VALUE_BROKEN |
> SDHCI_QUIRK2_ACMD23_BROKEN,
> --
> 2.30.1
>
Powered by blists - more mailing lists