mirror of
https://github.com/torvalds/linux.git
synced 2026-07-28 01:55:51 +02:00
wifi: ath9k_htc: allocate tx_buf and buf together
Use a flexible array member to combine allocations. No need to have them separate as they are always together. Signed-off-by: Rosen Penev <rosenp@gmail.com> Acked-by: Toke Høiland-Jørgensen <toke@toke.dk> Link: https://patch.msgid.link/20260521232020.261405-1-rosenp@gmail.com Signed-off-by: Jeff Johnson <jeff.johnson@oss.qualcomm.com>
This commit is contained in:
parent
f0e5e8703f
commit
38b2fb7d2d
|
|
@ -454,7 +454,6 @@ static void hif_usb_stop(void *hif_handle)
|
|||
usb_kill_urb(tx_buf->urb);
|
||||
list_del(&tx_buf->list);
|
||||
usb_free_urb(tx_buf->urb);
|
||||
kfree(tx_buf->buf);
|
||||
kfree(tx_buf);
|
||||
spin_lock_irqsave(&hif_dev->tx.tx_lock, flags);
|
||||
}
|
||||
|
|
@ -811,7 +810,6 @@ static void ath9k_hif_usb_dealloc_tx_urbs(struct hif_device_usb *hif_dev)
|
|||
&hif_dev->tx.tx_buf, list) {
|
||||
list_del(&tx_buf->list);
|
||||
usb_free_urb(tx_buf->urb);
|
||||
kfree(tx_buf->buf);
|
||||
kfree(tx_buf);
|
||||
}
|
||||
spin_unlock_irqrestore(&hif_dev->tx.tx_lock, flags);
|
||||
|
|
@ -828,7 +826,6 @@ static void ath9k_hif_usb_dealloc_tx_urbs(struct hif_device_usb *hif_dev)
|
|||
usb_kill_urb(tx_buf->urb);
|
||||
list_del(&tx_buf->list);
|
||||
usb_free_urb(tx_buf->urb);
|
||||
kfree(tx_buf->buf);
|
||||
kfree(tx_buf);
|
||||
spin_lock_irqsave(&hif_dev->tx.tx_lock, flags);
|
||||
}
|
||||
|
|
@ -849,14 +846,10 @@ static int ath9k_hif_usb_alloc_tx_urbs(struct hif_device_usb *hif_dev)
|
|||
init_usb_anchor(&hif_dev->mgmt_submitted);
|
||||
|
||||
for (i = 0; i < MAX_TX_URB_NUM; i++) {
|
||||
tx_buf = kzalloc_obj(*tx_buf);
|
||||
tx_buf = kzalloc_flex(*tx_buf, buf, MAX_TX_BUF_SIZE);
|
||||
if (!tx_buf)
|
||||
goto err;
|
||||
|
||||
tx_buf->buf = kzalloc(MAX_TX_BUF_SIZE, GFP_KERNEL);
|
||||
if (!tx_buf->buf)
|
||||
goto err;
|
||||
|
||||
tx_buf->urb = usb_alloc_urb(0, GFP_KERNEL);
|
||||
if (!tx_buf->urb)
|
||||
goto err;
|
||||
|
|
@ -871,10 +864,7 @@ static int ath9k_hif_usb_alloc_tx_urbs(struct hif_device_usb *hif_dev)
|
|||
|
||||
return 0;
|
||||
err:
|
||||
if (tx_buf) {
|
||||
kfree(tx_buf->buf);
|
||||
kfree(tx_buf);
|
||||
}
|
||||
kfree(tx_buf);
|
||||
ath9k_hif_usb_dealloc_tx_urbs(hif_dev);
|
||||
return -ENOMEM;
|
||||
}
|
||||
|
|
|
|||
|
|
@ -77,13 +77,13 @@ extern int htc_use_dev_fw;
|
|||
#define HIF_USB_MAX_TXPIPES 4
|
||||
|
||||
struct tx_buf {
|
||||
u8 *buf;
|
||||
u16 len;
|
||||
u16 offset;
|
||||
struct urb *urb;
|
||||
struct sk_buff_head skb_queue;
|
||||
struct hif_device_usb *hif_dev;
|
||||
struct list_head list;
|
||||
u8 buf[];
|
||||
};
|
||||
|
||||
struct rx_buf {
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user