|
Oliver Neukum |
31c9b8 |
From 0b86f02d91c82c8348a8f3be960486e14a3dd479 Mon Sep 17 00:00:00 2001
|
|
Oliver Neukum |
31c9b8 |
From: kernel test robot <lkp@intel.com>
|
|
Oliver Neukum |
31c9b8 |
Date: Wed, 16 Feb 2022 11:51:53 +0200
|
|
Oliver Neukum |
31c9b8 |
Subject: [PATCH] usb: xhci: fix minmax.cocci warnings
|
|
Oliver Neukum |
31c9b8 |
Git-commit: 0b86f02d91c82c8348a8f3be960486e14a3dd479
|
|
Oliver Neukum |
31c9b8 |
References: jsc#PED-531
|
|
Oliver Neukum |
31c9b8 |
Patch-mainline: v5.18-rc1
|
|
Oliver Neukum |
31c9b8 |
|
|
Oliver Neukum |
31c9b8 |
Simplify the code using max().
|
|
Oliver Neukum |
31c9b8 |
|
|
Oliver Neukum |
31c9b8 |
Generated by: scripts/coccinelle/misc/minmax.cocci
|
|
Oliver Neukum |
31c9b8 |
|
|
Oliver Neukum |
31c9b8 |
Reported-by: kernel test robot <lkp@intel.com>
|
|
Oliver Neukum |
31c9b8 |
Signed-off-by: kernel test robot <lkp@intel.com>
|
|
Oliver Neukum |
31c9b8 |
Signed-off-by: Julia Lawall <julia.lawall@inria.fr>
|
|
Oliver Neukum |
31c9b8 |
Signed-off-by: Mathias Nyman <mathias.nyman@linux.intel.com>
|
|
Oliver Neukum |
31c9b8 |
Link: https://lore.kernel.org/r/20220216095153.1303105-10-mathias.nyman@linux.intel.com
|
|
Oliver Neukum |
31c9b8 |
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
|
Oliver Neukum |
31c9b8 |
Signed-off-by: Oliver Neukum <oneukum@suse.com>
|
|
Oliver Neukum |
31c9b8 |
---
|
|
Oliver Neukum |
31c9b8 |
drivers/usb/host/xhci-mem.c | 3 +--
|
|
Oliver Neukum |
31c9b8 |
1 file changed, 1 insertion(+), 2 deletions(-)
|
|
Oliver Neukum |
31c9b8 |
|
|
Oliver Neukum |
31c9b8 |
diff --git a/drivers/usb/host/xhci-mem.c b/drivers/usb/host/xhci-mem.c
|
|
Oliver Neukum |
31c9b8 |
index 7a2dce730e9a..f8c2b6c79543 100644
|
|
Oliver Neukum |
31c9b8 |
--- a/drivers/usb/host/xhci-mem.c
|
|
Oliver Neukum |
31c9b8 |
+++ b/drivers/usb/host/xhci-mem.c
|
|
Oliver Neukum |
31c9b8 |
@@ -433,8 +433,7 @@ int xhci_ring_expansion(struct xhci_hcd *xhci, struct xhci_ring *ring,
|
|
Oliver Neukum |
31c9b8 |
(TRBS_PER_SEGMENT - 1);
|
|
Oliver Neukum |
31c9b8 |
|
|
Oliver Neukum |
31c9b8 |
/* Allocate number of segments we needed, or double the ring size */
|
|
Oliver Neukum |
31c9b8 |
- num_segs = ring->num_segs > num_segs_needed ?
|
|
Oliver Neukum |
31c9b8 |
- ring->num_segs : num_segs_needed;
|
|
Oliver Neukum |
31c9b8 |
+ num_segs = max(ring->num_segs, num_segs_needed);
|
|
Oliver Neukum |
31c9b8 |
|
|
Oliver Neukum |
31c9b8 |
ret = xhci_alloc_segments_for_ring(xhci, &first, &last,
|
|
Oliver Neukum |
31c9b8 |
num_segs, ring->cycle_state, ring->type,
|
|
Oliver Neukum |
31c9b8 |
--
|
|
Oliver Neukum |
31c9b8 |
2.35.3
|
|
Oliver Neukum |
31c9b8 |
|