* [PATCH] rtw88: 8822c: Remove set but not used variable 'corr_val'
@ 2019-10-04 9:18 zhengbin
2019-10-08 9:33 ` zhengbin (A)
0 siblings, 1 reply; 4+ messages in thread
From: zhengbin @ 2019-10-04 9:18 UTC (permalink / raw)
To: yhchuang, pkshih, kvalo, davem, linux-wireless, netdev; +Cc: zhengbin13
Fixes gcc '-Wunused-but-set-variable' warning:
drivers/net/wireless/realtek/rtw88/rtw8822c.c: In function rtw8822c_dpk_dc_corr_check:
drivers/net/wireless/realtek/rtw88/rtw8822c.c:2166:5: warning: variable corr_val set but not used [-Wunused-but-set-variable]
It is not used since commit 5227c2ee453d ("rtw88:
8822c: add SW DPK support")
Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: zhengbin <zhengbin13@huawei.com>
---
drivers/net/wireless/realtek/rtw88/rtw8822c.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/net/wireless/realtek/rtw88/rtw8822c.c b/drivers/net/wireless/realtek/rtw88/rtw8822c.c
index a300efd..52682d5 100644
--- a/drivers/net/wireless/realtek/rtw88/rtw8822c.c
+++ b/drivers/net/wireless/realtek/rtw88/rtw8822c.c
@@ -2163,7 +2163,7 @@ static void rtw8822c_dpk_rxbb_dc_cal(struct rtw_dev *rtwdev, u8 path)
static u8 rtw8822c_dpk_dc_corr_check(struct rtw_dev *rtwdev, u8 path)
{
u16 dc_i, dc_q;
- u8 corr_val, corr_idx;
+ u8 corr_idx;
rtw_write32(rtwdev, REG_RXSRAM_CTL, 0x000900f0);
dc_i = (u16)rtw_read32_mask(rtwdev, REG_STAT_RPT, GENMASK(27, 16));
@@ -2176,7 +2176,6 @@ static u8 rtw8822c_dpk_dc_corr_check(struct rtw_dev *rtwdev, u8 path)
rtw_write32(rtwdev, REG_RXSRAM_CTL, 0x000000f0);
corr_idx = (u8)rtw_read32_mask(rtwdev, REG_STAT_RPT, GENMASK(7, 0));
- corr_val = (u8)rtw_read32_mask(rtwdev, REG_STAT_RPT, GENMASK(15, 8));
if (dc_i > 200 || dc_q > 200 || corr_idx < 40 || corr_idx > 65)
return 1;
--
2.7.4
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] rtw88: 8822c: Remove set but not used variable 'corr_val'
2019-10-04 9:18 [PATCH] rtw88: 8822c: Remove set but not used variable 'corr_val' zhengbin
@ 2019-10-08 9:33 ` zhengbin (A)
2019-10-08 14:24 ` Kalle Valo
0 siblings, 1 reply; 4+ messages in thread
From: zhengbin (A) @ 2019-10-08 9:33 UTC (permalink / raw)
To: yhchuang, pkshih, kvalo, davem, linux-wireless, netdev
Sorry for the noise, please ignore this
On 2019/10/4 17:18, zhengbin wrote:
> Fixes gcc '-Wunused-but-set-variable' warning:
>
> drivers/net/wireless/realtek/rtw88/rtw8822c.c: In function rtw8822c_dpk_dc_corr_check:
> drivers/net/wireless/realtek/rtw88/rtw8822c.c:2166:5: warning: variable corr_val set but not used [-Wunused-but-set-variable]
>
> It is not used since commit 5227c2ee453d ("rtw88:
> 8822c: add SW DPK support")
>
> Reported-by: Hulk Robot <hulkci@huawei.com>
> Signed-off-by: zhengbin <zhengbin13@huawei.com>
> ---
> drivers/net/wireless/realtek/rtw88/rtw8822c.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/net/wireless/realtek/rtw88/rtw8822c.c b/drivers/net/wireless/realtek/rtw88/rtw8822c.c
> index a300efd..52682d5 100644
> --- a/drivers/net/wireless/realtek/rtw88/rtw8822c.c
> +++ b/drivers/net/wireless/realtek/rtw88/rtw8822c.c
> @@ -2163,7 +2163,7 @@ static void rtw8822c_dpk_rxbb_dc_cal(struct rtw_dev *rtwdev, u8 path)
> static u8 rtw8822c_dpk_dc_corr_check(struct rtw_dev *rtwdev, u8 path)
> {
> u16 dc_i, dc_q;
> - u8 corr_val, corr_idx;
> + u8 corr_idx;
>
> rtw_write32(rtwdev, REG_RXSRAM_CTL, 0x000900f0);
> dc_i = (u16)rtw_read32_mask(rtwdev, REG_STAT_RPT, GENMASK(27, 16));
> @@ -2176,7 +2176,6 @@ static u8 rtw8822c_dpk_dc_corr_check(struct rtw_dev *rtwdev, u8 path)
>
> rtw_write32(rtwdev, REG_RXSRAM_CTL, 0x000000f0);
> corr_idx = (u8)rtw_read32_mask(rtwdev, REG_STAT_RPT, GENMASK(7, 0));
> - corr_val = (u8)rtw_read32_mask(rtwdev, REG_STAT_RPT, GENMASK(15, 8));
>
> if (dc_i > 200 || dc_q > 200 || corr_idx < 40 || corr_idx > 65)
> return 1;
> --
> 2.7.4
>
>
> .
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] rtw88: 8822c: Remove set but not used variable 'corr_val'
2019-10-08 9:33 ` zhengbin (A)
@ 2019-10-08 14:24 ` Kalle Valo
2019-10-09 0:17 ` zhengbin (A)
0 siblings, 1 reply; 4+ messages in thread
From: Kalle Valo @ 2019-10-08 14:24 UTC (permalink / raw)
To: zhengbin (A); +Cc: yhchuang, pkshih, davem, linux-wireless, netdev
(PLEASE don't top post, fixing that manually)
"zhengbin (A)" <zhengbin13@huawei.com> writes:
>
> On 2019/10/4 17:18, zhengbin wrote:
>> Fixes gcc '-Wunused-but-set-variable' warning:
>>
>> drivers/net/wireless/realtek/rtw88/rtw8822c.c: In function rtw8822c_dpk_dc_corr_check:
>> drivers/net/wireless/realtek/rtw88/rtw8822c.c:2166:5: warning: variable corr_val set but not used [-Wunused-but-set-variable]
>>
>> It is not used since commit 5227c2ee453d ("rtw88:
>> 8822c: add SW DPK support")
>>
>> Reported-by: Hulk Robot <hulkci@huawei.com>
>> Signed-off-by: zhengbin <zhengbin13@huawei.com>
>
> Sorry for the noise, please ignore this
Why? What was wrong in the patch?
--
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] rtw88: 8822c: Remove set but not used variable 'corr_val'
2019-10-08 14:24 ` Kalle Valo
@ 2019-10-09 0:17 ` zhengbin (A)
0 siblings, 0 replies; 4+ messages in thread
From: zhengbin (A) @ 2019-10-09 0:17 UTC (permalink / raw)
To: Kalle Valo; +Cc: yhchuang, pkshih, davem, linux-wireless, netdev
On 2019/10/8 22:24, Kalle Valo wrote:
> (PLEASE don't top post, fixing that manually)
>
> "zhengbin (A)" <zhengbin13@huawei.com> writes:
>
>> On 2019/10/4 17:18, zhengbin wrote:
>>> Fixes gcc '-Wunused-but-set-variable' warning:
>>>
>>> drivers/net/wireless/realtek/rtw88/rtw8822c.c: In function rtw8822c_dpk_dc_corr_check:
>>> drivers/net/wireless/realtek/rtw88/rtw8822c.c:2166:5: warning: variable corr_val set but not used [-Wunused-but-set-variable]
>>>
>>> It is not used since commit 5227c2ee453d ("rtw88:
>>> 8822c: add SW DPK support")
>>>
>>> Reported-by: Hulk Robot <hulkci@huawei.com>
>>> Signed-off-by: zhengbin <zhengbin13@huawei.com>
>> Sorry for the noise, please ignore this
> Why? What was wrong in the patch?
I am not sure whether we can just remove
corr_val = (u8)rtw_read32_mask(rtwdev, REG_STAT_RPT, GENMASK(15, 8))?
rtw_read32_mask
rtw_read32
ops->read32
Thank you for your remind of do not top post.
BTW: patchset 'net/rtlwifi: remove some unused variables' & patch 'rtlwifi: rtl8192ee: Remove set but not used variable 'err''
are acked-by Ping-Ke Shih, can you help deal with it?
>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-10-09 0:17 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-04 9:18 [PATCH] rtw88: 8822c: Remove set but not used variable 'corr_val' zhengbin
2019-10-08 9:33 ` zhengbin (A)
2019-10-08 14:24 ` Kalle Valo
2019-10-09 0:17 ` zhengbin (A)
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).