[U-Boot] [PATCH] arch/powerpc/cpu/mpc8xxx/ddr/interactive.c: Fix GCC 4.6 build warning

Marek Vasut marek.vasut at gmail.com
Thu Nov 24 18:16:32 CET 2011


> Fix:
> 
> interactive.c: In function 'fsl_ddr_interactive':
> interactive.c:1357:15: warning: variable 'len' set but not used
> [-Wunused-but-set-variable]
> 
> Signed-off-by: Kumar Gala <galak at kernel.crashing.org>
> ---
>  arch/powerpc/cpu/mpc8xxx/ddr/interactive.c |    3 +--
>  1 files changed, 1 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/powerpc/cpu/mpc8xxx/ddr/interactive.c
> b/arch/powerpc/cpu/mpc8xxx/ddr/interactive.c index d7d66ef..5b72437 100644
> --- a/arch/powerpc/cpu/mpc8xxx/ddr/interactive.c
> +++ b/arch/powerpc/cpu/mpc8xxx/ddr/interactive.c
> @@ -1354,7 +1354,6 @@ unsigned long long fsl_ddr_interactive(fsl_ddr_info_t
> *pinfo) {
>  	unsigned long long ddrsize;
>  	const char *prompt = "FSL DDR>";
> -	unsigned int len;
>  	char buffer[CONFIG_SYS_CBSIZE];
>  	char *argv[CONFIG_SYS_MAXARGS + 1];	/* NULL terminated */
>  	int argc;
> @@ -1389,7 +1388,7 @@ unsigned long long fsl_ddr_interactive(fsl_ddr_info_t
> *pinfo) * No need to worry for buffer overflow here in
>  		 * this function;  readline() maxes out at CFG_CBSIZE
>  		 */
> -		len = readline_into_buffer(prompt,  buffer);
> +		readline_into_buffer(prompt,  buffer);
>  		argc = parse_line(buffer, argv);
>  		if (argc == 0)
>  			continue;

Ok, I'll pick this one.


More information about the U-Boot mailing list