staging: frontier: tranzport.c: remove err() usage
err() was a very old USB-specific macro that I thought had gone away. This patch removes it from being used in the driver and uses dev_err() instead. CC: "David Täht" <d@teklibre.com> CC: Hitoshi Nakamori <hitoshi.nakamori@gmail.com> CC: "Ken O'Brien" <kernel@kenobrien.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
aa8f827a4d
commit
aec238825f
|
@ -353,7 +353,7 @@ static int usb_tranzport_open(struct inode *inode, struct file *file)
|
||||||
interface = usb_find_interface(&usb_tranzport_driver, subminor);
|
interface = usb_find_interface(&usb_tranzport_driver, subminor);
|
||||||
|
|
||||||
if (!interface) {
|
if (!interface) {
|
||||||
err("%s - error, can't find device for minor %d\n",
|
printk(KERN_ERR "%s - error, can't find device for minor %d\n",
|
||||||
__func__, subminor);
|
__func__, subminor);
|
||||||
retval = -ENODEV;
|
retval = -ENODEV;
|
||||||
goto unlock_disconnect_exit;
|
goto unlock_disconnect_exit;
|
||||||
|
@ -517,9 +517,11 @@ static ssize_t usb_tranzport_read(struct file *file, char __user *buffer,
|
||||||
goto exit;
|
goto exit;
|
||||||
}
|
}
|
||||||
|
|
||||||
/* verify that the device wasn't unplugged */ if (dev->intf == NULL) {
|
/* verify that the device wasn't unplugged */
|
||||||
|
if (dev->intf == NULL) {
|
||||||
retval = -ENODEV;
|
retval = -ENODEV;
|
||||||
err("No device or device unplugged %d\n", retval);
|
printk(KERN_ERR "%s: No device or device unplugged %d\n",
|
||||||
|
__func__, retval);
|
||||||
goto unlock_exit;
|
goto unlock_exit;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -691,7 +693,8 @@ static ssize_t usb_tranzport_write(struct file *file,
|
||||||
/* verify that the device wasn't unplugged */
|
/* verify that the device wasn't unplugged */
|
||||||
if (dev->intf == NULL) {
|
if (dev->intf == NULL) {
|
||||||
retval = -ENODEV;
|
retval = -ENODEV;
|
||||||
err("No device or device unplugged %d\n", retval);
|
printk(KERN_ERR "%s: No device or device unplugged %d\n",
|
||||||
|
__func__, retval);
|
||||||
goto unlock_exit;
|
goto unlock_exit;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -726,7 +729,7 @@ static ssize_t usb_tranzport_write(struct file *file,
|
||||||
}
|
}
|
||||||
|
|
||||||
if (dev->interrupt_out_endpoint == NULL) {
|
if (dev->interrupt_out_endpoint == NULL) {
|
||||||
err("Endpoint should not be be null!\n");
|
dev_err(&dev->intf->dev, "Endpoint should not be be null!\n");
|
||||||
goto unlock_exit;
|
goto unlock_exit;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -746,7 +749,8 @@ static ssize_t usb_tranzport_write(struct file *file,
|
||||||
retval = usb_submit_urb(dev->interrupt_out_urb, GFP_KERNEL);
|
retval = usb_submit_urb(dev->interrupt_out_urb, GFP_KERNEL);
|
||||||
if (retval) {
|
if (retval) {
|
||||||
dev->interrupt_out_busy = 0;
|
dev->interrupt_out_busy = 0;
|
||||||
err("Couldn't submit interrupt_out_urb %d\n", retval);
|
dev_err(&dev->intf->dev,
|
||||||
|
"Couldn't submit interrupt_out_urb %d\n", retval);
|
||||||
goto unlock_exit;
|
goto unlock_exit;
|
||||||
}
|
}
|
||||||
retval = bytes_to_write;
|
retval = bytes_to_write;
|
||||||
|
|
Loading…
Reference in New Issue