Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Authenticated-By: 
From:   Pkshih <pkshih@realtek.com>
To:     Tian Tao <tiantao6@hisilicon.com>,
        "tony0620emma@gmail.com" <tony0620emma@gmail.com>,
        "kvalo@codeaurora.org" <kvalo@codeaurora.org>,
        "davem@davemloft.net" <davem@davemloft.net>,
        "kuba@kernel.org" <kuba@kernel.org>
CC:     "linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>
Subject: RE: [PATCH] rtw88: coex: remove useless if and else
Thread-Topic: [PATCH] rtw88: coex: remove useless if and else
Thread-Index: AQHW2F4yxyUKKBxgKEKYnoZyzpOw4KoPCTkg
Date:   Wed, 30 Dec 2020 03:46:58 +0000
Message-ID: <b5be281952ed470a8af59044909a8fda@realtek.com>
References: <1608640137-8914-1-git-send-email-tiantao6@hisilicon.com>
In-Reply-To: <1608640137-8914-1-git-send-email-tiantao6@hisilicon.com>
Accept-Language: en-US, zh-TW
Content-Language: zh-TW
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 8BIT
MIME-Version: 1.0
Precedence: bulk


> -----Original Message-----
> From: Tian Tao [mailto:tiantao6@hisilicon.com]
> Sent: Tuesday, December 22, 2020 8:29 PM
> To: tony0620emma@gmail.com; kvalo@codeaurora.org; davem@davemloft.net; kuba@kernel.org
> Cc: linux-wireless@vger.kernel.org; netdev@vger.kernel.org
> Subject: [PATCH] rtw88: coex: remove useless if and else
> 
> Fix the following coccinelle report:
> drivers/net/wireless/realtek/rtw88/coex.c:1619:3-5: WARNING:
> possible condition with no effect (if == else)
> 
> Signed-off-by: Tian Tao <tiantao6@hisilicon.com>
> ---
>  drivers/net/wireless/realtek/rtw88/coex.c | 7 +------
>  1 file changed, 1 insertion(+), 6 deletions(-)
> 
> diff --git a/drivers/net/wireless/realtek/rtw88/coex.c b/drivers/net/wireless/realtek/rtw88/coex.c
> index 24530ca..df6676a 100644
> --- a/drivers/net/wireless/realtek/rtw88/coex.c
> +++ b/drivers/net/wireless/realtek/rtw88/coex.c
> @@ -1616,12 +1616,7 @@ static void rtw_coex_action_bt_relink(struct rtw_dev *rtwdev)
>  	if (efuse->share_ant) { /* Shared-Ant */
>  		if (coex_stat->wl_gl_busy) {
>  			table_case = 26;
> -			if (coex_stat->bt_hid_exist &&
> -			    coex_stat->bt_profile_num == 1) {
> -				tdma_case = 20;
> -			} else {
> -				tdma_case = 20;
> -			}
> +			tdma_case = 20;
>  		} else {
>  			table_case = 1;
>  			tdma_case = 0;

I found we miss something in these branches, so I sent a patch,
namely "rtw88: coex: set 4 slot TDMA for BT link and WL busy", to fix it.

The link is
https://patchwork.kernel.org/project/linux-wireless/patch/20201230033602.13636-1-pkshih@realtek.com/

---
Ping-Ke