staging: use pci_zalloc_consistent
authorJoe Perches <joe@perches.com>
Fri, 8 Aug 2014 21:24:48 +0000 (14:24 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Fri, 8 Aug 2014 22:57:30 +0000 (15:57 -0700)
Remove the now unnecessary memset too.

Signed-off-by: Joe Perches <joe@perches.com>
Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Lior Dotan <liodot@gmail.com>
Cc: Christopher Harrer <charrer@alacritech.com>
Cc: Forest Bond <forest@alittletooquiet.net>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
drivers/staging/rtl8192e/rtl8192e/rtl_core.c
drivers/staging/rtl8192ee/pci.c
drivers/staging/rtl8821ae/pci.c
drivers/staging/slicoss/slicoss.c
drivers/staging/vt6655/device_main.c

index 2920e406030a8fd332a2636c36682a6f52a09284..5729cf678765e6646347bb62a45c08e72d1290f2 100644 (file)
@@ -2065,20 +2065,16 @@ static short rtl8192_alloc_rx_desc_ring(struct net_device *dev)
        int i, rx_queue_idx;
 
        for (rx_queue_idx = 0; rx_queue_idx < MAX_RX_QUEUE; rx_queue_idx++) {
-               priv->rx_ring[rx_queue_idx] = pci_alloc_consistent(priv->pdev,
-                                       sizeof(*priv->rx_ring[rx_queue_idx]) *
-                                       priv->rxringcount,
-                                       &priv->rx_ring_dma[rx_queue_idx]);
-
+               priv->rx_ring[rx_queue_idx] =
+                       pci_zalloc_consistent(priv->pdev,
+                                             sizeof(*priv->rx_ring[rx_queue_idx]) * priv->rxringcount,
+                                             &priv->rx_ring_dma[rx_queue_idx]);
                if (!priv->rx_ring[rx_queue_idx] ||
                    (unsigned long)priv->rx_ring[rx_queue_idx] & 0xFF) {
                        RT_TRACE(COMP_ERR, "Cannot allocate RX ring\n");
                        return -ENOMEM;
                }
 
-               memset(priv->rx_ring[rx_queue_idx], 0,
-                      sizeof(*priv->rx_ring[rx_queue_idx]) *
-                      priv->rxringcount);
                priv->rx_idx[rx_queue_idx] = 0;
 
                for (i = 0; i < priv->rxringcount; i++) {
@@ -2118,14 +2114,13 @@ static int rtl8192_alloc_tx_desc_ring(struct net_device *dev,
        dma_addr_t dma;
        int i;
 
-       ring = pci_alloc_consistent(priv->pdev, sizeof(*ring) * entries, &dma);
+       ring = pci_zalloc_consistent(priv->pdev, sizeof(*ring) * entries, &dma);
        if (!ring || (unsigned long)ring & 0xFF) {
                RT_TRACE(COMP_ERR, "Cannot allocate TX ring (prio = %d)\n",
                         prio);
                return -ENOMEM;
        }
 
-       memset(ring, 0, sizeof(*ring)*entries);
        priv->tx_ring[prio].desc = ring;
        priv->tx_ring[prio].dma = dma;
        priv->tx_ring[prio].idx = 0;
index f3abbcc9f3ba70a065d60b50627a2037716d6680..0215aef1eacc14004aa75f5d91f91f8f3ab40619 100644 (file)
@@ -1224,10 +1224,10 @@ static int _rtl_pci_init_tx_ring(struct ieee80211_hw *hw,
 
        /* alloc tx buffer desc for new trx flow*/
        if (rtlpriv->use_new_trx_flow) {
-               buffer_desc = pci_alloc_consistent(rtlpci->pdev,
-                                       sizeof(*buffer_desc) * entries,
-                                       &buffer_desc_dma);
-
+               buffer_desc =
+                       pci_zalloc_consistent(rtlpci->pdev,
+                                             sizeof(*buffer_desc) * entries,
+                                             &buffer_desc_dma);
                if (!buffer_desc || (unsigned long)buffer_desc & 0xFF) {
                        RT_TRACE(COMP_ERR, DBG_EMERG,
                                 ("Cannot allocate TX ring (prio = %d)\n",
@@ -1235,7 +1235,6 @@ static int _rtl_pci_init_tx_ring(struct ieee80211_hw *hw,
                        return -ENOMEM;
                }
 
-               memset(buffer_desc, 0, sizeof(*buffer_desc) * entries);
                rtlpci->tx_ring[prio].buffer_desc = buffer_desc;
                rtlpci->tx_ring[prio].buffer_desc_dma = buffer_desc_dma;
 
@@ -1245,16 +1244,14 @@ static int _rtl_pci_init_tx_ring(struct ieee80211_hw *hw,
        }
 
        /* alloc dma for this ring */
-       desc = pci_alloc_consistent(rtlpci->pdev,
-                                   sizeof(*desc) * entries, &desc_dma);
-
+       desc = pci_zalloc_consistent(rtlpci->pdev, sizeof(*desc) * entries,
+                                    &desc_dma);
        if (!desc || (unsigned long)desc & 0xFF) {
                RT_TRACE(COMP_ERR, DBG_EMERG,
                         ("Cannot allocate TX ring (prio = %d)\n", prio));
                return -ENOMEM;
        }
 
-       memset(desc, 0, sizeof(*desc) * entries);
        rtlpci->tx_ring[prio].desc = desc;
        rtlpci->tx_ring[prio].dma = desc_dma;
 
@@ -1290,11 +1287,9 @@ static int _rtl_pci_init_rx_ring(struct ieee80211_hw *hw, int rxring_idx)
                struct rtl_rx_buffer_desc *entry = NULL;
                /* alloc dma for this ring */
                rtlpci->rx_ring[rxring_idx].buffer_desc =
-                   pci_alloc_consistent(rtlpci->pdev,
-                                        sizeof(*rtlpci->rx_ring[rxring_idx].
-                                               buffer_desc) *
-                                               rtlpci->rxringcount,
-                                        &rtlpci->rx_ring[rxring_idx].dma);
+                       pci_zalloc_consistent(rtlpci->pdev,
+                                             sizeof(*rtlpci->rx_ring[rxring_idx].buffer_desc) * rtlpci->rxringcount,
+                                             &rtlpci->rx_ring[rxring_idx].dma);
                if (!rtlpci->rx_ring[rxring_idx].buffer_desc ||
                    (unsigned long)rtlpci->rx_ring[rxring_idx].buffer_desc & 0xFF) {
                        RT_TRACE(COMP_ERR, DBG_EMERG,
@@ -1302,10 +1297,6 @@ static int _rtl_pci_init_rx_ring(struct ieee80211_hw *hw, int rxring_idx)
                        return -ENOMEM;
                }
 
-               memset(rtlpci->rx_ring[rxring_idx].buffer_desc, 0,
-                      sizeof(*rtlpci->rx_ring[rxring_idx].buffer_desc) *
-                      rtlpci->rxringcount);
-
                /* init every desc in this ring */
                rtlpci->rx_ring[rxring_idx].idx = 0;
 
@@ -1320,19 +1311,15 @@ static int _rtl_pci_init_rx_ring(struct ieee80211_hw *hw, int rxring_idx)
                u8 tmp_one = 1;
                /* alloc dma for this ring */
                rtlpci->rx_ring[rxring_idx].desc =
-                   pci_alloc_consistent(rtlpci->pdev,
-                                        sizeof(*rtlpci->rx_ring[rxring_idx].
-                                       desc) * rtlpci->rxringcount,
-                                        &rtlpci->rx_ring[rxring_idx].dma);
+                       pci_zalloc_consistent(rtlpci->pdev,
+                                             sizeof(*rtlpci->rx_ring[rxring_idx].desc) * rtlpci->rxringcount,
+                                             &rtlpci->rx_ring[rxring_idx].dma);
                if (!rtlpci->rx_ring[rxring_idx].desc ||
                    (unsigned long)rtlpci->rx_ring[rxring_idx].desc & 0xFF) {
                        RT_TRACE(COMP_ERR, DBG_EMERG,
                                 ("Cannot allocate RX ring\n"));
                        return -ENOMEM;
                }
-               memset(rtlpci->rx_ring[rxring_idx].desc, 0,
-                      sizeof(*rtlpci->rx_ring[rxring_idx].desc) *
-                      rtlpci->rxringcount);
 
                /* init every desc in this ring */
                rtlpci->rx_ring[rxring_idx].idx = 0;
index f9847d1fbdebae821e8aaab5f32ae0109bc7dfe5..26d7b2fc852a4f07a54db7dc7465ced58b7d4a70 100644 (file)
@@ -1248,9 +1248,10 @@ static int _rtl_pci_init_tx_ring(struct ieee80211_hw *hw,
 
        /* alloc tx buffer desc for new trx flow*/
        if (rtlpriv->use_new_trx_flow) {
-               buffer_desc = pci_alloc_consistent(rtlpci->pdev,
-                                           sizeof(*buffer_desc) * entries,
-                                           &buffer_desc_dma);
+               buffer_desc =
+                       pci_zalloc_consistent(rtlpci->pdev,
+                                             sizeof(*buffer_desc) * entries,
+                                             &buffer_desc_dma);
 
                if (!buffer_desc || (unsigned long)buffer_desc & 0xFF) {
                        RT_TRACE(COMP_ERR, DBG_EMERG,
@@ -1259,7 +1260,6 @@ static int _rtl_pci_init_tx_ring(struct ieee80211_hw *hw,
                        return -ENOMEM;
                }
 
-               memset(buffer_desc, 0, sizeof(*buffer_desc) * entries);
                rtlpci->tx_ring[prio].buffer_desc = buffer_desc;
                rtlpci->tx_ring[prio].buffer_desc_dma = buffer_desc_dma;
 
@@ -1270,8 +1270,8 @@ static int _rtl_pci_init_tx_ring(struct ieee80211_hw *hw,
        }
 
        /* alloc dma for this ring */
-       desc = pci_alloc_consistent(rtlpci->pdev,
-                                   sizeof(*desc) * entries, &desc_dma);
+       desc = pci_zalloc_consistent(rtlpci->pdev, sizeof(*desc) * entries,
+                                    &desc_dma);
 
        if (!desc || (unsigned long)desc & 0xFF) {
                RT_TRACE(COMP_ERR, DBG_EMERG,
@@ -1279,7 +1279,6 @@ static int _rtl_pci_init_tx_ring(struct ieee80211_hw *hw,
                return -ENOMEM;
        }
 
-       memset(desc, 0, sizeof(*desc) * entries);
        rtlpci->tx_ring[prio].desc = desc;
        rtlpci->tx_ring[prio].dma = desc_dma;
 
@@ -1316,21 +1315,15 @@ static int _rtl_pci_init_rx_ring(struct ieee80211_hw *hw, int rxring_idx)
                struct rtl_rx_buffer_desc *entry = NULL;
                /* alloc dma for this ring */
                rtlpci->rx_ring[rxring_idx].buffer_desc =
-                   pci_alloc_consistent(rtlpci->pdev,
-                                        sizeof(*rtlpci->rx_ring[rxring_idx].
-                                               buffer_desc) *
-                                               rtlpci->rxringcount,
-                                        &rtlpci->rx_ring[rxring_idx].dma);
+                       pci_zalloc_consistent(rtlpci->pdev,
+                                             sizeof(*rtlpci->rx_ring[rxring_idx].buffer_desc) * rtlpci->rxringcount,
+                                             &rtlpci->rx_ring[rxring_idx].dma);
                if (!rtlpci->rx_ring[rxring_idx].buffer_desc ||
                    (unsigned long)rtlpci->rx_ring[rxring_idx].buffer_desc & 0xFF) {
                        RT_TRACE(COMP_ERR, DBG_EMERG, ("Cannot allocate RX ring\n"));
                        return -ENOMEM;
                }
 
-               memset(rtlpci->rx_ring[rxring_idx].buffer_desc, 0,
-                      sizeof(*rtlpci->rx_ring[rxring_idx].buffer_desc) *
-                      rtlpci->rxringcount);
-
                /* init every desc in this ring */
                rtlpci->rx_ring[rxring_idx].idx = 0;
                for (i = 0; i < rtlpci->rxringcount; i++) {
@@ -1344,10 +1337,9 @@ static int _rtl_pci_init_rx_ring(struct ieee80211_hw *hw, int rxring_idx)
                u8 tmp_one = 1;
                /* alloc dma for this ring */
                rtlpci->rx_ring[rxring_idx].desc =
-                   pci_alloc_consistent(rtlpci->pdev,
-                                        sizeof(*rtlpci->rx_ring[rxring_idx].
-                                               desc) * rtlpci->rxringcount,
-                                        &rtlpci->rx_ring[rxring_idx].dma);
+                       pci_zalloc_consistent(rtlpci->pdev,
+                                             sizeof(*rtlpci->rx_ring[rxring_idx].desc) * rtlpci->rxringcount,
+                                             &rtlpci->rx_ring[rxring_idx].dma);
                if (!rtlpci->rx_ring[rxring_idx].desc ||
                    (unsigned long)rtlpci->rx_ring[rxring_idx].desc & 0xFF) {
                        RT_TRACE(COMP_ERR, DBG_EMERG,
@@ -1355,10 +1347,6 @@ static int _rtl_pci_init_rx_ring(struct ieee80211_hw *hw, int rxring_idx)
                        return -ENOMEM;
                }
 
-               memset(rtlpci->rx_ring[rxring_idx].desc, 0,
-                      sizeof(*rtlpci->rx_ring[rxring_idx].desc) *
-                      rtlpci->rxringcount);
-
                /* init every desc in this ring */
                rtlpci->rx_ring[rxring_idx].idx = 0;
                for (i = 0; i < rtlpci->rxringcount; i++) {
index 50ece291fc6aef6040c08a869a0f0f6b353f155e..f35fa3dfe22c235c31411437b9d2120e7cc4a083 100644 (file)
@@ -1191,18 +1191,15 @@ static int slic_rspqueue_init(struct adapter *adapter)
        rspq->num_pages = SLIC_RSPQ_PAGES_GB;
 
        for (i = 0; i < rspq->num_pages; i++) {
-               rspq->vaddr[i] = pci_alloc_consistent(adapter->pcidev,
-                                                     PAGE_SIZE,
-                                                     &rspq->paddr[i]);
+               rspq->vaddr[i] = pci_zalloc_consistent(adapter->pcidev,
+                                                      PAGE_SIZE,
+                                                      &rspq->paddr[i]);
                if (!rspq->vaddr[i]) {
                        dev_err(&adapter->pcidev->dev,
                                "pci_alloc_consistent failed\n");
                        slic_rspqueue_free(adapter);
                        return -ENOMEM;
                }
-               /* FIXME:
-                * do we really need this assertions (4K PAGE_SIZE aligned addr)? */
-               memset(rspq->vaddr[i], 0, PAGE_SIZE);
 
                if (paddrh == 0) {
                        slic_reg32_write(&slic_regs->slic_rbar,
index c78d06eff7ea4501c4aa210fa18aed1b33de6a9c..0b583a37f5b3ddb9c8a902b1a67b86c4a4cdc004 100644 (file)
@@ -1111,25 +1111,17 @@ static bool device_init_rings(PSDevice pDevice)
        void *vir_pool;
 
        /*allocate all RD/TD rings a single pool*/
-       vir_pool = pci_alloc_consistent(pDevice->pcid,
-                                       pDevice->sOpts.nRxDescs0 * sizeof(SRxDesc) +
-                                       pDevice->sOpts.nRxDescs1 * sizeof(SRxDesc) +
-                                       pDevice->sOpts.nTxDescs[0] * sizeof(STxDesc) +
-                                       pDevice->sOpts.nTxDescs[1] * sizeof(STxDesc),
-                                       &pDevice->pool_dma);
-
+       vir_pool = pci_zalloc_consistent(pDevice->pcid,
+                                        pDevice->sOpts.nRxDescs0 * sizeof(SRxDesc) +
+                                        pDevice->sOpts.nRxDescs1 * sizeof(SRxDesc) +
+                                        pDevice->sOpts.nTxDescs[0] * sizeof(STxDesc) +
+                                        pDevice->sOpts.nTxDescs[1] * sizeof(STxDesc),
+                                        &pDevice->pool_dma);
        if (vir_pool == NULL) {
                DBG_PRT(MSG_LEVEL_ERR, KERN_ERR "%s : allocate desc dma memory failed\n", pDevice->dev->name);
                return false;
        }
 
-       memset(vir_pool, 0,
-              pDevice->sOpts.nRxDescs0 * sizeof(SRxDesc) +
-              pDevice->sOpts.nRxDescs1 * sizeof(SRxDesc) +
-              pDevice->sOpts.nTxDescs[0] * sizeof(STxDesc) +
-              pDevice->sOpts.nTxDescs[1] * sizeof(STxDesc)
-               );
-
        pDevice->aRD0Ring = vir_pool;
        pDevice->aRD1Ring = vir_pool +
                pDevice->sOpts.nRxDescs0 * sizeof(SRxDesc);
@@ -1138,13 +1130,12 @@ static bool device_init_rings(PSDevice pDevice)
        pDevice->rd1_pool_dma = pDevice->rd0_pool_dma +
                pDevice->sOpts.nRxDescs0 * sizeof(SRxDesc);
 
-       pDevice->tx0_bufs = pci_alloc_consistent(pDevice->pcid,
-                                                pDevice->sOpts.nTxDescs[0] * PKT_BUF_SZ +
-                                                pDevice->sOpts.nTxDescs[1] * PKT_BUF_SZ +
-                                                CB_BEACON_BUF_SIZE +
-                                                CB_MAX_BUF_SIZE,
-                                                &pDevice->tx_bufs_dma0);
-
+       pDevice->tx0_bufs = pci_zalloc_consistent(pDevice->pcid,
+                                                 pDevice->sOpts.nTxDescs[0] * PKT_BUF_SZ +
+                                                 pDevice->sOpts.nTxDescs[1] * PKT_BUF_SZ +
+                                                 CB_BEACON_BUF_SIZE +
+                                                 CB_MAX_BUF_SIZE,
+                                                 &pDevice->tx_bufs_dma0);
        if (pDevice->tx0_bufs == NULL) {
                DBG_PRT(MSG_LEVEL_ERR, KERN_ERR "%s: allocate buf dma memory failed\n", pDevice->dev->name);
                pci_free_consistent(pDevice->pcid,
@@ -1157,13 +1148,6 @@ static bool device_init_rings(PSDevice pDevice)
                return false;
        }
 
-       memset(pDevice->tx0_bufs, 0,
-              pDevice->sOpts.nTxDescs[0] * PKT_BUF_SZ +
-              pDevice->sOpts.nTxDescs[1] * PKT_BUF_SZ +
-              CB_BEACON_BUF_SIZE +
-              CB_MAX_BUF_SIZE
-               );
-
        pDevice->td0_pool_dma = pDevice->rd1_pool_dma +
                pDevice->sOpts.nRxDescs1 * sizeof(SRxDesc);