lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  PHC 
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:   Fri, 3 Nov 2017 10:11:21 +0100
From:   Arnd Bergmann <>
To:     Linus Walleij <>
Cc:     Jason Cooper <>, Andrew Lunn <>,
        Gregory Clement <>,
        Sebastian Hesselbarth <>,
        Ken Ma <>, Thierry Reding <>,
        "" <>,
        "" <>
Subject: Re: [PATCH] pinctrl: armada-37xx: remove unused variable

On Fri, Nov 3, 2017 at 8:46 AM, Linus Walleij <> wrote:
> On Thu, Nov 2, 2017 at 3:29 PM, Arnd Bergmann <> 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 <>
> 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.


Powered by blists - more mailing lists