On Fri, Nov 3, 2017 at 8:46 AM, Linus Walleij <linus.walleij@xxxxxxxxxx> wrote: > On Thu, Nov 2, 2017 at 3:29 PM, Arnd Bergmann <arnd@xxxxxxxx> wrote: > >> A cleanup left behind a temporary variable that is now unused: >> >> drivers/pinctrl/mvebu/pinctrl-armada-37xx.c: In function 'armada_37xx_irq_startup': >> drivers/pinctrl/mvebu/pinctrl-armada-37xx.c:693:20: error: unused variable 'chip' [-Werror=unused-variable] >> >> This removes the declarations as well. >> >> Fixes: 3ee9e605caea ("pinctrl: armada-37xx: Stop using struct gpio_chip.irq_base") >> Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx> > > It is used on the head of development, so it's fixed in -next. > > Is it such a big issue for v4.14 that you think I should send it > to Torvalds as a fix at this point or can I just leave it? I'm confused. The build warning showed up in linux-next yesterday after 3ee9e605caea got merged, which removes the user of that variable. In v4.14 it is still used. Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html