Hi Stephen,
On May 23, 2014, at 10:24 PM, Stephen Warren wrote:
> From: Stephen Warren <swar...@nvidia.com>
>
> The body of init_mmc_device() is now identical to that of do_mmc_rescan()
> except for the error codes returned. Modify do_mmc_rescan() to simply
> call init_mmc_device() and convert the error codes, to avoid code
> duplication.
>
> Signed-off-by: Stephen Warren <swar...@nvidia.com>
> ---
> common/cmd_mmc.c | 10 ++--------
> 1 file changed, 2 insertions(+), 8 deletions(-)
>
> diff --git a/common/cmd_mmc.c b/common/cmd_mmc.c
> index 6741ebee3bca..6c8db2e78c4f 100644
> --- a/common/cmd_mmc.c
> +++ b/common/cmd_mmc.c
> @@ -371,16 +371,10 @@ static int do_mmc_rescan(cmd_tbl_t *cmdtp, int flag,
> {
> struct mmc *mmc;
>
> - mmc = find_mmc_device(curr_device);
> - if (!mmc) {
> - printf("no mmc device at slot %x\n", curr_device);
> + mmc = init_mmc_device(curr_device, true);
> + if (!mmc)
> return CMD_RET_FAILURE;
> - }
> -
> - mmc->has_init = 0;
>
> - if (mmc_init(mmc))
> - return CMD_RET_FAILURE;
> return CMD_RET_SUCCESS;
> }
> static int do_mmc_part(cmd_tbl_t *cmdtp, int flag,
> --
> 1.8.1.5
>
Looks good.
Thanks, applied.
-- 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
