mmc:fix Call mmc_init() when executing mmc_get_dev()
authorŁukasz Majewski <l.majewski@samsung.com>
Thu, 19 Apr 2012 02:39:18 +0000 (02:39 +0000)
committerAndy Fleming <afleming@freescale.com>
Fri, 13 Jul 2012 22:04:50 +0000 (17:04 -0500)
This code adds call to mmc_init(), for partition related commands (e.g.
fatls, fatinfo etc.).

It is safe to call mmc_init() multiple times since mmc->has_init flag
prevents from multiple initialization.

The FAT related code calls get_dev high level method and then uses
elements from mmc->block_dev, which is uninitialized until the mmc_init
(and thereof mmc_startup) is called.

This problem appears on boards, which don't use mmc as the default
place for envs

Signed-off-by: Lukasz Majewski <l.majewski@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
Cc: Andy Fleming <afleming@gmail.com>
Signed-off-by: Andy Fleming <afleming@freescale.com>
drivers/mmc/mmc.c

index 9f746a304051297caf59aeb929bab89384a05689..98e5af3ec97b888ad715d23f65d69ca624ca9a23 100644 (file)
@@ -1306,8 +1306,11 @@ int mmc_register(struct mmc *mmc)
 block_dev_desc_t *mmc_get_dev(int dev)
 {
        struct mmc *mmc = find_mmc_device(dev);
+       if (!mmc)
+               return NULL;
 
-       return mmc ? &mmc->block_dev : NULL;
+       mmc_init(mmc);
+       return &mmc->block_dev;
 }
 #endif