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


Groups > linux.kernel > #1404110 > unrolled thread

linux-next: build warnings after merge of the net tree

Started byStephen Rothwell <sfr@canb.auug.org.au>
First post2016-05-20 05:40 +0200
Last post2016-05-20 08:00 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel


Contents

  linux-next: build warnings after merge of the net tree Stephen Rothwell <sfr@canb.auug.org.au> - 2016-05-20 05:40 +0200
    Re: linux-next: build warnings after merge of the net tree David Miller <davem@davemloft.net> - 2016-05-20 08:00 +0200

#1404110 — linux-next: build warnings after merge of the net tree

FromStephen Rothwell <sfr@canb.auug.org.au>
Date2016-05-20 05:40 +0200
Subjectlinux-next: build warnings after merge of the net tree
Message-ID<rAJs6-gB-7@gated-at.bofh.it>
Hi all,

After merging the net tree, today's linux-next build (arm
multi_v7_defconfig) produced these warnings:

drivers/net/usb/pegasus.c: In function 'read_bulk_callback':
drivers/net/usb/pegasus.c:475:1: warning: label 'goon' defined but not used [-Wunused-label]
 goon:
 ^
drivers/net/usb/pegasus.c:446:8: warning: unused variable 'pkt_len' [-Wunused-variable]
  __u16 pkt_len;
        ^
drivers/net/usb/pegasus.c:445:6: warning: unused variable 'buf' [-Wunused-variable]
  u8 *buf = urb->transfer_buffer;
      ^
drivers/net/usb/pegasus.c:443:17: warning: unused variable 'count' [-Wunused-variable]
  int rx_status, count = urb->actual_length;
                 ^

Introduced by commit

  e00be9e4d0ff ("net: pegasus: remove dead coding")



-- 
Cheers,
Stephen Rothwell

[toc] | [next] | [standalone]


#1404146

FromDavid Miller <davem@davemloft.net>
Date2016-05-20 08:00 +0200
Message-ID<rALDA-1B0-7@gated-at.bofh.it>
In reply to#1404110
From: Stephen Rothwell <sfr@canb.auug.org.au>
Date: Fri, 20 May 2016 13:33:30 +1000

> After merging the net tree, today's linux-next build (arm
> multi_v7_defconfig) produced these warnings:
 ...
> Introduced by commit
> 
>   e00be9e4d0ff ("net: pegasus: remove dead coding")

This change has other problems too and I have just reverted it.

Thanks.

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web