mmc: am654_sdhci: Drop a redundant power_domain_on in probe
authorLokesh Vutla <lokeshvutla@ti.com>
Tue, 24 Sep 2019 07:47:16 +0000 (13:17 +0530)
committerTom Rini <trini@konsulko.com>
Fri, 11 Oct 2019 17:31:18 +0000 (13:31 -0400)
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>
Reviewed-by: Peng Fan <peng.fan@nxp.com>
drivers/mmc/am654_sdhci.c

index 1793a3f99a5750858ffee39d1a2fb5bc516a978b..7cd5516197cd2394bbb2a8f52f500a5608923021 100644 (file)
@@ -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");