[PATCH 32/35] media: si4713: Refator return path
Ricardo Ribalda
ribalda at chromium.org
Mon Apr 15 12:34:49 PDT 2024
This is a nop, but let cocci now that this is not a good candidate for
min()
drivers/media/radio/si4713/radio-usb-si4713.c:309:15-16: WARNING opportunity for min()
drivers/media/radio/si4713/radio-usb-si4713.c:360:15-16: WARNING opportunity for min()
Signed-off-by: Ricardo Ribalda <ribalda at chromium.org>
---
drivers/media/radio/si4713/radio-usb-si4713.c | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/drivers/media/radio/si4713/radio-usb-si4713.c b/drivers/media/radio/si4713/radio-usb-si4713.c
index 2cf36c8abdde..0ad1bfe09004 100644
--- a/drivers/media/radio/si4713/radio-usb-si4713.c
+++ b/drivers/media/radio/si4713/radio-usb-si4713.c
@@ -306,7 +306,9 @@ static int send_command(struct si4713_usb_device *radio, u8 *payload, char *data
0x09, 0x21, 0x033f, 0, radio->buffer,
BUFFER_LENGTH, USB_TIMEOUT);
- return retval < 0 ? retval : 0;
+ if (retval < 0)
+ return retval;
+ return 0;
}
static int si4713_i2c_read(struct si4713_usb_device *radio, char *data, int len)
@@ -357,7 +359,9 @@ static int si4713_i2c_write(struct si4713_usb_device *radio, char *data, int len
data, len);
}
- return retval < 0 ? retval : 0;
+ if (retval < 0)
+ return retval;
+ return 0;
}
static int si4713_transfer(struct i2c_adapter *i2c_adapter,
--
2.44.0.683.g7961c838ac-goog
More information about the Linux-mediatek
mailing list