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

[Tom Rini]

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.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20141104/74beee07/attachment.pgp>