media: au0828: Use umin macro
authorRicardo Ribalda <ribalda@chromium.org>
Mon, 29 Apr 2024 15:04:55 +0000 (16:04 +0100)
committerMauro Carvalho Chehab <mchehab@kernel.org>
Fri, 3 May 2024 10:13:02 +0000 (11:13 +0100)
Simplifies the code.

Found by cocci:
drivers/media/usb/au0828/au0828-video.c:605:11-12: WARNING opportunity for min()

Link: https://lore.kernel.org/linux-media/20240429-fix-cocci-v3-16-3c4865f5a4b0@chromium.org
Signed-off-by: Ricardo Ribalda <ribalda@chromium.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
drivers/media/usb/au0828/au0828-video.c

index fd9fc43d47e0feacfa98d6d9603ebba95ed255ab..2ec49ea479d56ec986b52fa4f4979c83c8f46c36 100644 (file)
@@ -602,10 +602,7 @@ static inline int au0828_isoc_copy(struct au0828_dev *dev, struct urb *urb)
                vbi_field_size = dev->vbi_width * dev->vbi_height * 2;
                if (dev->vbi_read < vbi_field_size) {
                        remain  = vbi_field_size - dev->vbi_read;
-                       if (len < remain)
-                               lencopy = len;
-                       else
-                               lencopy = remain;
+                       lencopy = umin(len, remain);
 
                        if (vbi_buf != NULL)
                                au0828_copy_vbi(dev, vbi_dma_q, vbi_buf, p,