[RFC PATCH 20/39] staging: comedi: usbduxfast: use dev->class_dev for all dev_{level} messages

H Hartley Sweeten hsweeten at visionengravers.com
Wed May 1 22:25:18 UTC 2013


For consistency in the driver, change all the dev_{level} messages
so they use dev->class_dev for the device.

Signed-off-by: H Hartley Sweeten <hsweeten at visionengravers.com>
Cc: Ian Abbott <abbotti at mev.co.uk>
Cc: Greg Kroah-Hartman <gregkh at linuxfonudation.org>
---
 drivers/staging/comedi/drivers/usbduxfast.c | 82 ++++++++++++-----------------
 1 file changed, 35 insertions(+), 47 deletions(-)

diff --git a/drivers/staging/comedi/drivers/usbduxfast.c b/drivers/staging/comedi/drivers/usbduxfast.c
index cbdea75..f237019 100644
--- a/drivers/staging/comedi/drivers/usbduxfast.c
+++ b/drivers/staging/comedi/drivers/usbduxfast.c
@@ -183,7 +183,7 @@ static int send_dux_commands(struct comedi_device *dev, int cmd_type)
 			   devpriv->dux_commands,
 			   SIZEOFDUXBUFFER, &nsent, 10000);
 	if (tmp < 0)
-		dev_err(&devpriv->intf->dev,
+		dev_err(dev->class_dev,
 			"could not transmit dux_commands to the usb-device, err=%d\n",
 			tmp);
 	return tmp;
@@ -354,7 +354,7 @@ static void usbduxfast_ai_interrupt(struct urb *urb)
 	urb->status = 0;
 	err = usb_submit_urb(urb, GFP_ATOMIC);
 	if (err < 0) {
-		dev_err(&urb->dev->dev,
+		dev_err(dev->class_dev,
 			"urb resubm failed: %d", err);
 		async->events |= COMEDI_CB_EOA;
 		async->events |= COMEDI_CB_ERROR;
@@ -386,8 +386,7 @@ static int usbduxfastsub_start(struct comedi_device *dev)
 			      1,      /* Length */
 			      EZTIMEOUT);    /* Timeout */
 	if (ret < 0)
-		dev_err(&devpriv->intf->dev,
-			"control msg failed (start)\n");
+		dev_err(dev->class_dev, "control msg failed (start)\n");
 
 	kfree(local_transfer_buffer);
 	return ret;
@@ -414,8 +413,7 @@ static int usbduxfastsub_stop(struct comedi_device *dev)
 			      local_transfer_buffer, 1,	/* Length */
 			      EZTIMEOUT);	/* Timeout */
 	if (ret < 0)
-		dev_err(&devpriv->intf->dev,
-			"control msg failed (stop)\n");
+		dev_err(dev->class_dev, "control msg failed (stop)\n");
 
 	kfree(local_transfer_buffer);
 	return ret;
@@ -439,7 +437,7 @@ static int usbduxfastsub_upload(struct comedi_device *dev,
 			      len,	/* length */
 			      EZTIMEOUT);      /* timeout */
 	if (ret < 0) {
-		dev_err(&devpriv->intf->dev, "uppload failed\n");
+		dev_err(dev->class_dev, "uppload failed\n");
 		return ret;
 	}
 
@@ -461,8 +459,7 @@ static int usbduxfastsub_submit_InURBs(struct comedi_device *dev)
 
 	ret = usb_submit_urb(devpriv->urbIn, GFP_ATOMIC);
 	if (ret) {
-		dev_err(&devpriv->intf->dev,
-			"ai: usb_submit_urb error %d\n", ret);
+		dev_err(dev->class_dev, "usb_submit_urb error %d\n", ret);
 		return ret;
 	}
 	return 0;
@@ -574,7 +571,7 @@ static int usbduxfast_ai_inttrig(struct comedi_device *dev,
 	down(&devpriv->sem);
 
 	if (trignum != 0) {
-		dev_err(dev->class_dev, "%s: invalid trignum\n", __func__);
+		dev_err(dev->class_dev, "invalid trignum\n");
 		up(&devpriv->sem);
 		return -EINVAL;
 	}
@@ -582,16 +579,14 @@ static int usbduxfast_ai_inttrig(struct comedi_device *dev,
 		devpriv->ai_cmd_running = 1;
 		ret = usbduxfastsub_submit_InURBs(dev);
 		if (ret < 0) {
-			dev_err(dev->class_dev,
-				"%s: urbSubmit: err=%d\n", __func__, ret);
+			dev_err(dev->class_dev, "urbSubmit: err=%d\n", ret);
 			devpriv->ai_cmd_running = 0;
 			up(&devpriv->sem);
 			return ret;
 		}
 		s->async->inttrig = NULL;
 	} else {
-		dev_err(dev->class_dev,
-			"ai_inttrig but acqu is already running\n");
+		dev_err(dev->class_dev, "ai is already running\n");
 	}
 	up(&devpriv->sem);
 	return 1;
@@ -621,8 +616,7 @@ static int usbduxfast_ai_cmd(struct comedi_device *dev,
 
 	down(&devpriv->sem);
 	if (devpriv->ai_cmd_running) {
-		dev_err(dev->class_dev,
-			"ai_cmd not possible. Another ai_cmd is running.\n");
+		dev_err(dev->class_dev, "ai_cmd not possible\n");
 		up(&devpriv->sem);
 		return -EBUSY;
 	}
@@ -641,20 +635,19 @@ static int usbduxfast_ai_cmd(struct comedi_device *dev,
 			chan = CR_CHAN(cmd->chanlist[i]);
 			if (chan != i) {
 				dev_err(dev->class_dev,
-					"cmd is accepting only consecutive channels.\n");
+					"channels are not consecutive\n");
 				up(&devpriv->sem);
 				return -EINVAL;
 			}
 			if ((gain != CR_RANGE(cmd->chanlist[i]))
 			    && (cmd->chanlist_len > 3)) {
 				dev_err(dev->class_dev,
-					"the gain must be the same for all channels.\n");
+					"gain must be the same for all channels\n");
 				up(&devpriv->sem);
 				return -EINVAL;
 			}
 			if (i >= NUMCHANNELS) {
-				dev_err(dev->class_dev,
-					"channel list too long\n");
+				dev_err(dev->class_dev, "chanlist too long\n");
 				break;
 			}
 		}
@@ -662,7 +655,7 @@ static int usbduxfast_ai_cmd(struct comedi_device *dev,
 	steps = 0;
 	if (cmd->scan_begin_src == TRIG_TIMER) {
 		dev_err(dev->class_dev,
-			"scan_begin_src==TRIG_TIMER not valid.\n");
+			"scan_begin_src==TRIG_TIMER not valid\n");
 		up(&devpriv->sem);
 		return -EINVAL;
 	}
@@ -671,20 +664,20 @@ static int usbduxfast_ai_cmd(struct comedi_device *dev,
 
 	if ((steps < MIN_SAMPLING_PERIOD) && (cmd->chanlist_len != 1)) {
 		dev_err(dev->class_dev,
-			"ai_cmd: steps=%ld, scan_begin_arg=%d. Not properly tested by cmdtest?\n",
+			"steps=%ld, scan_begin_arg=%d. Not properly tested by cmdtest?\n",
 			steps, cmd->scan_begin_arg);
 		up(&devpriv->sem);
 		return -EINVAL;
 	}
 	if (steps > MAX_SAMPLING_PERIOD) {
-		dev_err(dev->class_dev, "ai_cmd: sampling rate too low.\n");
+		dev_err(dev->class_dev, "sampling rate too low\n");
 		up(&devpriv->sem);
 		return -EINVAL;
 	}
 	if ((cmd->start_src == TRIG_EXT) && (cmd->chanlist_len != 1)
 	    && (cmd->chanlist_len != 16)) {
 		dev_err(dev->class_dev,
-			"ai_cmd: TRIG_EXT only with 1 or 16 channels possible.\n");
+			"TRIG_EXT only with 1 or 16 channels possible\n");
 		up(&devpriv->sem);
 		return -EINVAL;
 	}
@@ -1006,7 +999,7 @@ static int usbduxfast_ai_cmd(struct comedi_device *dev,
 	result = send_dux_commands(dev, SENDADCOMMANDS);
 	if (result < 0) {
 		dev_err(dev->class_dev,
-			"adc command could not be submitted. Aborting...\n");
+			"adc command could not be submitted, aborting\n");
 		up(&devpriv->sem);
 		return result;
 	}
@@ -1014,7 +1007,7 @@ static int usbduxfast_ai_cmd(struct comedi_device *dev,
 		devpriv->ai_sample_count = cmd->stop_arg * cmd->scan_end_arg;
 		if (devpriv->ai_sample_count < 1) {
 			dev_err(dev->class_dev,
-				"(cmd->stop_arg)*(cmd->scan_end_arg)<1, aborting.\n");
+				"(cmd->stop_arg)*(cmd->scan_end_arg)<1, aborting\n");
 			up(&devpriv->sem);
 			return -EFAULT;
 		}
@@ -1062,14 +1055,14 @@ static int usbduxfast_ai_insn_read(struct comedi_device *dev,
 	int err;
 
 	if (!devpriv) {
-		dev_err(dev->class_dev, "%s: no usb dev.\n", __func__);
+		dev_err(dev->class_dev, "no usb dev\n");
 		return -ENODEV;
 	}
 
 	down(&devpriv->sem);
 	if (devpriv->ai_cmd_running) {
 		dev_err(dev->class_dev,
-			"ai_insn_read not possible. Async Command is running.\n");
+			"ai_insn_read not possible, async cmd is running\n");
 		up(&devpriv->sem);
 		return -EBUSY;
 	}
@@ -1126,7 +1119,7 @@ static int usbduxfast_ai_insn_read(struct comedi_device *dev,
 	err = send_dux_commands(dev, SENDADCOMMANDS);
 	if (err < 0) {
 		dev_err(dev->class_dev,
-			"adc command could not be submitted. Aborting...\n");
+			"adc command could not be submitted, aborting\n");
 		up(&devpriv->sem);
 		return err;
 	}
@@ -1137,7 +1130,7 @@ static int usbduxfast_ai_insn_read(struct comedi_device *dev,
 				   devpriv->transfer_buffer, SIZEINBUF,
 				   &actual_length, 10000);
 		if (err < 0) {
-			dev_err(dev->class_dev, "insn timeout. No data.\n");
+			dev_err(dev->class_dev, "insn timeout, no data\n");
 			up(&devpriv->sem);
 			return err;
 		}
@@ -1155,7 +1148,7 @@ static int usbduxfast_ai_insn_read(struct comedi_device *dev,
 		}
 		n = actual_length / sizeof(uint16_t);
 		if ((n % 16) != 0) {
-			dev_err(dev->class_dev, "insn data packet corrupted.\n");
+			dev_err(dev->class_dev, "insn data packet corrupted\n");
 			up(&devpriv->sem);
 			return -EINVAL;
 		}
@@ -1173,46 +1166,40 @@ static int usbduxfast_ai_insn_read(struct comedi_device *dev,
 static int firmwareUpload(struct comedi_device *dev,
 			  const u8 *firmwareBinary, int sizeFirmware)
 {
-	struct usbduxfast_private *devpriv = dev->private;
-	int ret;
 	uint8_t *fwBuf;
+	int ret;
 
 	if (!firmwareBinary)
 		return 0;
 
 	if (sizeFirmware > FIRMWARE_MAX_LEN) {
-		dev_err(&devpriv->intf->dev,
-			"comedi_: usbduxfast firmware binary it too large for FX2.\n");
+		dev_err(dev->class_dev, "firmware binary too large for FX2\n");
 		return -ENOMEM;
 	}
 
 	/* we generate a local buffer for the firmware */
 	fwBuf = kmemdup(firmwareBinary, sizeFirmware, GFP_KERNEL);
 	if (!fwBuf) {
-		dev_err(&devpriv->intf->dev,
-			"comedi_: mem alloc for firmware failed\n");
+		dev_err(dev->class_dev, "mem alloc for firmware failed\n");
 		return -ENOMEM;
 	}
 
 	ret = usbduxfastsub_stop(dev);
 	if (ret < 0) {
-		dev_err(&devpriv->intf->dev,
-			"comedi_: can not stop firmware\n");
+		dev_err(dev->class_dev, "can not stop firmware\n");
 		kfree(fwBuf);
 		return ret;
 	}
 
 	ret = usbduxfastsub_upload(dev, fwBuf, 0, sizeFirmware);
 	if (ret < 0) {
-		dev_err(&devpriv->intf->dev,
-			"comedi_: firmware upload failed\n");
+		dev_err(dev->class_dev, "firmware upload failed\n");
 		kfree(fwBuf);
 		return ret;
 	}
 	ret = usbduxfastsub_start(dev);
 	if (ret < 0) {
-		dev_err(&devpriv->intf->dev,
-			"comedi_: can not start firmware\n");
+		dev_err(dev->class_dev, "can not start firmware\n");
 		kfree(fwBuf);
 		return ret;
 	}
@@ -1280,7 +1267,7 @@ static int usbduxfast_auto_attach(struct comedi_device *dev,
 	int ret;
 
 	if (usb->speed != USB_SPEED_HIGH) {
-		dev_err(&intf->dev,
+		dev_err(dev->class_dev,
 			"This driver needs USB 2.0 to operate. Aborting...\n");
 		return -ENODEV;
 	}
@@ -1306,14 +1293,14 @@ static int usbduxfast_auto_attach(struct comedi_device *dev,
 	ret = usb_set_interface(devpriv->usb,
 				intf->altsetting->desc.bInterfaceNumber, 1);
 	if (ret < 0) {
-		dev_err(&intf->dev,
+		dev_err(dev->class_dev,
 			"could not switch to alternate setting 1\n");
 		return -ENODEV;
 	}
 
 	devpriv->urbIn = usb_alloc_urb(0, GFP_KERNEL);
 	if (!devpriv->urbIn) {
-		dev_err(&intf->dev, "Could not alloc. urb\n");
+		dev_err(dev->class_dev, "Could not alloc. urb\n");
 		return -ENOMEM;
 	}
 
@@ -1327,7 +1314,8 @@ static int usbduxfast_auto_attach(struct comedi_device *dev,
 	 */
 	ret = usbduxfast_request_firmware(dev);
 	if (ret) {
-		dev_err(&intf->dev, "could not load firmware (err=%d)\n", ret);
+		dev_err(dev->class_dev, "could not load firmware (err=%d)\n",
+			ret);
 		return ret;
 	}
 
-- 
1.8.1.4




More information about the devel mailing list