On 6/22/22 4:30 PM, Sergiu Moga wrote:

Whenever the SD Card would be removed and then re-inserted while in the U-Boot command line, the `SDBPWR` bit of the `SDMMC_PCR` register would remain unset afterwards. In order for the bit to be set again after re-insertion, register an additional `deferred_probe` method that the DM would then transparently call. This method will call the generic `sdhci_probe` which will, during its execution flow, set this bit to 1.

Signed-off-by: Sergiu Moga sergiu.moga@microchip.com Reported-by: Mihai Sain mihai.sain@microchip.com Reviewed-by: Eugen Hristev eugen.hristev@microchip.com

---

Hi Peng,

To take this patch into my tree I would be happy if you had a look or reviewed it first.

Thanks, Eugen

drivers/mmc/atmel_sdhci.c | 12 ++++++++++++ 1 file changed, 12 insertions(+)

diff --git a/drivers/mmc/atmel_sdhci.c b/drivers/mmc/atmel_sdhci.c index 2b5ceeab94..37b0beeed4 100644 --- a/drivers/mmc/atmel_sdhci.c +++ b/drivers/mmc/atmel_sdhci.c @@ -52,6 +52,17 @@ struct atmel_sdhci_plat { struct mmc mmc; };

+static int atmel_sdhci_deferred_probe(struct sdhci_host *host) +{

• struct udevice *dev = host->mmc->dev;
• return sdhci_probe(dev);

+}

+static const struct sdhci_ops atmel_sdhci_ops = {

• .deferred_probe = atmel_sdhci_deferred_probe,

+};

• static int atmel_sdhci_probe(struct udevice *dev) { struct mmc_uclass_priv *upriv = dev_get_uclass_priv(dev);

@@ -104,6 +115,7 @@ static int atmel_sdhci_probe(struct udevice *dev) return ret;

host->mmc->priv = host;

• host->ops = &atmel_sdhci_ops; upriv->mmc = host->mmc;

  clk_free(&clk);