diff options
author | Ricardo Ribalda <[email protected]> | 2024-04-29 16:04:55 +0100 |
---|---|---|
committer | Mauro Carvalho Chehab <[email protected]> | 2024-05-03 11:13:02 +0100 |
commit | 657f5bca97af98a3da6c0d0c2a51f1f9be9b0e41 (patch) | |
tree | 1827b8aa78a407c3632c09077749478bb193302f | |
parent | f2ccb530c5f9bebb0793d4f173a27ad709fcabc4 (diff) |
media: au0828: Use umin macro
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/[email protected]
Signed-off-by: Ricardo Ribalda <[email protected]>
Signed-off-by: Mauro Carvalho Chehab <[email protected]>
-rw-r--r-- | drivers/media/usb/au0828/au0828-video.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/media/usb/au0828/au0828-video.c b/drivers/media/usb/au0828/au0828-video.c index fd9fc43d47e0..2ec49ea479d5 100644 --- a/drivers/media/usb/au0828/au0828-video.c +++ b/drivers/media/usb/au0828/au0828-video.c @@ -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, |