]> git.dujemihanovic.xyz Git - u-boot.git/commitdiff
mmc: xenon_sdhci: Add missing common host capabilities
authorAndre Heider <a.heider@gmail.com>
Thu, 10 Sep 2020 17:53:40 +0000 (19:53 +0200)
committerPeng Fan <peng.fan@nxp.com>
Tue, 15 Sep 2020 02:15:56 +0000 (10:15 +0800)
Use mmc_of_parse() to set the common host properties. That includes
"bus-width", so parsing it can be removed from the driver.

But more importantly, "non-removable" is now respected, which fixes
the usage of eMMC.

Signed-off-by: Andre Heider <a.heider@gmail.com>
Reviewed-by: Konstantin Porotchkin <kostap@marvell.com>
Tested-by: Marek BehĂșn <marek.behun@nic.cz>
drivers/mmc/xenon_sdhci.c

index 7f9a579c838bb9950d8147d54f905848876a0f38..6ce9d00d0ae7db3ac59c9744589033798aa1abbb 100644 (file)
@@ -485,20 +485,10 @@ static int xenon_sdhci_probe(struct udevice *dev)
                armada_3700_soc_pad_voltage_set(host);
 
        host->host_caps = MMC_MODE_HS | MMC_MODE_HS_52MHz | MMC_MODE_DDR_52MHz;
-       switch (fdtdec_get_int(gd->fdt_blob, dev_of_offset(dev), "bus-width",
-               1)) {
-       case 8:
-               host->host_caps |= MMC_MODE_8BIT;
-               break;
-       case 4:
-               host->host_caps |= MMC_MODE_4BIT;
-               break;
-       case 1:
-               break;
-       default:
-               printf("Invalid \"bus-width\" value\n");
-               return -EINVAL;
-       }
+
+       ret = mmc_of_parse(dev, &plat->cfg);
+       if (ret)
+               return ret;
 
        host->ops = &xenon_sdhci_ops;