wl12xx: fix rx descriptor use
authorKalle Valo <kalle.valo@nokia.com>
Fri, 12 Jun 2009 11:15:08 +0000 (14:15 +0300)
committerJohn W. Linville <linville@tuxdriver.com>
Fri, 10 Jul 2009 18:57:43 +0000 (14:57 -0400)
Rx descriptor was incorrectly allocated from stack, use struct wl12xx
instead. Needed for DMA transfers.

Signed-off-by: Kalle Valo <kalle.valo@nokia.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/wl12xx/main.c
drivers/net/wireless/wl12xx/rx.c
drivers/net/wireless/wl12xx/wl12xx.h

index 6d27cd687672fd446f28fa1e66bee6d8c8849f1d..3c6cd774192ddbd0ba2712e9f630ad9adb4cef9e 100644 (file)
@@ -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_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
         * comes from the board-peripherals file */
        spi->bits_per_word = 32;
@@ -1313,6 +1320,9 @@ static int __devinit wl12xx_probe(struct spi_device *spi)
        free_irq(wl->irq, wl);
 
  out_free:
+       kfree(wl->rx_descriptor);
+       wl->rx_descriptor = NULL;
+
        ieee80211_free_hw(hw);
 
        return ret;
@@ -1333,6 +1343,10 @@ static int __devexit wl12xx_remove(struct spi_device *spi)
        wl->fw = NULL;
        kfree(wl->nvs);
        wl->nvs = NULL;
+
+       kfree(wl->rx_descriptor);
+       wl->rx_descriptor = NULL;
+
        ieee80211_free_hw(wl->hw);
 
        return 0;
index 5fd916a0b254e208880141284a6f9e5ebfac6daa..7ac26ef209b7d71cc6a0c864696d8e2816853e5f 100644 (file)
@@ -39,8 +39,7 @@ static void wl12xx_rx_header(struct wl12xx *wl,
        if (wl->rx_current_buffer)
                rx_packet_ring_addr += wl->data_path->rx_packet_ring_chunk_size;
 
-       wl12xx_spi_mem_read(wl, rx_packet_ring_addr, desc,
-                           sizeof(struct wl12xx_rx_descriptor));
+       wl12xx_spi_mem_read(wl, rx_packet_ring_addr, desc, sizeof(*desc));
 }
 
 static void wl12xx_rx_status(struct wl12xx *wl,
@@ -175,16 +174,18 @@ static void wl12xx_rx_ack(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)
                return;
 
+       rx_desc = wl->rx_descriptor;
+
        /* 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 */
-       wl12xx_rx_body(wl, &rx_desc);
+       wl12xx_rx_body(wl, rx_desc);
 
        /* Finally, we need to ACK the RX */
        wl12xx_rx_ack(wl);
index b81102098c7d7da6bdebb344440e01b669baed52..b87421461a7db6342a4eaaddb3193a51eb06b974 100644 (file)
@@ -387,6 +387,7 @@ struct wl12xx {
        u32 buffer_32;
        u32 buffer_cmd;
        u8 buffer_busyword[WL12XX_BUSY_WORD_LEN];
+       struct wl12xx_rx_descriptor *rx_descriptor;
 };
 
 int wl12xx_plt_start(struct wl12xx *wl);