From mboxrd@z Thu Jan 1 00:00:00 1970 From: Larry Finger Subject: Re: [PATCH][next] rtlwifi: btcoexist: remove redundant variable fw_ps_state Date: Tue, 9 Jan 2018 10:48:19 -0600 Message-ID: References: <20180109164329.15919-1-colin.king@canonical.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Cc: kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org To: Colin King , Ping-Ke Shih , Kalle Valo , Yan-Hsuan Chuang , linux-wireless@vger.kernel.org, netdev@vger.kernel.org Return-path: In-Reply-To: <20180109164329.15919-1-colin.king@canonical.com> Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On 01/09/2018 10:43 AM, Colin King wrote: > From: Colin Ian King > > Variable fw_ps_state is assigned a value but it is never read, hence > it is redundant and can be removed. Acked-by: Larry Finger Thanks, Larry > > Cleans up clang warning: > drivers/net/wireless/realtek/rtlwifi/btcoexist/halbtcoutsrc.c:736:2: > warning: Value stored to 'fw_ps_state' is never read > > Signed-off-by: Colin Ian King > --- > drivers/net/wireless/realtek/rtlwifi/btcoexist/halbtcoutsrc.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/net/wireless/realtek/rtlwifi/btcoexist/halbtcoutsrc.c b/drivers/net/wireless/realtek/rtlwifi/btcoexist/halbtcoutsrc.c > index 5f3eda31187a..873bf9303d5d 100644 > --- a/drivers/net/wireless/realtek/rtlwifi/btcoexist/halbtcoutsrc.c > +++ b/drivers/net/wireless/realtek/rtlwifi/btcoexist/halbtcoutsrc.c > @@ -677,7 +677,7 @@ static void halbtc_display_wifi_status(struct btc_coexist *btcoexist, > u32 wifi_link_status = 0x0; > bool bt_hs_on = false, under_ips = false, under_lps = false; > bool low_power = false, dc_mode = false; > - u8 wifi_chnl = 0, wifi_hs_chnl = 0, fw_ps_state; > + u8 wifi_chnl = 0, wifi_hs_chnl = 0; > u8 ap_num = 0; > > wifi_link_status = halbtc_get_wifi_link_status(btcoexist); > @@ -733,7 +733,6 @@ static void halbtc_display_wifi_status(struct btc_coexist *btcoexist, > dc_mode = true; /*TODO*/ > under_ips = rtlpriv->psc.inactive_pwrstate == ERFOFF ? 1 : 0; > under_lps = rtlpriv->psc.dot11_psmode == EACTIVE ? 0 : 1; > - fw_ps_state = 0; > low_power = 0; /*TODO*/ > seq_printf(m, "\n %-35s = %s%s%s%s", > "Power Status", >