Hi Stephen,
On May 23, 2014, at 9:47 PM, Stephen Warren wrote:
> From: Stephen Warren <swar...@nvidia.com>
>
> 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 <pa...@antoniou-consulting.com>
> Signed-off-by: Stephen Warren <swar...@nvidia.com>
> ---
> drivers/mmc/mmc.c | 7 ++++---
> 1 file changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/mmc/mmc.c b/drivers/mmc/mmc.c
> index 8b53ead98f80..221a293797e9 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>
> @@ -558,19 +559,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;
>
> --
> 1.8.1.5
>
This does indeed look good.
Thanks
-- Pantelis
Acked-by: Pantelis Antoniou <pa...@antoniou-consulting.com>
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
