Re: [PATCH] MIPS: AR7: Fix GCC 4.6.0 build error.

[Jonas Gorski <jonas.gorski+openwrt@gmail.com>]

On 13 May 2011 17:28, Alexander Clouter <alex@digriz.org.uk> wrote:
>  CC      arch/mips/ar7/gpio.o
> arch/mips/ar7/gpio.c: In function 'ar7_gpio_init':
> arch/mips/ar7/gpio.c:318:11: error: variable 'size' set but not used 
> [-Werror=unused-but-set-variable]
> cc1: all warnings being treated as errors
>
> Signed-off-by: Alexander Clouter <alex@digriz.org.uk>
> ---
>  arch/mips/ar7/gpio.c |   12 ++----------
>  1 files changed, 2 insertions(+), 10 deletions(-)
>
> diff --git a/arch/mips/ar7/gpio.c b/arch/mips/ar7/gpio.c
> index 425dfa5..6917427 100644
> --- a/arch/mips/ar7/gpio.c
> +++ b/arch/mips/ar7/gpio.c
> @@ -314,16 +314,8 @@ static void titan_gpio_init(void)
>  int __init ar7_gpio_init(void)
>  {
>        int ret;
> -       struct ar7_gpio_chip *gpch;
> -       unsigned size;
> -
> -       if (!ar7_is_titan()) {
> -               gpch = &ar7_gpio_chip;
> -               size = 0x10;
> -       } else {
> -               gpch = &titan_gpio_chip;
> -               size = 0x1f;
> -       }
> +       struct ar7_gpio_chip *gpch = (!ar7_is_titan())
> +               ? &ar7_gpio_chip : &titan_gpio_chip;
>
>        gpch->regs = ioremap_nocache(AR7_REGS_GPIO,
>                                        AR7_REGS_GPIO + 0x10);

Without any AR7 knowledge, it looks like the size is supposed to be
used here instead of the 0x10 - also the "AR7_REGS_GPIO + 0x10" looks
wrong - I don't think the regs are 0x8610910 bytes big ;-).

Regards
Jonas