]> git.dujemihanovic.xyz Git - u-boot.git/commitdiff
mmc: mmc-uclass: use the fixed devnum with alias node
authorJaehoon Chung <jh80.chung@samsung.com>
Thu, 2 Feb 2017 04:41:14 +0000 (13:41 +0900)
committerJaehoon Chung <jh80.chung@samsung.com>
Thu, 9 Feb 2017 11:37:05 +0000 (20:37 +0900)
If there are alias nodes as "mmc", use the devnum as alias index
number.
This patch is for fixing a problem of Exynos4 series.
Problem is the below thing.

Current legacy mode:
EXYNOS DWMMC: 0, SAMSUNG SDHCI: 1

After using DM:
SAMSUNG SDHCI: 0, EXYNOS DWMMC: 1

Dev index is swapped.
Then u-boot can't find the kernel image..because it is already set to 0 as mmcdev.
If change from legacy to DM, also needs to touch all exynos4 config file.
For using simply, just supporting the fixed devnum with alias node is better than it.

Usage:
alaise {
....
mmc0 = &sdhci2; /* eMMC */
mmc1 = &sdhci1; /* SD */
...
}

Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
drivers/mmc/mmc-uclass.c

index 2fe5d61e263761ba71e1d5f0917eccbbc22805f4..de7acb682629a2bd40a3e97f5b03d3d370da60b9 100644 (file)
@@ -13,6 +13,8 @@
 #include <dm/root.h>
 #include "mmc_private.h"
 
+DECLARE_GLOBAL_DATA_PTR;
+
 #ifdef CONFIG_DM_MMC_OPS
 int dm_mmc_send_cmd(struct udevice *dev, struct mmc_cmd *cmd,
                    struct mmc_data *data)
@@ -192,10 +194,15 @@ int mmc_bind(struct udevice *dev, struct mmc *mmc, const struct mmc_config *cfg)
 {
        struct blk_desc *bdesc;
        struct udevice *bdev;
-       int ret;
+       int ret, devnum = -1;
+
+#ifndef CONFIG_SPL_BUILD
+       /* Use the fixed index with aliase node's index */
+       fdtdec_get_alias_seq(gd->fdt_blob, "mmc", dev->of_offset, &devnum);
+#endif
 
-       ret = blk_create_devicef(dev, "mmc_blk", "blk", IF_TYPE_MMC, -1, 512,
-                                0, &bdev);
+       ret = blk_create_devicef(dev, "mmc_blk", "blk", IF_TYPE_MMC,
+                       devnum, 512, 0, &bdev);
        if (ret) {
                debug("Cannot create block device\n");
                return ret;