Re: [PATCH 1/5 v2]wireless:hostap_main.c warning: variable 'iface'set but not used

[Justin P. Mattock]

On 06/22/2010 11:13 AM, John W. Linville wrote:
> On Mon, Jun 21, 2010 at 03:02:13PM -0700, Justin P. Mattock wrote:
> > This is a resend from version one due to trying a different approach
> > than the original(probably important to leave netdev_priv() in).
> >
> > In any case have a look, if there's another approach let me know
> > and ill test it out. The below patch fixes a warning im seeing
> > when compiling with gcc 4.6.0
> >
> > CC [M]  drivers/net/wireless/hostap/hostap_main.o
> > drivers/net/wireless/hostap/hostap_main.c: In function 'hostap_set_multicast_list_queue':
> > drivers/net/wireless/hostap/hostap_main.c:744:27: warning: variable 'iface' set but not used
> >   Signed-off-by: Justin P. Mattock<justinmattock@xxxxxxxxx>
>
> I already applied the other version to wireless-next-2.6.  I can't
> imagine what you mean to accomplish by leaving in a call to netdev_priv
> w/o assigning the result to something.
>
> John

alright..
as for the netdev_priv, I was getting confused on this one.

Thanks for taking the time to look at this.

Justin P. Mattock
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/