[U-Boot] [U-boot] [Patch] keystone2: replace printf on puts where it's possible

Ivan Khoronzhuk ivan.khoronzhuk at ti.com
Tue Nov 4 18:05:55 CET 2014


On 11/04/2014 06:46 PM, Tom Rini wrote:
> On Tue, Nov 04, 2014 at 01:44:53AM +0200, Ivan Khoronzhuk wrote:
>
>> It is better to use simple puts() function instead of printf()
>> when it's possible. Also remove redundant sprintf().
>>
>> Signed-off-by: Ivan Khoronzhuk <ivan.khoronzhuk at ti.com>
> One of the outcomes of the mini-summit and I think was summarized on the
> list as well was that we're going to stop with the "use puts instead of
> printf!" thing as it ends up being more annoying to contributors than
> useful in terms of functionality and code size.
>
>> diff --git a/board/ti/ks2_evm/board.c b/board/ti/ks2_evm/board.c
>> index 4029493..ff7bc4b 100644
>> --- a/board/ti/ks2_evm/board.c
>> +++ b/board/ti/ks2_evm/board.c
>> @@ -122,7 +122,6 @@ void ft_board_setup(void *blob, bd_t *bd)
>>   	int nbanks;
>>   	u64 size[2];
>>   	u64 start[2];
>> -	char name[32];
>>   	int nodeoffset;
>>   	u32 ddr3a_size;
>>   	int unitrd_fixup = 0;
>> @@ -158,15 +157,13 @@ void ft_board_setup(void *blob, bd_t *bd)
>>   	}
>>   
>>   	/* reserve memory at start of bank */
>> -	sprintf(name, "mem_reserve_head");
>> -	env = getenv(name);
>> +	env = getenv("mem_reserve_head");
>>   	if (env) {
>>   		start[0] += ustrtoul(env, &endp, 0);
>>   		size[0] -= ustrtoul(env, &endp, 0);
>>   	}
>>   
>> -	sprintf(name, "mem_reserve");
>> -	env = getenv(name);
>> +	env = getenv("mem_reserve");
>>   	if (env)
>>   		size[0] -= ustrtoul(env, &endp, 0);
> This is good to fix however.
>

Ok,
Reject this patch.
I will send new patch that delete only sprintf.
Thanks!

-- 
Regards,
Ivan Khoronzhuk



More information about the U-Boot mailing list