mirror of https://gitee.com/openkylin/linux.git
ath10k: remove unused function argument
The original fix has been moved into a different place in the code. Signed-off-by: Michal Kazior <michal.kazior@tieto.com> Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>
This commit is contained in:
parent
581c25f82f
commit
f0e2770ff7
|
@ -304,8 +304,7 @@ static inline struct sk_buff *ath10k_htt_rx_netbuf_pop(struct ath10k_htt *htt)
|
||||||
/* return: < 0 fatal error, 0 - non chained msdu, 1 chained msdu */
|
/* return: < 0 fatal error, 0 - non chained msdu, 1 chained msdu */
|
||||||
static int ath10k_htt_rx_amsdu_pop(struct ath10k_htt *htt,
|
static int ath10k_htt_rx_amsdu_pop(struct ath10k_htt *htt,
|
||||||
u8 **fw_desc, int *fw_desc_len,
|
u8 **fw_desc, int *fw_desc_len,
|
||||||
struct sk_buff_head *amsdu,
|
struct sk_buff_head *amsdu)
|
||||||
u32 *attention)
|
|
||||||
{
|
{
|
||||||
struct ath10k *ar = htt->ar;
|
struct ath10k *ar = htt->ar;
|
||||||
int msdu_len, msdu_chaining = 0;
|
int msdu_len, msdu_chaining = 0;
|
||||||
|
@ -346,11 +345,6 @@ static int ath10k_htt_rx_amsdu_pop(struct ath10k_htt *htt,
|
||||||
return -EIO;
|
return -EIO;
|
||||||
}
|
}
|
||||||
|
|
||||||
*attention |= __le32_to_cpu(rx_desc->attention.flags) &
|
|
||||||
(RX_ATTENTION_FLAGS_TKIP_MIC_ERR |
|
|
||||||
RX_ATTENTION_FLAGS_DECRYPT_ERR |
|
|
||||||
RX_ATTENTION_FLAGS_FCS_ERR |
|
|
||||||
RX_ATTENTION_FLAGS_MGMT_TYPE);
|
|
||||||
/*
|
/*
|
||||||
* Copy the FW rx descriptor for this MSDU from the rx
|
* Copy the FW rx descriptor for this MSDU from the rx
|
||||||
* indication message into the MSDU's netbuf. HL uses the
|
* indication message into the MSDU's netbuf. HL uses the
|
||||||
|
@ -1362,7 +1356,6 @@ static void ath10k_htt_rx_handler(struct ath10k_htt *htt,
|
||||||
struct htt_rx_indication_mpdu_range *mpdu_ranges;
|
struct htt_rx_indication_mpdu_range *mpdu_ranges;
|
||||||
struct sk_buff_head amsdu;
|
struct sk_buff_head amsdu;
|
||||||
int num_mpdu_ranges;
|
int num_mpdu_ranges;
|
||||||
u32 attention;
|
|
||||||
int fw_desc_len;
|
int fw_desc_len;
|
||||||
u8 *fw_desc;
|
u8 *fw_desc;
|
||||||
bool channel_set;
|
bool channel_set;
|
||||||
|
@ -1412,11 +1405,9 @@ static void ath10k_htt_rx_handler(struct ath10k_htt *htt,
|
||||||
mpdu_count += mpdu_ranges[i].mpdu_count;
|
mpdu_count += mpdu_ranges[i].mpdu_count;
|
||||||
|
|
||||||
while (mpdu_count--) {
|
while (mpdu_count--) {
|
||||||
attention = 0;
|
|
||||||
__skb_queue_head_init(&amsdu);
|
__skb_queue_head_init(&amsdu);
|
||||||
ret = ath10k_htt_rx_amsdu_pop(htt, &fw_desc,
|
ret = ath10k_htt_rx_amsdu_pop(htt, &fw_desc,
|
||||||
&fw_desc_len, &amsdu,
|
&fw_desc_len, &amsdu);
|
||||||
&attention);
|
|
||||||
if (ret < 0) {
|
if (ret < 0) {
|
||||||
ath10k_warn(ar, "rx ring became corrupted: %d\n", ret);
|
ath10k_warn(ar, "rx ring became corrupted: %d\n", ret);
|
||||||
__skb_queue_purge(&amsdu);
|
__skb_queue_purge(&amsdu);
|
||||||
|
@ -1445,7 +1436,6 @@ static void ath10k_htt_rx_frag_handler(struct ath10k_htt *htt,
|
||||||
int ret;
|
int ret;
|
||||||
u8 *fw_desc;
|
u8 *fw_desc;
|
||||||
int fw_desc_len;
|
int fw_desc_len;
|
||||||
u32 attention = 0;
|
|
||||||
|
|
||||||
fw_desc_len = __le16_to_cpu(frag->fw_rx_desc_bytes);
|
fw_desc_len = __le16_to_cpu(frag->fw_rx_desc_bytes);
|
||||||
fw_desc = (u8 *)frag->fw_msdu_rx_desc;
|
fw_desc = (u8 *)frag->fw_msdu_rx_desc;
|
||||||
|
@ -1454,7 +1444,7 @@ static void ath10k_htt_rx_frag_handler(struct ath10k_htt *htt,
|
||||||
|
|
||||||
spin_lock_bh(&htt->rx_ring.lock);
|
spin_lock_bh(&htt->rx_ring.lock);
|
||||||
ret = ath10k_htt_rx_amsdu_pop(htt, &fw_desc, &fw_desc_len,
|
ret = ath10k_htt_rx_amsdu_pop(htt, &fw_desc, &fw_desc_len,
|
||||||
&amsdu, &attention);
|
&amsdu);
|
||||||
spin_unlock_bh(&htt->rx_ring.lock);
|
spin_unlock_bh(&htt->rx_ring.lock);
|
||||||
|
|
||||||
tasklet_schedule(&htt->rx_replenish_task);
|
tasklet_schedule(&htt->rx_replenish_task);
|
||||||
|
|
Loading…
Reference in New Issue