wifi: mwifiex: mwifiex_process_sleep_confirm_resp(): remove unused priv variable
authorSu Hui <suhui@nfschina.com>
Wed, 15 Nov 2023 09:23:29 +0000 (17:23 +0800)
committerKalle Valo <kvalo@kernel.org>
Wed, 22 Nov 2023 15:50:09 +0000 (17:50 +0200)
Clang static analyzer complains that value stored to 'priv' is never read.
'priv' is useless, so remove it to save space.

Signed-off-by: Su Hui <suhui@nfschina.com>
Acked-by: Brian Norris <briannorris@chromium.org>
Signed-off-by: Kalle Valo <kvalo@kernel.org>
Link: https://lore.kernel.org/r/20231115092328.1048103-1-suhui@nfschina.com
drivers/net/wireless/marvell/mwifiex/cmdevt.c

index 3756aa2..9eff29a 100644 (file)
@@ -1244,8 +1244,6 @@ mwifiex_process_sleep_confirm_resp(struct mwifiex_adapter *adapter,
                                   u8 *pbuf, u32 upld_len)
 {
        struct host_cmd_ds_command *cmd = (struct host_cmd_ds_command *) pbuf;
-       struct mwifiex_private *priv =
-               mwifiex_get_priv(adapter, MWIFIEX_BSS_ROLE_ANY);
        uint16_t result = le16_to_cpu(cmd->result);
        uint16_t command = le16_to_cpu(cmd->command);
        uint16_t seq_num = le16_to_cpu(cmd->seq_num);
@@ -1260,12 +1258,6 @@ mwifiex_process_sleep_confirm_resp(struct mwifiex_adapter *adapter,
                    "cmd: CMD_RESP: 0x%x, result %d, len %d, seqno 0x%x\n",
                    command, result, le16_to_cpu(cmd->size), seq_num);
 
-       /* Get BSS number and corresponding priv */
-       priv = mwifiex_get_priv_by_id(adapter, HostCmd_GET_BSS_NO(seq_num),
-                                     HostCmd_GET_BSS_TYPE(seq_num));
-       if (!priv)
-               priv = mwifiex_get_priv(adapter, MWIFIEX_BSS_ROLE_ANY);
-
        /* Update sequence number */
        seq_num = HostCmd_GET_SEQ_NO(seq_num);
        /* Clear RET_BIT from HostCmd */