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 <[email protected]>
---
V2 - use __maybe_unused instead of conditional compilation
---
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,7 @@ EXPORT_SYMBOL_GPL(p54_init_common);
int p54_register_common(struct ieee80211_hw *dev, struct device *pdev)
{
- struct p54_common *priv = dev->priv;
+ struct p54_common __maybe_unused *priv = dev->priv;
int err;
err = ieee80211_register_hw(dev);
On Tuesday 15 February 2011 16:37:06 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 <[email protected]>
> ---
>
> V2 - use __maybe_unused instead of conditional compilation
well, there is a third option [use the priv in the dev_info)
However, this approach is fine too.
Regards,
Chr
BTW, John... What's your ?WG511? doing?