Commit 3056a5ca authored by Andy Shevchenko's avatar Andy Shevchenko Committed by Greg Kroah-Hartman
Browse files

xhci: mem: Drop useless return:s 



When function returns void and we have if-else-if chain, there is
no need to explicitly call return. Drop them and indent lines better.

While at it, make if-chain sorted from testing bigger values to smaller.

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-5-mathias.nyman@linux.intel.com


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

drivers/usb/host/xhci-mem.c

+4 −7
Original line number Diff line number Diff line
@@ -544,14 +544,11 @@ static void xhci_free_stream_ctx(struct xhci_hcd *xhci, 
	size_t size = sizeof(struct xhci_stream_ctx) * num_stream_ctxs;



	if (size > MEDIUM_STREAM_ARRAY_SIZE)

		dma_free_coherent(dev, size,

				stream_ctx, dma);

	else if (size <= SMALL_STREAM_ARRAY_SIZE)

		return dma_pool_free(xhci->small_streams_pool,

				stream_ctx, dma);

		dma_free_coherent(dev, size, stream_ctx, dma);

	else if (size > SMALL_STREAM_ARRAY_SIZE)

		dma_pool_free(xhci->medium_streams_pool, stream_ctx, dma);

	else

		return dma_pool_free(xhci->medium_streams_pool,

				stream_ctx, dma);

		dma_pool_free(xhci->small_streams_pool, stream_ctx, dma);

}



/*