Re: [External] : [PATCH] mmc: dwcmshc_bf3_hw_reset: Log eMMC reset calls

Satyansh Shukla <[email protected]>
Newsgroups org.kernel.vger.linux-mmc,org.kernel.vger.linux-kernel
Message-ID <SA1PR10MB5735E6FEC39BC2AA383FAFBC8EF62@SA1PR10MB5735.namprd10.prod.outlook.com>
This is my first upstream patch submission. Feedback on both the change(even though it is tiny) and submission process is appreciated.

Thanks.



________________________________________
From: Satyansh Shukla <[email protected]>
Sent: Thursday, June 18, 2026 4:23 PM
To: Ulf Hansson <[email protected]>
Cc: Adrian Hunter <[email protected]>; [email protected] <[email protected]>; [email protected] <[email protected]>
Subject: [External] : [PATCH] mmc: dwcmshc_bf3_hw_reset: Log eMMC reset calls
 
Log when the BlueField-3 eMMC hardware reset path is invoked and
whether the RST_N SMC call succeeds.

This makes it easier to diagnose cases where the controller recovery path
depends on issuing an eMMC reset, and helps confirm that the reset sequence
was attempted on affected systems.

Signed-off-by: Satyansh Shukla <[email protected]>
---
 drivers/mmc/host/sdhci-of-dwcmshc.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/mmc/host/sdhci-of-dwcmshc.c b/drivers/mmc/host/sdhci-of-dwcmshc.c
index b9ecd91f44ad..8acd4652a12b 100644
--- a/drivers/mmc/host/sdhci-of-dwcmshc.c
+++ b/drivers/mmc/host/sdhci-of-dwcmshc.c
@@ -2019,10 +2019,13 @@ static void dwcmshc_bf3_hw_reset(struct sdhci_host *host)
 {
         struct arm_smccc_res res = { 0 };
 
+       pr_info("%s: resetting...\n", __func__);
         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));
+       else
+               pr_info("%s: RST_N succeeded.\n", mmc_hostname(host->mmc));
 }
 
 static const struct sdhci_ops sdhci_dwcmshc_bf3_ops = {
--
2.31.1
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.