[U-Boot] [PATCH] CRITICAL error in v2013.04-rc2 (true and false not working)

York Sun yorksun at freescale.com
Thu Apr 4 23:05:18 CEST 2013


On 04/04/2013 01:33 PM, Wolfgang Denk wrote:
> Dear Pali Rohár,
> 
> In message <201304041946.55455 at pali> you wrote:
>>
>> u-boot "true" and "false" hush commands are copiled incorrectly in uboot v2013.04-rc2 
>> (c8142633e169665b246352918df5b76fd243bb71)
>>
>> when I run help in u-boot it show:
>>
>> # help
>> 0       - do nothing, unsuccessfully
>> 1       - do nothing, successfully
>> ?       - alias for 'help'
>> base    - print or set address offset
>> ...
>>
>> Instead "0" and "1" there should be "false" and "true" command.
> 
> I confirm the problem.
> 
> York, your commit 472d546 "Consolidate bool type" is causing this
> issue, so I suggest you send a fix, please.
> 
> Please check for other, similar use cases.
> 
>

I found the cause but I don't know the best fix. Please help.

This is cause by common/cmd_test.c. It has command "true" and "false".
Shall I undef true and false before the U_BOOT_CMD in this file, or use
True and False as the names? If there is a better way to do that, please
enlighten me.

York




More information about the U-Boot mailing list