xhci: mem: Drop useless return:s
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Fri, 17 Mar 2023 15:47:05 +0000 (17:47 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 23 Mar 2023 16:25:22 +0000 (17:25 +0100)
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: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Mathias Nyman <mathias.nyman@linux.intel.com>
Link: https://lore.kernel.org/r/20230317154715.535523-5-mathias.nyman@linux.intel.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/host/xhci-mem.c

index 357883256a5a4ed9beb992e31bf77f3be3c48416..fa0c4ac2ca7fd3b410e5118fe4616568cfa042f5 100644 (file)
@@ -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);
 }
 
 /*