Skip to content

Commit 5a17458

Browse files
dmantipovKalle Valo
authored andcommitted
wifi: rt2x00: remove useless code in rt2x00queue_create_tx_descriptor()
In 'rt2x00queue_create_tx_descriptor()', there is no need to call 'ieee80211_get_rts_cts_rate()' while checking for RTS/CTS frame since this function returns NULL or pointer to internal bitrate table entry, and the return value is not actually used. Compile tested only. Found by Linux Verification Center (linuxtesting.org) with SVACE. Signed-off-by: Dmitry Antipov <[email protected]> Acked-by: Stanislaw Gruszka <[email protected]> Signed-off-by: Kalle Valo <[email protected]> Link: https://msgid.link/[email protected]
1 parent 48fa9b6 commit 5a17458

File tree

1 file changed

+0
-3
lines changed

1 file changed

+0
-3
lines changed

drivers/net/wireless/ralink/rt2x00/rt2x00queue.c

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -416,9 +416,6 @@ static void rt2x00queue_create_tx_descriptor(struct rt2x00_dev *rt2x00dev,
416416
__set_bit(ENTRY_TXD_RTS_FRAME, &txdesc->flags);
417417
else
418418
__set_bit(ENTRY_TXD_CTS_FRAME, &txdesc->flags);
419-
if (tx_info->control.rts_cts_rate_idx >= 0)
420-
rate =
421-
ieee80211_get_rts_cts_rate(rt2x00dev->hw, tx_info);
422419
}
423420

424421
/*

0 commit comments

Comments
 (0)