USB: usblp.c: remove dbg() usage
dbg() was a very old USB-specific macro that should no longer be used. This patch removes it from being used in the driver and uses dev_dbg() instead. CC: Pete Zaitcev <zaitcev@redhat.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Родитель
4ed48e5e8b
Коммит
4f306309fc
|
@ -172,27 +172,31 @@ struct usblp {
|
||||||
#ifdef DEBUG
|
#ifdef DEBUG
|
||||||
static void usblp_dump(struct usblp *usblp)
|
static void usblp_dump(struct usblp *usblp)
|
||||||
{
|
{
|
||||||
|
struct device *dev = &usblp->intf->dev;
|
||||||
int p;
|
int p;
|
||||||
|
|
||||||
dbg("usblp=0x%p", usblp);
|
dev_dbg(dev, "usblp=0x%p\n", usblp);
|
||||||
dbg("dev=0x%p", usblp->dev);
|
dev_dbg(dev, "dev=0x%p\n", usblp->dev);
|
||||||
dbg("present=%d", usblp->present);
|
dev_dbg(dev, "present=%d\n", usblp->present);
|
||||||
dbg("readbuf=0x%p", usblp->readbuf);
|
dev_dbg(dev, "readbuf=0x%p\n", usblp->readbuf);
|
||||||
dbg("readcount=%d", usblp->readcount);
|
dev_dbg(dev, "readcount=%d\n", usblp->readcount);
|
||||||
dbg("ifnum=%d", usblp->ifnum);
|
dev_dbg(dev, "ifnum=%d\n", usblp->ifnum);
|
||||||
for (p = USBLP_FIRST_PROTOCOL; p <= USBLP_LAST_PROTOCOL; p++) {
|
for (p = USBLP_FIRST_PROTOCOL; p <= USBLP_LAST_PROTOCOL; p++) {
|
||||||
dbg("protocol[%d].alt_setting=%d", p, usblp->protocol[p].alt_setting);
|
dev_dbg(dev, "protocol[%d].alt_setting=%d\n", p,
|
||||||
dbg("protocol[%d].epwrite=%p", p, usblp->protocol[p].epwrite);
|
usblp->protocol[p].alt_setting);
|
||||||
dbg("protocol[%d].epread=%p", p, usblp->protocol[p].epread);
|
dev_dbg(dev, "protocol[%d].epwrite=%p\n", p,
|
||||||
}
|
usblp->protocol[p].epwrite);
|
||||||
dbg("current_protocol=%d", usblp->current_protocol);
|
dev_dbg(dev, "protocol[%d].epread=%p\n", p,
|
||||||
dbg("minor=%d", usblp->minor);
|
usblp->protocol[p].epread);
|
||||||
dbg("wstatus=%d", usblp->wstatus);
|
}
|
||||||
dbg("rstatus=%d", usblp->rstatus);
|
dev_dbg(dev, "current_protocol=%d\n", usblp->current_protocol);
|
||||||
dbg("quirks=%d", usblp->quirks);
|
dev_dbg(dev, "minor=%d\n", usblp->minor);
|
||||||
dbg("used=%d", usblp->used);
|
dev_dbg(dev, "wstatus=%d\n", usblp->wstatus);
|
||||||
dbg("bidir=%d", usblp->bidir);
|
dev_dbg(dev, "rstatus=%d\n", usblp->rstatus);
|
||||||
dbg("device_id_string=\"%s\"",
|
dev_dbg(dev, "quirks=%d\n", usblp->quirks);
|
||||||
|
dev_dbg(dev, "used=%d\n", usblp->used);
|
||||||
|
dev_dbg(dev, "bidir=%d\n", usblp->bidir);
|
||||||
|
dev_dbg(dev, "device_id_string=\"%s\"\n",
|
||||||
usblp->device_id_string ?
|
usblp->device_id_string ?
|
||||||
usblp->device_id_string + 2 :
|
usblp->device_id_string + 2 :
|
||||||
(unsigned char *)"(null)");
|
(unsigned char *)"(null)");
|
||||||
|
@ -262,7 +266,8 @@ static int usblp_ctrl_msg(struct usblp *usblp, int request, int type, int dir, i
|
||||||
retval = usb_control_msg(usblp->dev,
|
retval = usb_control_msg(usblp->dev,
|
||||||
dir ? usb_rcvctrlpipe(usblp->dev, 0) : usb_sndctrlpipe(usblp->dev, 0),
|
dir ? usb_rcvctrlpipe(usblp->dev, 0) : usb_sndctrlpipe(usblp->dev, 0),
|
||||||
request, type | dir | recip, value, index, buf, len, USBLP_CTL_TIMEOUT);
|
request, type | dir | recip, value, index, buf, len, USBLP_CTL_TIMEOUT);
|
||||||
dbg("usblp_control_msg: rq: 0x%02x dir: %d recip: %d value: %d idx: %d len: %#x result: %d",
|
dev_dbg(&usblp->intf->dev,
|
||||||
|
"usblp_control_msg: rq: 0x%02x dir: %d recip: %d value: %d idx: %d len: %#x result: %d\n",
|
||||||
request, !!dir, recip, value, index, len, retval);
|
request, !!dir, recip, value, index, len, retval);
|
||||||
return retval < 0 ? retval : 0;
|
return retval < 0 ? retval : 0;
|
||||||
}
|
}
|
||||||
|
@ -500,8 +505,9 @@ static long usblp_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
|
||||||
goto done;
|
goto done;
|
||||||
}
|
}
|
||||||
|
|
||||||
dbg("usblp_ioctl: cmd=0x%x (%c nr=%d len=%d dir=%d)", cmd, _IOC_TYPE(cmd),
|
dev_dbg(&usblp->intf->dev,
|
||||||
_IOC_NR(cmd), _IOC_SIZE(cmd), _IOC_DIR(cmd));
|
"usblp_ioctl: cmd=0x%x (%c nr=%d len=%d dir=%d)\n", cmd,
|
||||||
|
_IOC_TYPE(cmd), _IOC_NR(cmd), _IOC_SIZE(cmd), _IOC_DIR(cmd));
|
||||||
|
|
||||||
if (_IOC_TYPE(cmd) == 'P') /* new-style ioctl number */
|
if (_IOC_TYPE(cmd) == 'P') /* new-style ioctl number */
|
||||||
|
|
||||||
|
@ -594,7 +600,8 @@ static long usblp_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
|
||||||
goto done;
|
goto done;
|
||||||
}
|
}
|
||||||
|
|
||||||
dbg("usblp%d requested/got HP channel %ld/%d",
|
dev_dbg(&usblp->intf->dev,
|
||||||
|
"usblp%d requested/got HP channel %ld/%d\n",
|
||||||
usblp->minor, arg, newChannel);
|
usblp->minor, arg, newChannel);
|
||||||
break;
|
break;
|
||||||
|
|
||||||
|
@ -614,7 +621,8 @@ static long usblp_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
|
||||||
goto done;
|
goto done;
|
||||||
}
|
}
|
||||||
|
|
||||||
dbg("usblp%d is bus=%d, device=%d",
|
dev_dbg(&usblp->intf->dev,
|
||||||
|
"usblp%d is bus=%d, device=%d\n",
|
||||||
usblp->minor, twoints[0], twoints[1]);
|
usblp->minor, twoints[0], twoints[1]);
|
||||||
break;
|
break;
|
||||||
|
|
||||||
|
@ -634,7 +642,8 @@ static long usblp_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
|
||||||
goto done;
|
goto done;
|
||||||
}
|
}
|
||||||
|
|
||||||
dbg("usblp%d is VID=0x%4.4X, PID=0x%4.4X",
|
dev_dbg(&usblp->intf->dev,
|
||||||
|
"usblp%d is VID=0x%4.4X, PID=0x%4.4X\n",
|
||||||
usblp->minor, twoints[0], twoints[1]);
|
usblp->minor, twoints[0], twoints[1]);
|
||||||
break;
|
break;
|
||||||
|
|
||||||
|
@ -987,7 +996,7 @@ static int usblp_submit_read(struct usblp *usblp)
|
||||||
usblp->rcomplete = 0;
|
usblp->rcomplete = 0;
|
||||||
spin_unlock_irqrestore(&usblp->lock, flags);
|
spin_unlock_irqrestore(&usblp->lock, flags);
|
||||||
if ((rc = usb_submit_urb(urb, GFP_KERNEL)) < 0) {
|
if ((rc = usb_submit_urb(urb, GFP_KERNEL)) < 0) {
|
||||||
dbg("error submitting urb (%d)", rc);
|
dev_dbg(&usblp->intf->dev, "error submitting urb (%d)\n", rc);
|
||||||
spin_lock_irqsave(&usblp->lock, flags);
|
spin_lock_irqsave(&usblp->lock, flags);
|
||||||
usblp->rstatus = rc;
|
usblp->rstatus = rc;
|
||||||
usblp->rcomplete = 1;
|
usblp->rcomplete = 1;
|
||||||
|
@ -1129,7 +1138,8 @@ static int usblp_probe(struct usb_interface *intf,
|
||||||
/* Analyze and pick initial alternate settings and endpoints. */
|
/* Analyze and pick initial alternate settings and endpoints. */
|
||||||
protocol = usblp_select_alts(usblp);
|
protocol = usblp_select_alts(usblp);
|
||||||
if (protocol < 0) {
|
if (protocol < 0) {
|
||||||
dbg("incompatible printer-class device 0x%4.4X/0x%4.4X",
|
dev_dbg(&intf->dev,
|
||||||
|
"incompatible printer-class device 0x%4.4X/0x%4.4X\n",
|
||||||
le16_to_cpu(dev->descriptor.idVendor),
|
le16_to_cpu(dev->descriptor.idVendor),
|
||||||
le16_to_cpu(dev->descriptor.idProduct));
|
le16_to_cpu(dev->descriptor.idProduct));
|
||||||
retval = -ENODEV;
|
retval = -ENODEV;
|
||||||
|
@ -1158,14 +1168,14 @@ static int usblp_probe(struct usb_interface *intf,
|
||||||
|
|
||||||
retval = usb_register_dev(intf, &usblp_class);
|
retval = usb_register_dev(intf, &usblp_class);
|
||||||
if (retval) {
|
if (retval) {
|
||||||
printk(KERN_ERR "usblp: Not able to get a minor"
|
dev_err(&intf->dev,
|
||||||
" (base %u, slice default): %d\n",
|
"usblp: Not able to get a minor (base %u, slice default): %d\n",
|
||||||
USBLP_MINOR_BASE, retval);
|
USBLP_MINOR_BASE, retval);
|
||||||
goto abort_intfdata;
|
goto abort_intfdata;
|
||||||
}
|
}
|
||||||
usblp->minor = intf->minor;
|
usblp->minor = intf->minor;
|
||||||
printk(KERN_INFO "usblp%d: USB %sdirectional printer dev %d "
|
dev_info(&intf->dev,
|
||||||
"if %d alt %d proto %d vid 0x%4.4X pid 0x%4.4X\n",
|
"usblp%d: USB %sdirectional printer dev %d if %d alt %d proto %d vid 0x%4.4X pid 0x%4.4X\n",
|
||||||
usblp->minor, usblp->bidir ? "Bi" : "Uni", dev->devnum,
|
usblp->minor, usblp->bidir ? "Bi" : "Uni", dev->devnum,
|
||||||
usblp->ifnum,
|
usblp->ifnum,
|
||||||
usblp->protocol[usblp->current_protocol].alt_setting,
|
usblp->protocol[usblp->current_protocol].alt_setting,
|
||||||
|
@ -1302,7 +1312,8 @@ static int usblp_set_protocol(struct usblp *usblp, int protocol)
|
||||||
|
|
||||||
usblp->bidir = (usblp->protocol[protocol].epread != NULL);
|
usblp->bidir = (usblp->protocol[protocol].epread != NULL);
|
||||||
usblp->current_protocol = protocol;
|
usblp->current_protocol = protocol;
|
||||||
dbg("usblp%d set protocol %d", usblp->minor, protocol);
|
dev_dbg(&usblp->intf->dev, "usblp%d set protocol %d\n",
|
||||||
|
usblp->minor, protocol);
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -1315,7 +1326,8 @@ static int usblp_cache_device_id_string(struct usblp *usblp)
|
||||||
|
|
||||||
err = usblp_get_id(usblp, 0, usblp->device_id_string, USBLP_DEVICE_ID_SIZE - 1);
|
err = usblp_get_id(usblp, 0, usblp->device_id_string, USBLP_DEVICE_ID_SIZE - 1);
|
||||||
if (err < 0) {
|
if (err < 0) {
|
||||||
dbg("usblp%d: error = %d reading IEEE-1284 Device ID string",
|
dev_dbg(&usblp->intf->dev,
|
||||||
|
"usblp%d: error = %d reading IEEE-1284 Device ID string\n",
|
||||||
usblp->minor, err);
|
usblp->minor, err);
|
||||||
usblp->device_id_string[0] = usblp->device_id_string[1] = '\0';
|
usblp->device_id_string[0] = usblp->device_id_string[1] = '\0';
|
||||||
return -EIO;
|
return -EIO;
|
||||||
|
@ -1331,7 +1343,7 @@ static int usblp_cache_device_id_string(struct usblp *usblp)
|
||||||
length = USBLP_DEVICE_ID_SIZE - 1;
|
length = USBLP_DEVICE_ID_SIZE - 1;
|
||||||
usblp->device_id_string[length] = '\0';
|
usblp->device_id_string[length] = '\0';
|
||||||
|
|
||||||
dbg("usblp%d Device ID string [len=%d]=\"%s\"",
|
dev_dbg(&usblp->intf->dev, "usblp%d Device ID string [len=%d]=\"%s\"\n",
|
||||||
usblp->minor, length, &usblp->device_id_string[2]);
|
usblp->minor, length, &usblp->device_id_string[2]);
|
||||||
|
|
||||||
return length;
|
return length;
|
||||||
|
|
Загрузка…
Ссылка в новой задаче