From: Peter Chen <[email protected]>

We already have sp_array to store each scratch buffer address for xHC,
it doesn't need another sp_dma_buffers array to store it.

Signed-off-by: Peter Chen <[email protected]>
Signed-off-by: Mathias Nyman <[email protected]>
---
 drivers/usb/host/xhci-mem.c | 18 ++++--------------
 drivers/usb/host/xhci.h     |  1 -
 2 files changed, 4 insertions(+), 15 deletions(-)

diff --git a/drivers/usb/host/xhci-mem.c b/drivers/usb/host/xhci-mem.c
index 2481335..f80bd03 100644
--- a/drivers/usb/host/xhci-mem.c
+++ b/drivers/usb/host/xhci-mem.c
@@ -1721,36 +1721,27 @@ static int scratchpad_alloc(struct xhci_hcd *xhci, 
gfp_t flags)
        if (!xhci->scratchpad->sp_buffers)
                goto fail_sp3;
 
-       xhci->scratchpad->sp_dma_buffers =
-               kzalloc(sizeof(dma_addr_t) * num_sp, flags);
-
-       if (!xhci->scratchpad->sp_dma_buffers)
-               goto fail_sp4;
-
        xhci->dcbaa->dev_context_ptrs[0] = 
cpu_to_le64(xhci->scratchpad->sp_dma);
        for (i = 0; i < num_sp; i++) {
                dma_addr_t dma;
                void *buf = dma_alloc_coherent(dev, xhci->page_size, &dma,
                                flags);
                if (!buf)
-                       goto fail_sp5;
+                       goto fail_sp4;
 
                xhci->scratchpad->sp_array[i] = dma;
                xhci->scratchpad->sp_buffers[i] = buf;
-               xhci->scratchpad->sp_dma_buffers[i] = dma;
        }
 
        return 0;
 
- fail_sp5:
+ fail_sp4:
        for (i = i - 1; i >= 0; i--) {
                dma_free_coherent(dev, xhci->page_size,
                                    xhci->scratchpad->sp_buffers[i],
-                                   xhci->scratchpad->sp_dma_buffers[i]);
+                                   xhci->scratchpad->sp_array[i]);
        }
-       kfree(xhci->scratchpad->sp_dma_buffers);
 
- fail_sp4:
        kfree(xhci->scratchpad->sp_buffers);
 
  fail_sp3:
@@ -1780,9 +1771,8 @@ static void scratchpad_free(struct xhci_hcd *xhci)
        for (i = 0; i < num_sp; i++) {
                dma_free_coherent(dev, xhci->page_size,
                                    xhci->scratchpad->sp_buffers[i],
-                                   xhci->scratchpad->sp_dma_buffers[i]);
+                                   xhci->scratchpad->sp_array[i]);
        }
-       kfree(xhci->scratchpad->sp_dma_buffers);
        kfree(xhci->scratchpad->sp_buffers);
        dma_free_coherent(dev, num_sp * sizeof(u64),
                            xhci->scratchpad->sp_array,
diff --git a/drivers/usb/host/xhci.h b/drivers/usb/host/xhci.h
index 44123dd..cae8fb9 100644
--- a/drivers/usb/host/xhci.h
+++ b/drivers/usb/host/xhci.h
@@ -1606,7 +1606,6 @@ struct xhci_scratchpad {
        u64 *sp_array;
        dma_addr_t sp_dma;
        void **sp_buffers;
-       dma_addr_t *sp_dma_buffers;
 };
 
 struct urb_priv {
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to