[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