usb: host: ehci: Use dma_pool_zalloc()
authorSouptick Joarder <jrdr.linux@gmail.com>
Wed, 14 Feb 2018 17:48:48 +0000 (23:18 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 22 Feb 2018 14:17:05 +0000 (15:17 +0100)
Use dma_pool_zalloc() instead of dma_pool_alloc + memset

Signed-off-by: Souptick Joarder <jrdr.linux@gmail.com>
Acked-by: Alan Stern <stern@rowland.harvard.edu>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/host/ehci-mem.c
drivers/usb/host/ehci-sched.c

index 21307d862af674bfb995577f5046b713304d5ddc..4c6c08b675b5c70f53c3815a460cd1b90d929678 100644 (file)
@@ -73,10 +73,9 @@ static struct ehci_qh *ehci_qh_alloc (struct ehci_hcd *ehci, gfp_t flags)
        if (!qh)
                goto done;
        qh->hw = (struct ehci_qh_hw *)
-               dma_pool_alloc(ehci->qh_pool, flags, &dma);
+               dma_pool_zalloc(ehci->qh_pool, flags, &dma);
        if (!qh->hw)
                goto fail;
-       memset(qh->hw, 0, sizeof *qh->hw);
        qh->qh_dma = dma;
        // INIT_LIST_HEAD (&qh->qh_list);
        INIT_LIST_HEAD (&qh->qtd_list);
index e56db44708bccd86ac43a870d51fc039224305c8..28e2a338b48153496485bbf29b448f63a404e9e4 100644 (file)
@@ -1287,7 +1287,7 @@ itd_urb_transaction(
                } else {
  alloc_itd:
                        spin_unlock_irqrestore(&ehci->lock, flags);
-                       itd = dma_pool_alloc(ehci->itd_pool, mem_flags,
+                       itd = dma_pool_zalloc(ehci->itd_pool, mem_flags,
                                        &itd_dma);
                        spin_lock_irqsave(&ehci->lock, flags);
                        if (!itd) {
@@ -1297,7 +1297,6 @@ itd_urb_transaction(
                        }
                }
 
-               memset(itd, 0, sizeof(*itd));
                itd->itd_dma = itd_dma;
                itd->frame = NO_FRAME;
                list_add(&itd->itd_list, &sched->td_list);
@@ -2081,7 +2080,7 @@ sitd_urb_transaction(
                } else {
  alloc_sitd:
                        spin_unlock_irqrestore(&ehci->lock, flags);
-                       sitd = dma_pool_alloc(ehci->sitd_pool, mem_flags,
+                       sitd = dma_pool_zalloc(ehci->sitd_pool, mem_flags,
                                        &sitd_dma);
                        spin_lock_irqsave(&ehci->lock, flags);
                        if (!sitd) {
@@ -2091,7 +2090,6 @@ sitd_urb_transaction(
                        }
                }
 
-               memset(sitd, 0, sizeof(*sitd));
                sitd->sitd_dma = sitd_dma;
                sitd->frame = NO_FRAME;
                list_add(&sitd->sitd_list, &iso_sched->td_list);