From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [PATCH net-next] net: ethernet: dpaa: remove unused variables Date: Wed, 26 Sep 2018 10:35:06 -0700 (PDT) Message-ID: <20180926.103506.950692305206471144.davem@davemloft.net> References: <20180926131225.2661703-1-arnd@arndb.de> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: madalin.bucur@nxp.com, andrew@lunn.ch, f.fainelli@gmail.com, yangbo.lu@nxp.com, mail@jakemoroni.com, gustavo@embeddedor.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org To: arnd@arndb.de Return-path: In-Reply-To: <20180926131225.2661703-1-arnd@arndb.de> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org From: Arnd Bergmann Date: Wed, 26 Sep 2018 15:12:13 +0200 > The patch that removed the only users of the oldadv/newadv variables > accidentally left the now-unused declarations behind: > > drivers/net/ethernet/freescale/dpaa/dpaa_ethtool.c: In function 'dpaa_set_pauseparam': > drivers/net/ethernet/freescale/dpaa/dpaa_ethtool.c:185:14: error: unused variable 'oldadv' [-Werror=unused-variable] > drivers/net/ethernet/freescale/dpaa/dpaa_ethtool.c:185:6: error: unused variable 'newadv' [-Werror=unused-variable] > > Fixes: 70814e819c11 ("net: ethernet: Add helper for set_pauseparam for Asym Pause") > Signed-off-by: Arnd Bergmann Applied.