All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH net-next] rtl8xxxu: Fix non static symbol warning
@ 2016-11-01 14:48 Wei Yongjun
  2016-11-17 12:53 ` [net-next] " Kalle Valo
       [not found] ` <20161117125314.0968961635@smtp.codeaurora.org>
  0 siblings, 2 replies; 3+ messages in thread
From: Wei Yongjun @ 2016-11-01 14:48 UTC (permalink / raw)
  To: Jes Sorensen, Kalle Valo; +Cc: Wei Yongjun, linux-wireless

From: Wei Yongjun <weiyongjun1@huawei.com>

Fixes the following sparse warning:

drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_8192e.c:1559:6: warning:
 symbol 'rtl8192eu_power_off' was not declared. Should it be static?

Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
---
 drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_8192e.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_8192e.c b/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_8192e.c
index a793fed..a1178c5 100644
--- a/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_8192e.c
+++ b/drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_8192e.c
@@ -1556,7 +1556,7 @@ static int rtl8192eu_power_on(struct rtl8xxxu_priv *priv)
 	return ret;
 }
 
-void rtl8192eu_power_off(struct rtl8xxxu_priv *priv)
+static void rtl8192eu_power_off(struct rtl8xxxu_priv *priv)
 {
 	u8 val8;
 	u16 val16;

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [net-next] rtl8xxxu: Fix non static symbol warning
  2016-11-01 14:48 [PATCH net-next] rtl8xxxu: Fix non static symbol warning Wei Yongjun
@ 2016-11-17 12:53 ` Kalle Valo
       [not found] ` <20161117125314.0968961635@smtp.codeaurora.org>
  1 sibling, 0 replies; 3+ messages in thread
From: Kalle Valo @ 2016-11-17 12:53 UTC (permalink / raw)
  To: Wei Yongjun; +Cc: Jes Sorensen, Wei Yongjun, linux-wireless

Wei Yongjun <weiyj.lk@gmail.com> wrote:
> From: Wei Yongjun <weiyongjun1@huawei.com>
> 
> Fixes the following sparse warning:
> 
> drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_8192e.c:1559:6: warning:
>  symbol 'rtl8192eu_power_off' was not declared. Should it be static?
> 
> Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>

Jes will take this.

-- 
https://patchwork.kernel.org/patch/9407491/

Documentation about submitting wireless patches and checking status
from patchwork:

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [net-next] rtl8xxxu: Fix non static symbol warning
       [not found] ` <20161117125314.0968961635@smtp.codeaurora.org>
@ 2016-11-17 13:03   ` Jes Sorensen
  0 siblings, 0 replies; 3+ messages in thread
From: Jes Sorensen @ 2016-11-17 13:03 UTC (permalink / raw)
  To: Kalle Valo; +Cc: Wei Yongjun, Wei Yongjun, linux-wireless

Kalle Valo <kvalo@codeaurora.org> writes:
> Wei Yongjun <weiyj.lk@gmail.com> wrote:
>> From: Wei Yongjun <weiyongjun1@huawei.com>
>> 
>> Fixes the following sparse warning:
>> 
>> drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_8192e.c:1559:6: warning:
>>  symbol 'rtl8192eu_power_off' was not declared. Should it be static?
>> 
>> Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
>
> Jes will take this.

Thanks Kalle - yes it's on my list, Wei your patch has not been
forgotten.

Cheers,
Jes

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2016-11-17 17:48 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-11-01 14:48 [PATCH net-next] rtl8xxxu: Fix non static symbol warning Wei Yongjun
2016-11-17 12:53 ` [net-next] " Kalle Valo
     [not found] ` <20161117125314.0968961635@smtp.codeaurora.org>
2016-11-17 13:03   ` Jes Sorensen

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.