2016-07-12 11:33:48

by weiyj_lk

Subject: [PATCH -next] libertas: fix non static symbol warning

From: Wei Yongjun <[email protected]>

Fixes the following sparse warning:

drivers/net/wireless/marvell/libertas/cfg.c:2047:5: warning:
symbol 'lbs_set_power_mgmt' was not declared. Should it be static?

Signed-off-by: Wei Yongjun <[email protected]>
---
drivers/net/wireless/marvell/libertas/cfg.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/wireless/marvell/libertas/cfg.c b/drivers/net/wireless/marvell/libertas/cfg.c
index ea48024..7ff2efa 100644
--- a/drivers/net/wireless/marvell/libertas/cfg.c
+++ b/drivers/net/wireless/marvell/libertas/cfg.c
@@ -2044,8 +2044,8 @@ static int lbs_leave_ibss(struct wiphy *wiphy, struct net_device *dev)



-int lbs_set_power_mgmt(struct wiphy *wiphy, struct net_device *dev,
- bool enabled, int timeout)
+static int lbs_set_power_mgmt(struct wiphy *wiphy, struct net_device *dev,
+ bool enabled, int timeout)
{
struct lbs_private *priv = wiphy_priv(wiphy);

2016-07-18 19:39:31

by Kalle Valo

Subject: Re: [-next] libertas: fix non static symbol warning

[email protected] wrote:
> From: Wei Yongjun <[email protected]>
>
> Fixes the following sparse warning:
>
> drivers/net/wireless/marvell/libertas/cfg.c:2047:5: warning:
> symbol 'lbs_set_power_mgmt' was not declared. Should it be static?
>
> Signed-off-by: Wei Yongjun <[email protected]>

Thanks, 1 patch applied to wireless-drivers-next.git:

83d58d53e062 libertas: fix non static symbol warning

--
Sent by pwcli
https://patchwork.kernel.org/patch/9225147/