Commit 4721213f authored by Kalle Valo's avatar Kalle Valo Committed by John W. Linville

wl12xx: fix rx descriptor use

Rx descriptor was incorrectly allocated from stack, use struct wl12xx
instead. Needed for DMA transfers.
Signed-off-by: default avatarKalle Valo <kalle.valo@nokia.com>
Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
parent 53d65423
...@@ -1261,6 +1261,13 @@ static int __devinit wl12xx_probe(struct spi_device *spi) ...@@ -1261,6 +1261,13 @@ static int __devinit wl12xx_probe(struct spi_device *spi)
wl->tx_mgmt_frm_rate = DEFAULT_HW_GEN_TX_RATE; wl->tx_mgmt_frm_rate = DEFAULT_HW_GEN_TX_RATE;
wl->tx_mgmt_frm_mod = DEFAULT_HW_GEN_MODULATION_TYPE; wl->tx_mgmt_frm_mod = DEFAULT_HW_GEN_MODULATION_TYPE;
wl->rx_descriptor = kmalloc(sizeof(*wl->rx_descriptor), GFP_KERNEL);
if (!wl->rx_descriptor) {
wl12xx_error("could not allocate memory for rx descriptor");
ret = -ENOMEM;
goto out_free;
}
/* This is the only SPI value that we need to set here, the rest /* This is the only SPI value that we need to set here, the rest
* comes from the board-peripherals file */ * comes from the board-peripherals file */
spi->bits_per_word = 32; spi->bits_per_word = 32;
...@@ -1313,6 +1320,9 @@ static int __devinit wl12xx_probe(struct spi_device *spi) ...@@ -1313,6 +1320,9 @@ static int __devinit wl12xx_probe(struct spi_device *spi)
free_irq(wl->irq, wl); free_irq(wl->irq, wl);
out_free: out_free:
kfree(wl->rx_descriptor);
wl->rx_descriptor = NULL;
ieee80211_free_hw(hw); ieee80211_free_hw(hw);
return ret; return ret;
...@@ -1333,6 +1343,10 @@ static int __devexit wl12xx_remove(struct spi_device *spi) ...@@ -1333,6 +1343,10 @@ static int __devexit wl12xx_remove(struct spi_device *spi)
wl->fw = NULL; wl->fw = NULL;
kfree(wl->nvs); kfree(wl->nvs);
wl->nvs = NULL; wl->nvs = NULL;
kfree(wl->rx_descriptor);
wl->rx_descriptor = NULL;
ieee80211_free_hw(wl->hw); ieee80211_free_hw(wl->hw);
return 0; return 0;
......
...@@ -39,8 +39,7 @@ static void wl12xx_rx_header(struct wl12xx *wl, ...@@ -39,8 +39,7 @@ static void wl12xx_rx_header(struct wl12xx *wl,
if (wl->rx_current_buffer) if (wl->rx_current_buffer)
rx_packet_ring_addr += wl->data_path->rx_packet_ring_chunk_size; rx_packet_ring_addr += wl->data_path->rx_packet_ring_chunk_size;
wl12xx_spi_mem_read(wl, rx_packet_ring_addr, desc, wl12xx_spi_mem_read(wl, rx_packet_ring_addr, desc, sizeof(*desc));
sizeof(struct wl12xx_rx_descriptor));
} }
static void wl12xx_rx_status(struct wl12xx *wl, static void wl12xx_rx_status(struct wl12xx *wl,
...@@ -175,16 +174,18 @@ static void wl12xx_rx_ack(struct wl12xx *wl) ...@@ -175,16 +174,18 @@ static void wl12xx_rx_ack(struct wl12xx *wl)
void wl12xx_rx(struct wl12xx *wl) void wl12xx_rx(struct wl12xx *wl)
{ {
struct wl12xx_rx_descriptor rx_desc; struct wl12xx_rx_descriptor *rx_desc;
if (wl->state != WL12XX_STATE_ON) if (wl->state != WL12XX_STATE_ON)
return; return;
rx_desc = wl->rx_descriptor;
/* We first read the frame's header */ /* We first read the frame's header */
wl12xx_rx_header(wl, &rx_desc); wl12xx_rx_header(wl, rx_desc);
/* Now we can read the body */ /* Now we can read the body */
wl12xx_rx_body(wl, &rx_desc); wl12xx_rx_body(wl, rx_desc);
/* Finally, we need to ACK the RX */ /* Finally, we need to ACK the RX */
wl12xx_rx_ack(wl); wl12xx_rx_ack(wl);
......
...@@ -387,6 +387,7 @@ struct wl12xx { ...@@ -387,6 +387,7 @@ struct wl12xx {
u32 buffer_32; u32 buffer_32;
u32 buffer_cmd; u32 buffer_cmd;
u8 buffer_busyword[WL12XX_BUSY_WORD_LEN]; u8 buffer_busyword[WL12XX_BUSY_WORD_LEN];
struct wl12xx_rx_descriptor *rx_descriptor;
}; };
int wl12xx_plt_start(struct wl12xx *wl); int wl12xx_plt_start(struct wl12xx *wl);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment