Use *_pool_zalloc rather than *_pool_alloc followed by memset with 0.
Found by coccinelle spatch "api/alloc/pool_zalloc-simple.cocci"

Signed-off-by: Thomas Meyer <tho...@m3y3r.de>
---

diff -u -p a/drivers/mailbox/bcm-flexrm-mailbox.c 
b/drivers/mailbox/bcm-flexrm-mailbox.c
--- a/drivers/mailbox/bcm-flexrm-mailbox.c
+++ b/drivers/mailbox/bcm-flexrm-mailbox.c
@@ -1258,14 +1258,13 @@ static int flexrm_startup(struct mbox_ch
        }
 
        /* Allocate completion memory */
-       ring->cmpl_base = dma_pool_alloc(ring->mbox->cmpl_pool,
-                                        GFP_KERNEL, &ring->cmpl_dma_base);
+       ring->cmpl_base = dma_pool_zalloc(ring->mbox->cmpl_pool, GFP_KERNEL,
+                                         &ring->cmpl_dma_base);
        if (!ring->cmpl_base) {
                dev_err(ring->mbox->dev, "can't allocate completion memory\n");
                ret = -ENOMEM;
                goto fail_free_bd_memory;
        }
-       memset(ring->cmpl_base, 0, RING_CMPL_SIZE);
 
        /* Request IRQ */
        if (ring->irq == UINT_MAX) {

Reply via email to