Commit 53ee2663 authored by Andy Shevchenko's avatar Andy Shevchenko Committed by Greg Kroah-Hartman
Browse files

xhci: mem: Use dma_poll_zalloc() instead of explicit memset() 



Use dma_poll_zalloc() instead of explicit memset() call in
xhci_alloc_stream_ctx(). Note, that dma_alloc_coherent() is
always issues zeroed memory chunk.

Signed-off-by: default avatarAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: default avatarMathias Nyman <mathias.nyman@linux.intel.com>
Link: https://lore.kernel.org/r/20230317154715.535523-3-mathias.nyman@linux.intel.com


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 34728498

drivers/usb/host/xhci-mem.c

+2 −4
Original line number Diff line number Diff line
@@ -575,10 +575,10 @@ static struct xhci_stream_ctx *xhci_alloc_stream_ctx(struct xhci_hcd *xhci, 
		return dma_alloc_coherent(dev, size,

				dma, mem_flags);

	else if (size <= SMALL_STREAM_ARRAY_SIZE)

		return dma_pool_alloc(xhci->small_streams_pool,

		return dma_pool_zalloc(xhci->small_streams_pool,

				mem_flags, dma);

	else

		return dma_pool_alloc(xhci->medium_streams_pool,

		return dma_pool_zalloc(xhci->medium_streams_pool,

				mem_flags, dma);

}
@@ -643,8 +643,6 @@ struct xhci_stream_info *xhci_alloc_stream_info(struct xhci_hcd *xhci, 
			mem_flags);

	if (!stream_info->stream_ctx_array)

		goto cleanup_ring_array;

	memset(stream_info->stream_ctx_array, 0,

			sizeof(struct xhci_stream_ctx)*num_stream_ctxs);



	/* Allocate everything needed to free the stream rings later */

	stream_info->free_streams_command =