Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1653020 > unrolled thread

[PATCH] gpiolib: remove unused variable

Started byArnd Bergmann <arnd@arndb.de>
First post2017-05-30 11:30 +0200
Last post2017-05-31 02:00 +0200
Articles 3 — 3 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] gpiolib: remove unused variable Arnd Bergmann <arnd@arndb.de> - 2017-05-30 11:30 +0200
    Re: [PATCH] gpiolib: remove unused variable Andy Shevchenko <andriy.shevchenko@linux.intel.com> - 2017-05-30 12:10 +0200
    Re: [PATCH] gpiolib: remove unused variable Linus Walleij <linus.walleij@linaro.org> - 2017-05-31 02:00 +0200

#1653020 — [PATCH] gpiolib: remove unused variable

FromArnd Bergmann <arnd@arndb.de>
Date2017-05-30 11:30 +0200
Subject[PATCH] gpiolib: remove unused variable
Message-ID<tMLDs-5ux-19@gated-at.bofh.it>
This was left behind by a cleanup patch:

drivers/gpio/gpiolib.c: In function 'gpiochip_irqchip_init_valid_mask':
drivers/gpio/gpiolib.c:1474:6: error: unused variable 'i' [-Werror=unused-variable]

Fixes: 923a654c186c ("gpiolib: Re-use bitmap_fill() instead of open coded loop")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
 drivers/gpio/gpiolib.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/gpio/gpiolib.c b/drivers/gpio/gpiolib.c
index be8097097326..62ffb4e293d2 100644
--- a/drivers/gpio/gpiolib.c
+++ b/drivers/gpio/gpiolib.c
@@ -1471,8 +1471,6 @@ static struct gpio_chip *find_chip_by_name(const char *name)
 
 static int gpiochip_irqchip_init_valid_mask(struct gpio_chip *gpiochip)
 {
-	int i;
-
 	if (!gpiochip->irq_need_valid_mask)
 		return 0;
 
-- 
2.9.0

[toc] | [next] | [standalone]


#1653067

FromAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Date2017-05-30 12:10 +0200
Message-ID<tMMga-5Z0-11@gated-at.bofh.it>
In reply to#1653020
On Tue, 2017-05-30 at 11:21 +0200, Arnd Bergmann wrote:
> This was left behind by a cleanup patch:
> 
> drivers/gpio/gpiolib.c: In function
> 'gpiochip_irqchip_init_valid_mask':
> drivers/gpio/gpiolib.c:1474:6: error: unused variable 'i' [-
> Werror=unused-variable]


Right, thanks for it!
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

> 
> Fixes: 923a654c186c ("gpiolib: Re-use bitmap_fill() instead of open
> coded loop")
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> ---
>  drivers/gpio/gpiolib.c | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/drivers/gpio/gpiolib.c b/drivers/gpio/gpiolib.c
> index be8097097326..62ffb4e293d2 100644
> --- a/drivers/gpio/gpiolib.c
> +++ b/drivers/gpio/gpiolib.c
> @@ -1471,8 +1471,6 @@ static struct gpio_chip *find_chip_by_name(const
> char *name)
>  
>  static int gpiochip_irqchip_init_valid_mask(struct gpio_chip
> *gpiochip)
>  {
> -	int i;
> -
>  	if (!gpiochip->irq_need_valid_mask)
>  		return 0;
>  

-- 
Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Intel Finland Oy

[toc] | [prev] | [next] | [standalone]


#1653725

FromLinus Walleij <linus.walleij@linaro.org>
Date2017-05-31 02:00 +0200
Message-ID<tMZdn-5ry-5@gated-at.bofh.it>
In reply to#1653020
On Tue, May 30, 2017 at 11:21 AM, Arnd Bergmann <arnd@arndb.de> wrote:

> This was left behind by a cleanup patch:
>
> drivers/gpio/gpiolib.c: In function 'gpiochip_irqchip_init_valid_mask':
> drivers/gpio/gpiolib.c:1474:6: error: unused variable 'i' [-Werror=unused-variable]
>
> Fixes: 923a654c186c ("gpiolib: Re-use bitmap_fill() instead of open coded loop")
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>

Patch applied.

Yours,
Linus Walleij

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web