On Mon, Feb 14, 2011 at 10:28:02PM -0600, Larry Finger wrote: > If any of the p54-based drivers are built with CONFIG_P54_LEDS not > defined, the following warning is generated: > > CC [M] drivers/net/wireless/p54/main.o > drivers/net/wireless/p54/main.c: In function âp54_register_commonâ: > drivers/net/wireless/p54/main.c:614:21: warning: unused variable âprivâ > > Signed-off-by: Larry Finger <Larry.Finger@xxxxxxxxxxxx> > --- > > > Index: wireless-testing/drivers/net/wireless/p54/main.c > =================================================================== > --- wireless-testing.orig/drivers/net/wireless/p54/main.c > +++ wireless-testing/drivers/net/wireless/p54/main.c > @@ -611,7 +611,9 @@ EXPORT_SYMBOL_GPL(p54_init_common); > > int p54_register_common(struct ieee80211_hw *dev, struct device *pdev) > { > +#ifdef CONFIG_P54_LEDS > struct p54_common *priv = dev->priv; > +#endif /* CONFIG_P54_LEDS */ > int err; > > err = ieee80211_register_hw(dev); > Use __maybe_unused instead? -- John W. Linville Someday the world will need a hero, and you linville@xxxxxxxxxxxxx might be all we have. Be ready. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html