usb: usbtmc: Fix timeout value in get_stb
authorDave Penkler <dpenkler@gmail.com>
Wed, 21 May 2025 12:16:56 +0000 (14:16 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 21 May 2025 13:01:09 +0000 (15:01 +0200)
wait_event_interruptible_timeout requires a timeout argument
in units of jiffies. It was being called in usbtmc_get_stb
with the usb timeout value which is in units of milliseconds.

Pass the timeout argument converted to jiffies.

Fixes: 048c6d88a021 ("usb: usbtmc: Add ioctls to set/get usb timeout")
Cc: stable@vger.kernel.org
Signed-off-by: Dave Penkler <dpenkler@gmail.com>
Link: https://lore.kernel.org/r/20250521121656.18174-4-dpenkler@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/class/usbtmc.c

index 08511442a27f9d8359f7ef10d04df10d9466aa0e..75de29725a450cd17ad8f6da8794d7f619bd0c27 100644 (file)
@@ -483,6 +483,7 @@ static int usbtmc_get_stb(struct usbtmc_file_data *file_data, __u8 *stb)
        u8 tag;
        int rv;
        long wait_rv;
+       unsigned long expire;
 
        dev_dbg(dev, "Enter ioctl_read_stb iin_ep_present: %d\n",
                data->iin_ep_present);
@@ -512,10 +513,11 @@ static int usbtmc_get_stb(struct usbtmc_file_data *file_data, __u8 *stb)
        }
 
        if (data->iin_ep_present) {
+               expire = msecs_to_jiffies(file_data->timeout);
                wait_rv = wait_event_interruptible_timeout(
                        data->waitq,
                        atomic_read(&data->iin_data_valid) != 0,
-                       file_data->timeout);
+                       expire);
                if (wait_rv < 0) {
                        dev_dbg(dev, "wait interrupted %ld\n", wait_rv);
                        rv = wait_rv;