2009-11-30 06:01:27

by Stephen Rothwell

[permalink] [raw]
Subject: linux-next: usb tree build warning

Hi Greg,

Today's linux-next build (x86_64 allmodconfig) produced this warning:

drivers/net/wimax/i2400m/usb.c: In function 'i2400mu_suspend':
drivers/net/wimax/i2400m/usb.c:595: warning: unused variable 'usb_dev'

Introduced by commit a85c653a3945350875611d3cb071a0143cfeb2d2 ("USB:
remove the auto_pm flag") which removed the last usage in this function.

--
Cheers,
Stephen Rothwell [email protected]
http://www.canb.auug.org.au/~sfr/

Attachments:
(No filename) (462.00 B)
 (No filename) (198.00 B)
 Download all attachments

2009-12-01 16:22:57

by Alan Stern

[permalink] [raw]
Subject: Re: linux-next: usb tree build warning

On Mon, 30 Nov 2009, Stephen Rothwell wrote:

> Hi Greg,
>
> Today's linux-next build (x86_64 allmodconfig) produced this warning:
>
> drivers/net/wimax/i2400m/usb.c: In function 'i2400mu_suspend':
> drivers/net/wimax/i2400m/usb.c:595: warning: unused variable 'usb_dev'
>
> Introduced by commit a85c653a3945350875611d3cb071a0143cfeb2d2 ("USB:
> remove the auto_pm flag") which removed the last usage in this function.

Greg, can you fix this? Simply modify the patch to eliminate the
unused local variable and the enclosing #ifdef.

Thanks,

Alan Stern

2009-12-08 01:05:25

by Greg KH

[permalink] [raw]
Subject: Re: linux-next: usb tree build warning

On Tue, Dec 01, 2009 at 11:23:02AM -0500, Alan Stern wrote:
> On Mon, 30 Nov 2009, Stephen Rothwell wrote:
>
> > Hi Greg,
> >
> > Today's linux-next build (x86_64 allmodconfig) produced this warning:
> >
> > drivers/net/wimax/i2400m/usb.c: In function 'i2400mu_suspend':
> > drivers/net/wimax/i2400m/usb.c:595: warning: unused variable 'usb_dev'
> >
> > Introduced by commit a85c653a3945350875611d3cb071a0143cfeb2d2 ("USB:
> > remove the auto_pm flag") which removed the last usage in this function.
>
> Greg, can you fix this? Simply modify the patch to eliminate the
> unused local variable and the enclosing #ifdef.

Now fixed up.

thanks,

greg k-h