* [PATCH next] staging: rtl8723bs: HalBtc8723b1Ant.c: Remove unused variables
@ 2021-05-26 7:45 Bixuan Cui
2021-05-27 11:01 ` Greg KH
0 siblings, 1 reply; 3+ messages in thread
From: Bixuan Cui @ 2021-05-26 7:45 UTC (permalink / raw)
To: gregkh
Cc: dan.carpenter, desmondcheongzx, marcocesati, fabioaiuto83,
linux-staging, linux-kernel, Bixuan Cui
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="y", Size: 1875 bytes --]
Fix the build warning: [-Wunused-but-set-variable]
drivers/staging/rtl8723bs/hal/HalBtc8723b1Ant.c:2710:6: warning: variable ‘u4Tmp’ set but not used
2710 | u32 u4Tmp;
| ^~~~~
drivers/staging/rtl8723bs/hal/HalBtc8723b1Ant.c:2709:13: warning: variable ‘u1Tmpb’ set but not used
2709 | u8 u1Tmpa, u1Tmpb;
| ^~~~~~
drivers/staging/rtl8723bs/hal/HalBtc8723b1Ant.c:2709:5: warning: variable ‘u1Tmpa’ set but not used
2709 | u8 u1Tmpa, u1Tmpb;
| ^~~~~~
Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Bixuan Cui <cuibixuan@huawei.com>
---
drivers/staging/rtl8723bs/hal/HalBtc8723b1Ant.c | 9 +++------
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/drivers/staging/rtl8723bs/hal/HalBtc8723b1Ant.c b/drivers/staging/rtl8723bs/hal/HalBtc8723b1Ant.c
index 518d5354bda4..adfdc4f14b08 100644
--- a/drivers/staging/rtl8723bs/hal/HalBtc8723b1Ant.c
+++ b/drivers/staging/rtl8723bs/hal/HalBtc8723b1Ant.c
@@ -2706,9 +2706,6 @@ void EXhalbtc8723b1ant_ScanNotify(struct btc_coexist *pBtCoexist, u8 type)
bool bBtCtrlAggBufSize = false;
u8 aggBufSize = 5;
- u8 u1Tmpa, u1Tmpb;
- u32 u4Tmp;
-
if (pBtCoexist->bManualControl || pBtCoexist->bStopCoexDm)
return;
@@ -2716,9 +2713,9 @@ void EXhalbtc8723b1ant_ScanNotify(struct btc_coexist *pBtCoexist, u8 type)
pCoexSta->bWiFiIsHighPriTask = true;
halbtc8723b1ant_PsTdma(pBtCoexist, FORCE_EXEC, false, 8); /* Force antenna setup for no scan result issue */
- u4Tmp = pBtCoexist->fBtcRead4Byte(pBtCoexist, 0x948);
- u1Tmpa = pBtCoexist->fBtcRead1Byte(pBtCoexist, 0x765);
- u1Tmpb = pBtCoexist->fBtcRead1Byte(pBtCoexist, 0x67);
+ pBtCoexist->fBtcRead4Byte(pBtCoexist, 0x948);
+ pBtCoexist->fBtcRead1Byte(pBtCoexist, 0x765);
+ pBtCoexist->fBtcRead1Byte(pBtCoexist, 0x67);
} else {
pCoexSta->bWiFiIsHighPriTask = false;
--
2.17.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH next] staging: rtl8723bs: HalBtc8723b1Ant.c: Remove unused variables
2021-05-26 7:45 [PATCH next] staging: rtl8723bs: HalBtc8723b1Ant.c: Remove unused variables Bixuan Cui
@ 2021-05-27 11:01 ` Greg KH
2021-05-28 1:27 ` Bixuan Cui
0 siblings, 1 reply; 3+ messages in thread
From: Greg KH @ 2021-05-27 11:01 UTC (permalink / raw)
To: Bixuan Cui
Cc: dan.carpenter, desmondcheongzx, marcocesati, fabioaiuto83,
linux-staging, linux-kernel
On Wed, May 26, 2021 at 03:45:37PM +0800, Bixuan Cui wrote:
> Fix the build warning: [-Wunused-but-set-variable]
>
> drivers/staging/rtl8723bs/hal/HalBtc8723b1Ant.c:2710:6: warning: variable ‘u4Tmp’ set but not used
> 2710 | u32 u4Tmp;
> | ^~~~~
> drivers/staging/rtl8723bs/hal/HalBtc8723b1Ant.c:2709:13: warning: variable ‘u1Tmpb’ set but not used
> 2709 | u8 u1Tmpa, u1Tmpb;
> | ^~~~~~
> drivers/staging/rtl8723bs/hal/HalBtc8723b1Ant.c:2709:5: warning: variable ‘u1Tmpa’ set but not used
> 2709 | u8 u1Tmpa, u1Tmpb;
> | ^~~~~~
>
> Reported-by: Hulk Robot <hulkci@huawei.com>
> Signed-off-by: Bixuan Cui <cuibixuan@huawei.com>
> ---
> drivers/staging/rtl8723bs/hal/HalBtc8723b1Ant.c | 9 +++------
> 1 file changed, 3 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/staging/rtl8723bs/hal/HalBtc8723b1Ant.c b/drivers/staging/rtl8723bs/hal/HalBtc8723b1Ant.c
> index 518d5354bda4..adfdc4f14b08 100644
> --- a/drivers/staging/rtl8723bs/hal/HalBtc8723b1Ant.c
> +++ b/drivers/staging/rtl8723bs/hal/HalBtc8723b1Ant.c
> @@ -2706,9 +2706,6 @@ void EXhalbtc8723b1ant_ScanNotify(struct btc_coexist *pBtCoexist, u8 type)
> bool bBtCtrlAggBufSize = false;
> u8 aggBufSize = 5;
>
> - u8 u1Tmpa, u1Tmpb;
> - u32 u4Tmp;
> -
> if (pBtCoexist->bManualControl || pBtCoexist->bStopCoexDm)
> return;
>
> @@ -2716,9 +2713,9 @@ void EXhalbtc8723b1ant_ScanNotify(struct btc_coexist *pBtCoexist, u8 type)
> pCoexSta->bWiFiIsHighPriTask = true;
>
> halbtc8723b1ant_PsTdma(pBtCoexist, FORCE_EXEC, false, 8); /* Force antenna setup for no scan result issue */
> - u4Tmp = pBtCoexist->fBtcRead4Byte(pBtCoexist, 0x948);
> - u1Tmpa = pBtCoexist->fBtcRead1Byte(pBtCoexist, 0x765);
> - u1Tmpb = pBtCoexist->fBtcRead1Byte(pBtCoexist, 0x67);
> + pBtCoexist->fBtcRead4Byte(pBtCoexist, 0x948);
> + pBtCoexist->fBtcRead1Byte(pBtCoexist, 0x765);
> + pBtCoexist->fBtcRead1Byte(pBtCoexist, 0x67);
> } else {
> pCoexSta->bWiFiIsHighPriTask = false;
>
> --
> 2.17.1
>
>
Does not apply to my tree :(
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH next] staging: rtl8723bs: HalBtc8723b1Ant.c: Remove unused variables
2021-05-27 11:01 ` Greg KH
@ 2021-05-28 1:27 ` Bixuan Cui
0 siblings, 0 replies; 3+ messages in thread
From: Bixuan Cui @ 2021-05-28 1:27 UTC (permalink / raw)
To: Greg KH
Cc: dan.carpenter, desmondcheongzx, marcocesati, fabioaiuto83,
linux-staging, linux-kernel
On 2021/5/27 19:01, Greg KH wrote:
>> halbtc8723b1ant_PsTdma(pBtCoexist, FORCE_EXEC, false, 8); /* Force antenna setup for no scan result issue */
>> - u4Tmp = pBtCoexist->fBtcRead4Byte(pBtCoexist, 0x948);
>> - u1Tmpa = pBtCoexist->fBtcRead1Byte(pBtCoexist, 0x765);
>> - u1Tmpb = pBtCoexist->fBtcRead1Byte(pBtCoexist, 0x67);
>> + pBtCoexist->fBtcRead4Byte(pBtCoexist, 0x948);
>> + pBtCoexist->fBtcRead1Byte(pBtCoexist, 0x765);
>> + pBtCoexist->fBtcRead1Byte(pBtCoexist, 0x67);
>> } else {
>> pCoexSta->bWiFiIsHighPriTask = false;
>>
>> --
>> 2.17.1
>>
>>
> Does not apply to my tree :(
Hi,
This warning has been fixed by commit db52e8f48067. Please ignore my patch.
Thanks
Bixuan Cui
> .
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-05-28 1:27 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-05-26 7:45 [PATCH next] staging: rtl8723bs: HalBtc8723b1Ant.c: Remove unused variables Bixuan Cui
2021-05-27 11:01 ` Greg KH
2021-05-28 1:27 ` Bixuan Cui
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.