Subject: [PATCH] mmc: am654_sdhci: Drop a redundant power_domain_on in probe

Power-domain is enabled by default in device_probe. am654 mmc driver is enabling power-domain again in probe. As the second call is redundant, drop power_domain_on from probe.

Tested-by: Suman Anna s-anna@ti.com Signed-off-by: Lokesh Vutla lokeshvutla@ti.com

---

drivers/mmc/am654_sdhci.c | 13 ------------- 1 file changed, 13 deletions(-)

diff --git a/drivers/mmc/am654_sdhci.c b/drivers/mmc/am654_sdhci.c index 1793a3f99a..7cd5516197 100644 --- a/drivers/mmc/am654_sdhci.c +++ b/drivers/mmc/am654_sdhci.c @@ -219,23 +219,10 @@ static int am654_sdhci_probe(struct udevice *dev) struct mmc_uclass_priv *upriv = dev_get_uclass_priv(dev); struct sdhci_host *host = dev_get_priv(dev); struct mmc_config *cfg = &plat->cfg;

• struct power_domain sdhci_pwrdmn; struct clk clk; unsigned long clock; int ret;

• ret = power_domain_get_by_index(dev, &sdhci_pwrdmn, 0);

• if (!ret) {

• ret = power_domain_on(&sdhci_pwrdmn);
  

• if (ret) {
  

• 	dev_err(dev, "Power domain on failed (%d)\n", ret);
  

• 	return ret;
  

• }
  

• } else if (ret != -ENOENT && ret != -ENODEV && ret != -ENOSYS) {

• dev_err(dev, "failed to get power domain (%d)\n", ret);
  

• return ret;
  

• }

• ret = clk_get_by_index(dev, 0, &clk); if (ret) { dev_err(dev, "failed to get clock\n");

Reviewed-by: Peng Fan peng.fan@nxp.com

-- 2.22.0