niansa-misc/linux
1
0
Fork 0
mirror of synced 2025-03-06 20:59:54 +01:00
Code Issues Projects Releases Packages Wiki Activity

USB: FIx locks and urb->status in adutux (updated)

Browse source 
Two main issues fixed here are:
 - An improper use of in-struct lock to protect an open count
 - Use of urb status for -EINPROGRESS

Also, along the way:
 - Change usb_unlink_urb to usb_kill_urb. Apparently there's no need
   to use usb_unlink_urb whatsoever in this driver, and the old use of
   usb_kill_urb was outright racy (it unlinked and immediately freed).
 - Fix indentation in adu_write. Looks like it was damaged by a script.
 - Vitaly wants -EBUSY on multiply opens.
 - bInterval was taken from a wrong endpoint.

Signed-off-by: Pete Zaitcev <zaitcev@redhat.com>
Signed-off-by: Vitaliy Ivanov <vitalivanov@gmail.com>
Tested-by: Vitaliy Ivanov <vitalivanov@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This commit is contained in:
Pete Zaitcev 2007-10-31 15:59:30 -07:00 • committed by Greg Kroah-Hartman
parent 3c886c5048
commit f08812d5eb
1 changed files with 141 additions and 125 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

266
drivers/usb/misc/adutux.c
View file

@ -79,12 +79,22 @@ MODULE_DEVICE_TABLE(usb, device_table);
#define COMMAND_TIMEOUT (2*HZ) /* 60 second timeout for a command */ #define COMMAND_TIMEOUT (2*HZ) /* 60 second timeout for a command */
/*
* The locking scheme is a vanilla 3-lock:
* adu_device.buflock: A spinlock, covers what IRQs touch.
* adutux_mutex: A Static lock to cover open_count. It would also cover
* any globals, but we don't have them in 2.6.
* adu_device.mtx: A mutex to hold across sleepers like copy_from_user.
* It covers all of adu_device, except the open_count
* and what .buflock covers.
*/
/* Structure to hold all of our device specific stuff */ /* Structure to hold all of our device specific stuff */
struct adu_device { struct adu_device {
struct mutex mtx; /* locks this structure */ struct mutex mtx;
struct usb_device* udev; /* save off the usb device pointer */ struct usb_device* udev; /* save off the usb device pointer */
struct usb_interface* interface; struct usb_interface* interface;
unsigned char minor; /* the starting minor number for this device */ unsigned int minor; /* the starting minor number for this device */
char serial_number[8]; char serial_number[8];
int open_count; /* number of times this port has been opened */ int open_count; /* number of times this port has been opened */
@ -107,8 +117,11 @@ struct adu_device {
char* interrupt_out_buffer; char* interrupt_out_buffer;
struct usb_endpoint_descriptor* interrupt_out_endpoint; struct usb_endpoint_descriptor* interrupt_out_endpoint;
struct urb* interrupt_out_urb; struct urb* interrupt_out_urb;
int out_urb_finished;
}; };
static DEFINE_MUTEX(adutux_mutex);
static struct usb_driver adu_driver; static struct usb_driver adu_driver;
static void adu_debug_data(int level, const char *function, int size, static void adu_debug_data(int level, const char *function, int size,
@ -132,27 +145,31 @@ static void adu_debug_data(int level, const char *function, int size,
*/ */
static void adu_abort_transfers(struct adu_device *dev) static void adu_abort_transfers(struct adu_device *dev)
{ {
dbg(2," %s : enter", __FUNCTION__); unsigned long flags;
if (dev == NULL) { dbg(2," %s : enter", __FUNCTION__);
dbg(1," %s : dev is null", __FUNCTION__);
goto exit;
}
if (dev->udev == NULL) { if (dev->udev == NULL) {
dbg(1," %s : udev is null", __FUNCTION__); dbg(1," %s : udev is null", __FUNCTION__);
goto exit; goto exit;
} }
dbg(2," %s : udev state %d", __FUNCTION__, dev->udev->state);
if (dev->udev->state == USB_STATE_NOTATTACHED) {
dbg(1," %s : udev is not attached", __FUNCTION__);
goto exit;
}
/* shutdown transfer */ /* shutdown transfer */
usb_unlink_urb(dev->interrupt_in_urb);
usb_unlink_urb(dev->interrupt_out_urb); /* XXX Anchor these instead */
spin_lock_irqsave(&dev->buflock, flags);
if (!dev->read_urb_finished) {
spin_unlock_irqrestore(&dev->buflock, flags);
usb_kill_urb(dev->interrupt_in_urb);
} else
spin_unlock_irqrestore(&dev->buflock, flags);
spin_lock_irqsave(&dev->buflock, flags);
if (!dev->out_urb_finished) {
spin_unlock_irqrestore(&dev->buflock, flags);
usb_kill_urb(dev->interrupt_out_urb);
} else
spin_unlock_irqrestore(&dev->buflock, flags);
exit: exit:
dbg(2," %s : leave", __FUNCTION__); dbg(2," %s : leave", __FUNCTION__);
@ -162,8 +179,6 @@ static void adu_delete(struct adu_device *dev)
{ {
dbg(2, "%s enter", __FUNCTION__); dbg(2, "%s enter", __FUNCTION__);
adu_abort_transfers(dev);
/* free data structures */ /* free data structures */
usb_free_urb(dev->interrupt_in_urb); usb_free_urb(dev->interrupt_in_urb);
usb_free_urb(dev->interrupt_out_urb); usb_free_urb(dev->interrupt_out_urb);
@ -239,7 +254,10 @@ static void adu_interrupt_out_callback(struct urb *urb)
goto exit; goto exit;
} }
wake_up_interruptible(&dev->write_wait); spin_lock(&dev->buflock);
dev->out_urb_finished = 1;
wake_up(&dev->write_wait);
spin_unlock(&dev->buflock);
exit: exit:
adu_debug_data(5, __FUNCTION__, urb->actual_length, adu_debug_data(5, __FUNCTION__, urb->actual_length,
@ -252,12 +270,17 @@ static int adu_open(struct inode *inode, struct file *file)
struct adu_device *dev = NULL; struct adu_device *dev = NULL;
struct usb_interface *interface; struct usb_interface *interface;
int subminor; int subminor;
int retval = 0; int retval;
dbg(2,"%s : enter", __FUNCTION__); dbg(2,"%s : enter", __FUNCTION__);
subminor = iminor(inode); subminor = iminor(inode);
if ((retval = mutex_lock_interruptible(&adutux_mutex))) {
dbg(2, "%s : mutex lock failed", __FUNCTION__);
goto exit_no_lock;
}
interface = usb_find_interface(&adu_driver, subminor); interface = usb_find_interface(&adu_driver, subminor);
if (!interface) { if (!interface) {
err("%s - error, can't find device for minor %d", err("%s - error, can't find device for minor %d",
@ -267,54 +290,54 @@ static int adu_open(struct inode *inode, struct file *file)
} }
dev = usb_get_intfdata(interface); dev = usb_get_intfdata(interface);
if (!dev) { if (!dev || !dev->udev) {
retval = -ENODEV; retval = -ENODEV;
goto exit_no_device; goto exit_no_device;
} }
/* lock this device */ /* check that nobody else is using the device */
if ((retval = mutex_lock_interruptible(&dev->mtx))) { if (dev->open_count) {
dbg(2, "%s : mutex lock failed", __FUNCTION__); retval = -EBUSY;
goto exit_no_device; goto exit_no_device;
} }
/* increment our usage count for the device */
++dev->open_count; ++dev->open_count;
dbg(2,"%s : open count %d", __FUNCTION__, dev->open_count); dbg(2,"%s : open count %d", __FUNCTION__, dev->open_count);
/* save device in the file's private structure */ /* save device in the file's private structure */
file->private_data = dev; file->private_data = dev;
if (dev->open_count == 1) { /* initialize in direction */
/* initialize in direction */ dev->read_buffer_length = 0;
dev->read_buffer_length = 0;
/* fixup first read by having urb waiting for it */ /* fixup first read by having urb waiting for it */
usb_fill_int_urb(dev->interrupt_in_urb,dev->udev, usb_fill_int_urb(dev->interrupt_in_urb,dev->udev,
usb_rcvintpipe(dev->udev, usb_rcvintpipe(dev->udev,
dev->interrupt_in_endpoint->bEndpointAddress), dev->interrupt_in_endpoint->bEndpointAddress),
dev->interrupt_in_buffer, dev->interrupt_in_buffer,
le16_to_cpu(dev->interrupt_in_endpoint->wMaxPacketSize), le16_to_cpu(dev->interrupt_in_endpoint->wMaxPacketSize),
adu_interrupt_in_callback, dev, adu_interrupt_in_callback, dev,
dev->interrupt_in_endpoint->bInterval); dev->interrupt_in_endpoint->bInterval);
/* dev->interrupt_in_urb->transfer_flags |= URB_ASYNC_UNLINK; */ dev->read_urb_finished = 0;
dev->read_urb_finished = 0; if (usb_submit_urb(dev->interrupt_in_urb, GFP_KERNEL))
retval = usb_submit_urb(dev->interrupt_in_urb, GFP_KERNEL); dev->read_urb_finished = 1;
if (retval) /* we ignore failure */
--dev->open_count; /* end of fixup for first read */
}
mutex_unlock(&dev->mtx); /* initialize out direction */
dev->out_urb_finished = 1;
retval = 0;
exit_no_device: exit_no_device:
mutex_unlock(&adutux_mutex);
exit_no_lock:
dbg(2,"%s : leave, return value %d ", __FUNCTION__, retval); dbg(2,"%s : leave, return value %d ", __FUNCTION__, retval);
return retval; return retval;
} }
static int adu_release_internal(struct adu_device *dev) static void adu_release_internal(struct adu_device *dev)
{ {
int retval = 0;
dbg(2," %s : enter", __FUNCTION__); dbg(2," %s : enter", __FUNCTION__);
/* decrement our usage count for the device */ /* decrement our usage count for the device */
@ -326,12 +349,11 @@ static int adu_release_internal(struct adu_device *dev)
} }
dbg(2," %s : leave", __FUNCTION__); dbg(2," %s : leave", __FUNCTION__);
return retval;
} }
static int adu_release(struct inode *inode, struct file *file) static int adu_release(struct inode *inode, struct file *file)
{ {
struct adu_device *dev = NULL; struct adu_device *dev;
int retval = 0; int retval = 0;
dbg(2," %s : enter", __FUNCTION__); dbg(2," %s : enter", __FUNCTION__);
@ -343,15 +365,13 @@ static int adu_release(struct inode *inode, struct file *file)
} }
dev = file->private_data; dev = file->private_data;
if (dev == NULL) { if (dev == NULL) {
dbg(1," %s : object is NULL", __FUNCTION__); dbg(1," %s : object is NULL", __FUNCTION__);
retval = -ENODEV; retval = -ENODEV;
goto exit; goto exit;
} }
/* lock our device */ mutex_lock(&adutux_mutex); /* not interruptible */
mutex_lock(&dev->mtx); /* not interruptible */
if (dev->open_count <= 0) { if (dev->open_count <= 0) {
dbg(1," %s : device not opened", __FUNCTION__); dbg(1," %s : device not opened", __FUNCTION__);
@ -359,19 +379,15 @@ static int adu_release(struct inode *inode, struct file *file)
goto exit; goto exit;
} }
adu_release_internal(dev);
if (dev->udev == NULL) { if (dev->udev == NULL) {
/* the device was unplugged before the file was released */ /* the device was unplugged before the file was released */
mutex_unlock(&dev->mtx); if (!dev->open_count) /* ... and we're the last user */
adu_delete(dev); adu_delete(dev);
dev = NULL;
} else {
/* do the work */
retval = adu_release_internal(dev);
} }
exit: exit:
if (dev) mutex_unlock(&adutux_mutex);
mutex_unlock(&dev->mtx);
dbg(2," %s : leave, return value %d", __FUNCTION__, retval); dbg(2," %s : leave, return value %d", __FUNCTION__, retval);
return retval; return retval;
} }
@ -393,12 +409,12 @@ static ssize_t adu_read(struct file *file, __user char *buffer, size_t count,
dev = file->private_data; dev = file->private_data;
dbg(2," %s : dev=%p", __FUNCTION__, dev); dbg(2," %s : dev=%p", __FUNCTION__, dev);
/* lock this object */
if (mutex_lock_interruptible(&dev->mtx)) if (mutex_lock_interruptible(&dev->mtx))
return -ERESTARTSYS; return -ERESTARTSYS;
/* verify that the device wasn't unplugged */ /* verify that the device wasn't unplugged */
if (dev->udev == NULL || dev->minor == 0) { if (dev->udev == NULL) {
retval = -ENODEV; retval = -ENODEV;
err("No device or device unplugged %d", retval); err("No device or device unplugged %d", retval);
goto exit; goto exit;
@ -452,7 +468,7 @@ static ssize_t adu_read(struct file *file, __user char *buffer, size_t count,
should_submit = 1; should_submit = 1;
} else { } else {
/* even the primary was empty - we may need to do IO */ /* even the primary was empty - we may need to do IO */
if (dev->interrupt_in_urb->status == -EINPROGRESS) { if (!dev->read_urb_finished) {
/* somebody is doing IO */ /* somebody is doing IO */
spin_unlock_irqrestore(&dev->buflock, flags); spin_unlock_irqrestore(&dev->buflock, flags);
dbg(2," %s : submitted already", __FUNCTION__); dbg(2," %s : submitted already", __FUNCTION__);
@ -460,6 +476,7 @@ static ssize_t adu_read(struct file *file, __user char *buffer, size_t count,
/* we must initiate input */ /* we must initiate input */
dbg(2," %s : initiate input", __FUNCTION__); dbg(2," %s : initiate input", __FUNCTION__);
dev->read_urb_finished = 0; dev->read_urb_finished = 0;
spin_unlock_irqrestore(&dev->buflock, flags);
usb_fill_int_urb(dev->interrupt_in_urb,dev->udev, usb_fill_int_urb(dev->interrupt_in_urb,dev->udev,
usb_rcvintpipe(dev->udev, usb_rcvintpipe(dev->udev,
@ -469,15 +486,12 @@ static ssize_t adu_read(struct file *file, __user char *buffer, size_t count,
adu_interrupt_in_callback, adu_interrupt_in_callback,
dev, dev,
dev->interrupt_in_endpoint->bInterval); dev->interrupt_in_endpoint->bInterval);
retval = usb_submit_urb(dev->interrupt_in_urb, GFP_ATOMIC); retval = usb_submit_urb(dev->interrupt_in_urb, GFP_KERNEL);
if (!retval) { if (retval) {
spin_unlock_irqrestore(&dev->buflock, flags); dev->read_urb_finished = 1;
dbg(2," %s : submitted OK", __FUNCTION__);
} else {
if (retval == -ENOMEM) { if (retval == -ENOMEM) {
retval = bytes_read ? bytes_read : -ENOMEM; retval = bytes_read ? bytes_read : -ENOMEM;
} }
spin_unlock_irqrestore(&dev->buflock, flags);
dbg(2," %s : submit failed", __FUNCTION__); dbg(2," %s : submit failed", __FUNCTION__);
goto exit; goto exit;
} }
@ -486,10 +500,14 @@ static ssize_t adu_read(struct file *file, __user char *buffer, size_t count,
/* we wait for I/O to complete */ /* we wait for I/O to complete */
set_current_state(TASK_INTERRUPTIBLE); set_current_state(TASK_INTERRUPTIBLE);
add_wait_queue(&dev->read_wait, &wait); add_wait_queue(&dev->read_wait, &wait);
if (!dev->read_urb_finished) spin_lock_irqsave(&dev->buflock, flags);
if (!dev->read_urb_finished) {
spin_unlock_irqrestore(&dev->buflock, flags);
timeout = schedule_timeout(COMMAND_TIMEOUT); timeout = schedule_timeout(COMMAND_TIMEOUT);
else } else {
spin_unlock_irqrestore(&dev->buflock, flags);
set_current_state(TASK_RUNNING); set_current_state(TASK_RUNNING);
}
remove_wait_queue(&dev->read_wait, &wait); remove_wait_queue(&dev->read_wait, &wait);
if (timeout <= 0) { if (timeout <= 0) {
@ -509,19 +527,23 @@ static ssize_t adu_read(struct file *file, __user char *buffer, size_t count,
retval = bytes_read; retval = bytes_read;
/* if the primary buffer is empty then use it */ /* if the primary buffer is empty then use it */
if (should_submit && !dev->interrupt_in_urb->status==-EINPROGRESS) { spin_lock_irqsave(&dev->buflock, flags);
if (should_submit && dev->read_urb_finished) {
dev->read_urb_finished = 0;
spin_unlock_irqrestore(&dev->buflock, flags);
usb_fill_int_urb(dev->interrupt_in_urb,dev->udev, usb_fill_int_urb(dev->interrupt_in_urb,dev->udev,
usb_rcvintpipe(dev->udev, usb_rcvintpipe(dev->udev,
dev->interrupt_in_endpoint->bEndpointAddress), dev->interrupt_in_endpoint->bEndpointAddress),
dev->interrupt_in_buffer, dev->interrupt_in_buffer,
le16_to_cpu(dev->interrupt_in_endpoint->wMaxPacketSize), le16_to_cpu(dev->interrupt_in_endpoint->wMaxPacketSize),
adu_interrupt_in_callback, adu_interrupt_in_callback,
dev, dev,
dev->interrupt_in_endpoint->bInterval); dev->interrupt_in_endpoint->bInterval);
/* dev->interrupt_in_urb->transfer_flags |= URB_ASYNC_UNLINK; */ if (usb_submit_urb(dev->interrupt_in_urb, GFP_KERNEL) != 0)
dev->read_urb_finished = 0; dev->read_urb_finished = 1;
usb_submit_urb(dev->interrupt_in_urb, GFP_KERNEL);
/* we ignore failure */ /* we ignore failure */
} else {
spin_unlock_irqrestore(&dev->buflock, flags);
} }
exit: exit:
@ -535,24 +557,24 @@ exit:
static ssize_t adu_write(struct file *file, const __user char *buffer, static ssize_t adu_write(struct file *file, const __user char *buffer,
size_t count, loff_t *ppos) size_t count, loff_t *ppos)
{ {
DECLARE_WAITQUEUE(waita, current);
struct adu_device *dev; struct adu_device *dev;
size_t bytes_written = 0; size_t bytes_written = 0;
size_t bytes_to_write; size_t bytes_to_write;
size_t buffer_size; size_t buffer_size;
unsigned long flags;
int retval; int retval;
int timeout = 0;
dbg(2," %s : enter, count = %Zd", __FUNCTION__, count); dbg(2," %s : enter, count = %Zd", __FUNCTION__, count);
dev = file->private_data; dev = file->private_data;
/* lock this object */
retval = mutex_lock_interruptible(&dev->mtx); retval = mutex_lock_interruptible(&dev->mtx);
if (retval) if (retval)
goto exit_nolock; goto exit_nolock;
/* verify that the device wasn't unplugged */ /* verify that the device wasn't unplugged */
if (dev->udev == NULL || dev->minor == 0) { if (dev->udev == NULL) {
retval = -ENODEV; retval = -ENODEV;
err("No device or device unplugged %d", retval); err("No device or device unplugged %d", retval);
goto exit; goto exit;
@ -564,42 +586,37 @@ static ssize_t adu_write(struct file *file, const __user char *buffer,
goto exit; goto exit;
} }
while (count > 0) { while (count > 0) {
if (dev->interrupt_out_urb->status == -EINPROGRESS) { add_wait_queue(&dev->write_wait, &waita);
timeout = COMMAND_TIMEOUT; set_current_state(TASK_INTERRUPTIBLE);
spin_lock_irqsave(&dev->buflock, flags);
if (!dev->out_urb_finished) {
spin_unlock_irqrestore(&dev->buflock, flags);
while (timeout > 0) {
if (signal_pending(current)) {
dbg(1," %s : interrupted", __FUNCTION__);
retval = -EINTR;
goto exit;
}
mutex_unlock(&dev->mtx); mutex_unlock(&dev->mtx);
timeout = interruptible_sleep_on_timeout(&dev->write_wait, timeout); if (signal_pending(current)) {
dbg(1," %s : interrupted", __FUNCTION__);
set_current_state(TASK_RUNNING);
retval = -EINTR;
goto exit_onqueue;
}
if (schedule_timeout(COMMAND_TIMEOUT) == 0) {
dbg(1, "%s - command timed out.", __FUNCTION__);
retval = -ETIMEDOUT;
goto exit_onqueue;
}
remove_wait_queue(&dev->write_wait, &waita);
retval = mutex_lock_interruptible(&dev->mtx); retval = mutex_lock_interruptible(&dev->mtx);
if (retval) { if (retval) {
retval = bytes_written ? bytes_written : retval; retval = bytes_written ? bytes_written : retval;
goto exit_nolock; goto exit_nolock;
} }
if (timeout > 0) {
break;
}
dbg(1," %s : interrupted timeout: %d", __FUNCTION__, timeout);
}
dbg(1," %s : final timeout: %d", __FUNCTION__, timeout);
if (timeout == 0) {
dbg(1, "%s - command timed out.", __FUNCTION__);
retval = -ETIMEDOUT;
goto exit;
}
dbg(4," %s : in progress, count = %Zd", __FUNCTION__, count);
dbg(4," %s : in progress, count = %Zd", __FUNCTION__, count);
} else { } else {
spin_unlock_irqrestore(&dev->buflock, flags);
set_current_state(TASK_RUNNING);
remove_wait_queue(&dev->write_wait, &waita);
dbg(4," %s : sending, count = %Zd", __FUNCTION__, count); dbg(4," %s : sending, count = %Zd", __FUNCTION__, count);
/* write the data into interrupt_out_buffer from userspace */ /* write the data into interrupt_out_buffer from userspace */
@ -622,11 +639,12 @@ static ssize_t adu_write(struct file *file, const __user char *buffer,
bytes_to_write, bytes_to_write,
adu_interrupt_out_callback, adu_interrupt_out_callback,
dev, dev,
dev->interrupt_in_endpoint->bInterval); dev->interrupt_out_endpoint->bInterval);
/* dev->interrupt_in_urb->transfer_flags |= URB_ASYNC_UNLINK; */
dev->interrupt_out_urb->actual_length = bytes_to_write; dev->interrupt_out_urb->actual_length = bytes_to_write;
dev->out_urb_finished = 0;
retval = usb_submit_urb(dev->interrupt_out_urb, GFP_KERNEL); retval = usb_submit_urb(dev->interrupt_out_urb, GFP_KERNEL);
if (retval < 0) { if (retval < 0) {
dev->out_urb_finished = 1;
err("Couldn't submit interrupt_out_urb %d", retval); err("Couldn't submit interrupt_out_urb %d", retval);
goto exit; goto exit;
} }
@ -637,16 +655,17 @@ static ssize_t adu_write(struct file *file, const __user char *buffer,
bytes_written += bytes_to_write; bytes_written += bytes_to_write;
} }
} }
mutex_unlock(&dev->mtx);
retval = bytes_written; return bytes_written;
exit: exit:
/* unlock the device */
mutex_unlock(&dev->mtx); mutex_unlock(&dev->mtx);
exit_nolock: exit_nolock:
dbg(2," %s : leave, return value %d", __FUNCTION__, retval); dbg(2," %s : leave, return value %d", __FUNCTION__, retval);
return retval;
exit_onqueue:
remove_wait_queue(&dev->write_wait, &waita);
return retval; return retval;
} }
@ -831,25 +850,22 @@ static void adu_disconnect(struct usb_interface *interface)
dbg(2," %s : enter", __FUNCTION__); dbg(2," %s : enter", __FUNCTION__);
dev = usb_get_intfdata(interface); dev = usb_get_intfdata(interface);
usb_set_intfdata(interface, NULL);
mutex_lock(&dev->mtx); /* not interruptible */
dev->udev = NULL; /* poison */
minor = dev->minor; minor = dev->minor;
/* give back our minor */
usb_deregister_dev(interface, &adu_class); usb_deregister_dev(interface, &adu_class);
dev->minor = 0; mutex_unlock(&dev->mtx);
mutex_lock(&dev->mtx); /* not interruptible */ mutex_lock(&adutux_mutex);
usb_set_intfdata(interface, NULL);
/* if the device is not opened, then we clean up right now */ /* if the device is not opened, then we clean up right now */
dbg(2," %s : open count %d", __FUNCTION__, dev->open_count); dbg(2," %s : open count %d", __FUNCTION__, dev->open_count);
if (!dev->open_count) { if (!dev->open_count)
mutex_unlock(&dev->mtx);
adu_delete(dev); adu_delete(dev);
} else {
dev->udev = NULL; mutex_unlock(&adutux_mutex);
mutex_unlock(&dev->mtx);
}
dev_info(&interface->dev, "ADU device adutux%d now disconnected\n", dev_info(&interface->dev, "ADU device adutux%d now disconnected\n",
(minor - ADU_MINOR_BASE)); (minor - ADU_MINOR_BASE));