diff options
| author | Stephen Warren <swarren@nvidia.com> | 2014-05-23 12:47:06 -0600 |
|---|---|---|
| committer | Pantelis Antoniou <panto@antoniou-consulting.com> | 2014-06-12 14:15:59 +0300 |
| commit | d4622df34280830cfe0678f098d3d9f62e6b5d94 (patch) | |
| tree | 0f8331826fab5b370cd0ca2604635236f85d0a5d /drivers/mmc/mmc.c | |
| parent | e153b13c8e300236e6d505bea629cc81fd0988cb (diff) | |
| download | u-boot-imx-d4622df34280830cfe0678f098d3d9f62e6b5d94.zip u-boot-imx-d4622df34280830cfe0678f098d3d9f62e6b5d94.tar.gz u-boot-imx-d4622df34280830cfe0678f098d3d9f62e6b5d94.tar.bz2 | |
mmc: return meaningful error codes from mmc_select_hwpart
Rather than just returning -1 everywhere, try to return something
meaningful from mmc_select_hwpart(). Note that most other MMC functions
don't do this, including functions called from mmc_select_hwpart(), so
I'm not sure how effective this will be. Still, it's one less place with
hard-coded -1.
Suggested-by: Pantelis Antoniou <panto@antoniou-consulting.com>
Signed-off-by: Stephen Warren <swarren@nvidia.com>
Acked-by: Pantelis Antoniou <panto@antoniou-consulting.com>
Diffstat (limited to 'drivers/mmc/mmc.c')
| -rw-r--r-- | drivers/mmc/mmc.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/drivers/mmc/mmc.c b/drivers/mmc/mmc.c index 55c2c68..b5477b1 100644 --- a/drivers/mmc/mmc.c +++ b/drivers/mmc/mmc.c @@ -10,6 +10,7 @@ #include <config.h> #include <common.h> #include <command.h> +#include <errno.h> #include <mmc.h> #include <part.h> #include <malloc.h> @@ -564,19 +565,19 @@ int mmc_select_hwpart(int dev_num, int hwpart) int ret; if (!mmc) - return -1; + return -ENODEV; if (mmc->part_num == hwpart) return 0; if (mmc->part_config == MMCPART_NOAVAILABLE) { printf("Card doesn't support part_switch\n"); - return -1; + return -EMEDIUMTYPE; } ret = mmc_switch_part(dev_num, hwpart); if (ret) - return -1; + return ret; mmc->part_num = hwpart; |