| Message ID | 20201102114512.1062724-31-lee.jones@linaro.org |
|---|---|
| State | Changes Requested |
| Delegated to: | David Miller |
| Headers | show |
| Series | [01/30] net: fddi: skfp: ecm: Protect 'if' when AIX_EVENT is not defined | expand |
| Context | Check | Description |
|---|---|---|
| jkicinski/cover_letter | warning | Series does not have a cover letter |
| jkicinski/fixes_present | success | Link |
| jkicinski/patch_count | fail | Series longer than 15 patches |
| jkicinski/tree_selection | success | Guessed tree name to be net-next |
| jkicinski/subject_prefix | success | Link |
| jkicinski/source_inline | success | Was 0 now: 0 |
| jkicinski/verify_signedoff | success | Link |
| jkicinski/module_param | success | Was 0 now: 0 |
| jkicinski/build_32bit | success | Errors and warnings before: 0 this patch: 0 |
| jkicinski/kdoc | success | Errors and warnings before: 6 this patch: 6 |
| jkicinski/verify_fixes | success | Link |
| jkicinski/checkpatch | success | total: 0 errors, 0 warnings, 0 checks, 8 lines checked |
| jkicinski/build_allmodconfig_warn | success | Errors and warnings before: 0 this patch: 0 |
| jkicinski/header_inline | success | Link |
| jkicinski/stable | success | Stable not CCed |
On Mon, 02 Nov 2020, Lee Jones wrote: > 'pkt_len' is used to interact with a hardware register. It might not > be safe to remove it entirely. Mark it as __maybe_unused instead. > > Fixes the following W=1 kernel build warning(s): > > drivers/net/ethernet/smsc/smc91x.c: In function ‘smc_tx’: > drivers/net/ethernet/smsc/smc91x.c:706:51: warning: variable ‘pkt_len’ set but not used [-Wunused-but-set-variable] > > Cc: Nicolas Pitre <nico@fluxnic.net> > Cc: "David S. Miller" <davem@davemloft.net> > Cc: Jakub Kicinski <kuba@kernel.org> > Cc: Erik Stahlman <erik@vt.edu> > Cc: Peter Cammaert <pc@denkart.be> > Cc: Daris A Nevil <dnevil@snmc.com> > Cc: Russell King <rmk@arm.linux.org.uk> > Cc: netdev@vger.kernel.org > Signed-off-by: Lee Jones <lee.jones@linaro.org> > --- > drivers/net/ethernet/smsc/smc91x.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/ethernet/smsc/smc91x.c b/drivers/net/ethernet/smsc/smc91x.c > index b5d053292e717..a3f37b1f86491 100644 > --- a/drivers/net/ethernet/smsc/smc91x.c > +++ b/drivers/net/ethernet/smsc/smc91x.c > @@ -703,7 +703,7 @@ static void smc_tx(struct net_device *dev) > { > struct smc_local *lp = netdev_priv(dev); > void __iomem *ioaddr = lp->base; > - unsigned int saved_packet, packet_no, tx_status, pkt_len; > + unsigned int saved_packet, packet_no, tx_status, __maybe_unused pkt_len; > > DBG(3, dev, "%s\n", __func__); This one is still lingering. Looks like it's still relevant.
> This one is still lingering. Looks like it's still relevant.
Hi Lee
It is part of this patchset:
https://www.mail-archive.com/netdev@vger.kernel.org/msg368589.html
Andrew
diff --git a/drivers/net/ethernet/smsc/smc91x.c b/drivers/net/ethernet/smsc/smc91x.c index b5d053292e717..a3f37b1f86491 100644 --- a/drivers/net/ethernet/smsc/smc91x.c +++ b/drivers/net/ethernet/smsc/smc91x.c @@ -703,7 +703,7 @@ static void smc_tx(struct net_device *dev) { struct smc_local *lp = netdev_priv(dev); void __iomem *ioaddr = lp->base; - unsigned int saved_packet, packet_no, tx_status, pkt_len; + unsigned int saved_packet, packet_no, tx_status, __maybe_unused pkt_len; DBG(3, dev, "%s\n", __func__);
'pkt_len' is used to interact with a hardware register. It might not be safe to remove it entirely. Mark it as __maybe_unused instead. Fixes the following W=1 kernel build warning(s): drivers/net/ethernet/smsc/smc91x.c: In function ‘smc_tx’: drivers/net/ethernet/smsc/smc91x.c:706:51: warning: variable ‘pkt_len’ set but not used [-Wunused-but-set-variable] Cc: Nicolas Pitre <nico@fluxnic.net> Cc: "David S. Miller" <davem@davemloft.net> Cc: Jakub Kicinski <kuba@kernel.org> Cc: Erik Stahlman <erik@vt.edu> Cc: Peter Cammaert <pc@denkart.be> Cc: Daris A Nevil <dnevil@snmc.com> Cc: Russell King <rmk@arm.linux.org.uk> Cc: netdev@vger.kernel.org Signed-off-by: Lee Jones <lee.jones@linaro.org> --- drivers/net/ethernet/smsc/smc91x.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)