On Mon Mar 30 12:11:36 2026 +0200, Johan Hovold wrote:
> Use the common USB helper for looking up interrupt-in endpoints instead
> of open coding.
> 
> Signed-off-by: Johan Hovold <[email protected]>
> Signed-off-by: Hans Verkuil <[email protected]>

Patch committed.

Thanks,
Hans Verkuil

 drivers/media/radio/si470x/radio-si470x-usb.c | 11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)

---

diff --git a/drivers/media/radio/si470x/radio-si470x-usb.c 
b/drivers/media/radio/si470x/radio-si470x-usb.c
index 318b5f6d4202..869b1e7e34b9 100644
--- a/drivers/media/radio/si470x/radio-si470x-usb.c
+++ b/drivers/media/radio/si470x/radio-si470x-usb.c
@@ -565,8 +565,7 @@ static int si470x_usb_driver_probe(struct usb_interface 
*intf,
 {
        struct si470x_device *radio;
        struct usb_host_interface *iface_desc;
-       struct usb_endpoint_descriptor *endpoint;
-       int i, int_end_size, retval;
+       int int_end_size, retval;
        unsigned char version_warning = 0;
 
        /* private data allocation and initialization */
@@ -595,12 +594,8 @@ static int si470x_usb_driver_probe(struct usb_interface 
*intf,
        iface_desc = intf->cur_altsetting;
 
        /* Set up interrupt endpoint information. */
-       for (i = 0; i < iface_desc->desc.bNumEndpoints; ++i) {
-               endpoint = &iface_desc->endpoint[i].desc;
-               if (usb_endpoint_is_int_in(endpoint))
-                       radio->int_in_endpoint = endpoint;
-       }
-       if (!radio->int_in_endpoint) {
+       retval = usb_find_int_in_endpoint(iface_desc, &radio->int_in_endpoint);
+       if (retval) {
                dev_info(&intf->dev, "could not find interrupt in endpoint\n");
                retval = -EIO;
                goto err_usbbuf;
_______________________________________________
linuxtv-commits mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to