On Thu, 2020-12-03 at 17:51 +0000, Colin King wrote: > From: Colin Ian King <colin.k...@canonical.com> > > Currently the variable 'interval' is not initialized and is only set > to 1 when oex_stat->bt_418_hid_existi is true. Fix this by inintializing > variable interval to 0 (which I'm assuming is the intended default). > > Addresses-Coverity: ("Uninitalized scalar variable") > Fixes: 5b2e9a35e456 ("rtw88: coex: add feature to enhance HID coexistence > performance") > Signed-off-by: Colin Ian King <colin.k...@canonical.com>
Thanks for your fix. Acked-by: Ping-Ke Shih <pks...@realtek.com> > --- > drivers/net/wireless/realtek/rtw88/coex.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/wireless/realtek/rtw88/coex.c > b/drivers/net/wireless/realtek/rtw88/coex.c > index c704c6885a18..24530cafcba7 100644 > --- a/drivers/net/wireless/realtek/rtw88/coex.c > +++ b/drivers/net/wireless/realtek/rtw88/coex.c > @@ -2051,7 +2051,7 @@ static void rtw_coex_action_bt_a2dp_hid(struct rtw_dev > *rtwdev) > struct rtw_coex_dm *coex_dm = &coex->dm; > struct rtw_efuse *efuse = &rtwdev->efuse; > struct rtw_chip_info *chip = rtwdev->chip; > - u8 table_case, tdma_case, interval; > + u8 table_case, tdma_case, interval = 0; > u32 slot_type = 0; > bool is_toggle_table = false; > > -- > 2.29.2 > > > ------Please consider the environment before printing this e-mail.