From mboxrd@z Thu Jan  1 00:00:00 1970
From: Marek Vasut <marek.vasut@gmail.com>
Date: Mon, 21 Nov 2011 23:39:49 +0100
Subject: [U-Boot] [PATCH 9/9] ARM: xm250: Fix GCC 4.6 warnings
In-Reply-To: <1321703956-4224-10-git-send-email-agust@denx.de>
References: <1321703956-4224-1-git-send-email-agust@denx.de>
	<1321703956-4224-10-git-send-email-agust@denx.de>
Message-ID: <201111212339.50052.marek.vasut@gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

> Fix:
> flash.c: In function 'flash_erase':
> flash.c:252:6: warning: variable 'flag' set but not used
> [-Wunused-but-set-variable]
> flash.c: In function 'write_data':
> flash.c:421:6: warning: variable 'flag' set but not used
> [-Wunused-but-set-variable]
> 
> Signed-off-by: Anatolij Gustschin <agust@denx.de>
> ---
>  board/xm250/flash.c |    7 +++----
>  1 files changed, 3 insertions(+), 4 deletions(-)
> 
> diff --git a/board/xm250/flash.c b/board/xm250/flash.c
> index e825aba..b953502 100644
> --- a/board/xm250/flash.c
> +++ b/board/xm250/flash.c
> @@ -249,7 +249,7 @@ static ulong flash_get_size (FPW *addr, flash_info_t
> *info)
> 
>  int flash_erase (flash_info_t *info, int s_first, int s_last)
>  {
> -	int flag, prot, sect;
> +	int prot, sect;
>  	ulong type, start;
>  	int rcode = 0;
> 
> @@ -284,7 +284,7 @@ int flash_erase (flash_info_t *info, int s_first, int
> s_last) }
> 
>  	/* Disable interrupts which might cause a timeout here */
> -	flag = disable_interrupts ();
> +	disable_interrupts();
> 
>  	/* Start erase on unprotected sectors */
>  	for (sect = s_first; sect <= s_last; sect++) {
> @@ -418,7 +418,6 @@ static int write_data (flash_info_t *info, ulong dest,
> FPW data) {
>  	FPWV *addr = (FPWV *) dest;
>  	ulong status;
> -	int flag;
>  	ulong start;
> 
>  	/* Check if Flash is (sufficiently) erased */
> @@ -427,7 +426,7 @@ static int write_data (flash_info_t *info, ulong dest,
> FPW data) return (2);
>  	}
>  	/* Disable interrupts which might cause a timeout here */
> -	flag = disable_interrupts ();
> +	disable_interrupts();
> 
>  	*addr = (FPW) 0x00400040;	/* write setup */
>  	*addr = data;

This one might as well be removed too ? No maintainer in MAINTAINERS so I guess 
it can.