ath9k: Cleanup MCI init/deinit routines
authorSujith Manoharan <c_manoha@qca.qualcomm.com>
Wed, 22 Feb 2012 07:10:09 +0000 (12:40 +0530)
committerJohn W. Linville <linville@tuxdriver.com>
Mon, 27 Feb 2012 19:06:33 +0000 (14:06 -0500)
This patch simplifies the buffer allocation functions
for MCI and removes unneeded memset calls. Also, a couple
of unused variables are removed and a memory leak in DMA
allocation is fixed.

[ 1263.788267] WARNING: at /home/sujith/dev/wireless-testing/lib/dma-debug.c:875 check_unmap+0x173/0x7e0()
[ 1263.788273] ath9k 0000:06:00.0: DMA-API: device driver frees DMA memory with different size
               [device address=0x0000000071908000] [map size=512 bytes] [unmap size=256 bytes]
[ 1263.788345] Pid: 774, comm: rmmod Tainted: G        W  O 3.3.0-rc3-wl #18
[ 1263.788348] Call Trace:
[ 1263.788355]  [<ffffffff8105110f>] warn_slowpath_common+0x7f/0xc0
[ 1263.788359]  [<ffffffff81051206>] warn_slowpath_fmt+0x46/0x50
[ 1263.788363]  [<ffffffff8125a713>] check_unmap+0x173/0x7e0
[ 1263.788368]  [<ffffffff8123fc22>] ? prio_tree_left+0x32/0xc0
[ 1263.788373]  [<ffffffff8125aded>] debug_dma_free_coherent+0x6d/0x80
[ 1263.788381]  [<ffffffffa0701c3c>] ath_mci_cleanup+0x7c/0x110 [ath9k]
[ 1263.788387]  [<ffffffffa06f4033>] ath9k_deinit_softc+0x113/0x120 [ath9k]
[ 1263.788392]  [<ffffffffa06f55cc>] ath9k_deinit_device+0x5c/0x70 [ath9k]
[ 1263.788397]  [<ffffffffa0704934>] ath_pci_remove+0x54/0xa0 [ath9k]
[ 1263.788401]  [<ffffffff81267d06>] pci_device_remove+0x46/0x110
[ 1263.788406]  [<ffffffff813102bc>] __device_release_driver+0x7c/0xe0
[ 1263.788410]  [<ffffffff81310a00>] driver_detach+0xd0/0xe0
[ 1263.788414]  [<ffffffff81310118>] bus_remove_driver+0x88/0xe0
[ 1263.788418]  [<ffffffff813111c2>] driver_unregister+0x62/0xa0
[ 1263.788421]  [<ffffffff812680c4>] pci_unregister_driver+0x44/0xc0
[ 1263.788427]  [<ffffffffa0705015>] ath_pci_exit+0x15/0x20 [ath9k]
[ 1263.788432]  [<ffffffffa070a92d>] ath9k_exit+0x15/0x31 [ath9k]
[ 1263.788436]  [<ffffffff810b971c>] sys_delete_module+0x18c/0x270
[ 1263.788441]  [<ffffffff81436edd>] ? retint_swapgs+0x13/0x1b
[ 1263.788446]  [<ffffffff812483be>] ? trace_hardirqs_on_thunk+0x3a/0x3f
[ 1263.788450]  [<ffffffff814378e9>] system_call_fastpath+0x16/0x1b
[ 1263.788453] ---[ end trace 3ab4d030ffde40d4 ]---

Signed-off-by: Sujith Manoharan <c_manoha@qca.qualcomm.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/ath/ath9k/ar9003_mci.c
drivers/net/wireless/ath/ath9k/btcoex.h
drivers/net/wireless/ath/ath9k/mci.c
drivers/net/wireless/ath/ath9k/mci.h

index 8439656a09133fc0d54b8bae2b139adf338c22eb..2cd607f9705ec0f719c358bce2cef08b33e9679a 100644 (file)
@@ -973,7 +973,6 @@ void ar9003_mci_setup(struct ath_hw *ah, u32 gpm_addr, void *gpm_buf,
                      u16 len, u32 sched_addr)
 {
        struct ath9k_hw_mci *mci = &ah->btcoex_hw.mci;
-       void *sched_buf = (void *)((char *) gpm_buf + (sched_addr - gpm_addr));
 
        if (!ATH9K_HW_CAP_MCI)
                return;
@@ -982,7 +981,6 @@ void ar9003_mci_setup(struct ath_hw *ah, u32 gpm_addr, void *gpm_buf,
        mci->gpm_buf = gpm_buf;
        mci->gpm_len = len;
        mci->sched_addr = sched_addr;
-       mci->sched_buf = sched_buf;
 
        ar9003_mci_reset(ah, true, true, true);
 }
@@ -990,14 +988,11 @@ EXPORT_SYMBOL(ar9003_mci_setup);
 
 void ar9003_mci_cleanup(struct ath_hw *ah)
 {
-       struct ath_common *common = ath9k_hw_common(ah);
-
        if (!ATH9K_HW_CAP_MCI)
                return;
 
        /* Turn off MCI and Jupiter mode. */
        REG_WRITE(ah, AR_BTCOEX_CTRL, 0x00);
-       ath_dbg(common, MCI, "MCI ar9003_mci_cleanup\n");
        ar9003_mci_disable_interrupt(ah);
 }
 EXPORT_SYMBOL(ar9003_mci_cleanup);
index 278361c867ca265171e900d5635b62b9bdfbeede..0cb7ce98394bbb705d23cd26198fea0009f1c2d0 100644 (file)
@@ -67,7 +67,6 @@ struct ath9k_hw_mci {
        u32 wlan_cal_done;
        u32 config;
        u8 *gpm_buf;
-       u8 *sched_buf;
        bool ready;
        bool update_2g5g;
        bool is_2g;
index 05c23ea4c63362a36072b699719ecd4ce2810173..85a957ab89be1d7eed86bd89296b59090f17ad7f 100644 (file)
@@ -383,84 +383,60 @@ static void ath_mci_msg(struct ath_softc *sc, u8 opcode, u8 *rx_payload)
        }
 }
 
-static int ath_mci_buf_alloc(struct ath_softc *sc, struct ath_mci_buf *buf)
-{
-       int error = 0;
-
-       buf->bf_addr = dma_alloc_coherent(sc->dev, buf->bf_len,
-                                         &buf->bf_paddr, GFP_KERNEL);
-
-       if (buf->bf_addr == NULL) {
-               error = -ENOMEM;
-               goto fail;
-       }
-
-       return 0;
-
-fail:
-       memset(buf, 0, sizeof(*buf));
-       return error;
-}
-
-static void ath_mci_buf_free(struct ath_softc *sc, struct ath_mci_buf *buf)
-{
-       if (buf->bf_addr) {
-               dma_free_coherent(sc->dev, buf->bf_len, buf->bf_addr,
-                                                       buf->bf_paddr);
-               memset(buf, 0, sizeof(*buf));
-       }
-}
-
 int ath_mci_setup(struct ath_softc *sc)
 {
        struct ath_common *common = ath9k_hw_common(sc->sc_ah);
        struct ath_mci_coex *mci = &sc->mci_coex;
-       int error = 0;
+       struct ath_mci_buf *buf = &mci->sched_buf;
 
        if (!ATH9K_HW_CAP_MCI)
                return 0;
 
-       mci->sched_buf.bf_len = ATH_MCI_SCHED_BUF_SIZE + ATH_MCI_GPM_BUF_SIZE;
+       buf->bf_addr = dma_alloc_coherent(sc->dev,
+                                 ATH_MCI_SCHED_BUF_SIZE + ATH_MCI_GPM_BUF_SIZE,
+                                 &buf->bf_paddr, GFP_KERNEL);
 
-       if (ath_mci_buf_alloc(sc, &mci->sched_buf)) {
+       if (buf->bf_addr == NULL) {
                ath_dbg(common, FATAL, "MCI buffer alloc failed\n");
-               error = -ENOMEM;
-               goto fail;
+               return -ENOMEM;
        }
 
-       mci->sched_buf.bf_len = ATH_MCI_SCHED_BUF_SIZE;
+       memset(buf->bf_addr, MCI_GPM_RSVD_PATTERN,
+              ATH_MCI_SCHED_BUF_SIZE + ATH_MCI_GPM_BUF_SIZE);
 
-       memset(mci->sched_buf.bf_addr, MCI_GPM_RSVD_PATTERN,
-                                               mci->sched_buf.bf_len);
+       mci->sched_buf.bf_len = ATH_MCI_SCHED_BUF_SIZE;
 
        mci->gpm_buf.bf_len = ATH_MCI_GPM_BUF_SIZE;
-       mci->gpm_buf.bf_addr = (u8 *)mci->sched_buf.bf_addr +
-                                                       mci->sched_buf.bf_len;
+       mci->gpm_buf.bf_addr = (u8 *)mci->sched_buf.bf_addr + mci->sched_buf.bf_len;
        mci->gpm_buf.bf_paddr = mci->sched_buf.bf_paddr + mci->sched_buf.bf_len;
 
-       /* initialize the buffer */
-       memset(mci->gpm_buf.bf_addr, MCI_GPM_RSVD_PATTERN, mci->gpm_buf.bf_len);
-
        ar9003_mci_setup(sc->sc_ah, mci->gpm_buf.bf_paddr,
                         mci->gpm_buf.bf_addr, (mci->gpm_buf.bf_len >> 4),
                         mci->sched_buf.bf_paddr);
-fail:
-       return error;
+
+       ath_dbg(common, MCI, "MCI Initialized\n");
+
+       return 0;
 }
 
 void ath_mci_cleanup(struct ath_softc *sc)
 {
+       struct ath_common *common = ath9k_hw_common(sc->sc_ah);
        struct ath_hw *ah = sc->sc_ah;
        struct ath_mci_coex *mci = &sc->mci_coex;
+       struct ath_mci_buf *buf = &mci->sched_buf;
 
        if (!ATH9K_HW_CAP_MCI)
                return;
 
-       /*
-        * both schedule and gpm buffers will be released
-        */
-       ath_mci_buf_free(sc, &mci->sched_buf);
+       if (buf->bf_addr)
+               dma_free_coherent(sc->dev,
+                                 ATH_MCI_SCHED_BUF_SIZE + ATH_MCI_GPM_BUF_SIZE,
+                                 buf->bf_addr, buf->bf_paddr);
+
        ar9003_mci_cleanup(ah);
+
+       ath_dbg(common, MCI, "MCI De-Initialized\n");
 }
 
 void ath_mci_intr(struct ath_softc *sc)
index 29e3e51d078f52e661f0693f267d6e2f031f4479..b805bf24ea4c5e70d0b655a47e5bd6667c1fef89 100644 (file)
@@ -113,7 +113,6 @@ struct ath_mci_profile {
        u8 num_bdr;
 };
 
-
 struct ath_mci_buf {
        void *bf_addr;          /* virtual addr of desc */
        dma_addr_t bf_paddr;    /* physical addr of buffer */
@@ -121,10 +120,8 @@ struct ath_mci_buf {
 };
 
 struct ath_mci_coex {
-       atomic_t mci_cal_flag;
        struct ath_mci_buf sched_buf;
        struct ath_mci_buf gpm_buf;
-       u32 bt_cal_start;
 };
 
 void ath_mci_flush_profile(struct ath_mci_profile *mci);