[U-Boot] [PATCH 10/10] Convert cmd_usage() calls in common to use a return value

Simon Glass sjg at chromium.org
Wed Dec 7 17:29:53 CET 2011


Hi Igor,

On Wed, Dec 7, 2011 at 4:47 AM, Igor Grinberg <grinberg at compulab.co.il> wrote:
> Hi Simon,
>
> On 12/07/11 07:47, Simon Glass wrote:
>> Change all files in common/ to use CMD_RET_USAGE instead of calling
>> cmd_usage() directly. I'm not completely sure about this patch since
>> the code since impact is small (100 byte or so on ARM) and it might
>> need splitting into smaller patches. But for now here it is.
>>
>> Signed-off-by: Simon Glass <sjg at chromium.org>
>> ---
>
> [...]
>
>> diff --git a/common/cmd_mmc_spi.c b/common/cmd_mmc_spi.c
>> index cfd0fb1..3153610 100644
>> --- a/common/cmd_mmc_spi.c
>> +++ b/common/cmd_mmc_spi.c
>> @@ -78,7 +78,7 @@ static int do_mmc_spi(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
>>       return 0;
>>
>>  usage:
>> -     cmd_usage(cmdtp);
>> +     return CMD_RET_USAGE;
>>       return 1;
>
> You, probably, also want to remove the above line...

Will do, thanks.

Regards,
Simon

>
> [...]
>
>
> --
> Regards,
> Igor.


More information about the U-Boot mailing list