[PATCH v2] Staging: comedi: fix warning issue in pcl818.c

[PATCH v2] Staging: comedi: fix warning issue in pcl818.c

[Ravishnakar]

From: Ravishankar <ravi.shankar@greenturtles.in>

This is a patch to the pcl818.c file that fixes up a printk(),brace coding style and initilise statics to NULL warning found by the checkpatch.pl tool

Signed-off-by: Ravishankar <ravishankarkm32@gmail.com>
---
 drivers/staging/comedi/drivers/pcl818.c |  121 +++++++++++++++++----------------
 1 files changed, 59 insertions(+), 62 deletions(-)

diff --git a/drivers/staging/comedi/drivers/pcl818.c b/drivers/staging/comedi/drivers/pcl818.c
index b45a9bd..18d318c 100644
--- a/drivers/staging/comedi/drivers/pcl818.c
+++ b/drivers/staging/comedi/drivers/pcl818.c
@@ -252,8 +252,8 @@ static int pcl818_attach(struct comedi_device *dev,
 static int pcl818_detach(struct comedi_device *dev);
 
 #ifdef unused
-static int RTC_lock = 0;	/* RTC lock */
-static int RTC_timer_lock = 0;	/* RTC int lock */
+static int RTC_lock;	/* RTC lock */
+static int RTC_timer_lock;	/* RTC int lock */
 #endif
 
 struct pcl818_board {
@@ -463,10 +463,8 @@ static int pcl818_ao_insn_read(struct comedi_device *dev,
 	int n;
 	int chan = CR_CHAN(insn->chanspec);
 
-	for (n = 0; n < insn->n; n++) {
+	for (n = 0; n < insn->n; n++)
 		data[n] = devpriv->ao_readback[chan];
-	}
-
 	return n;
 }
 
@@ -571,9 +569,9 @@ conv_finish:
 		return IRQ_HANDLED;
 	}
 	devpriv->act_chanlist_pos++;
-	if (devpriv->act_chanlist_pos >= devpriv->act_chanlist_len) {
+	if (devpriv->act_chanlist_pos >= devpriv->act_chanlist_len)
 		devpriv->act_chanlist_pos = 0;
-	}
+
 	s->async->cur_chan++;
 	if (s->async->cur_chan >= devpriv->ai_n_chan) {
 		/*  printk("E"); */
@@ -645,9 +643,9 @@ static irqreturn_t interrupt_pcl818_ai_mode13_dma(int irq, void *d)
 		comedi_buf_put(s->async, ptr[bufptr++] >> 4);	/*  get one sample */
 
 		devpriv->act_chanlist_pos++;
-		if (devpriv->act_chanlist_pos >= devpriv->act_chanlist_len) {
+		if (devpriv->act_chanlist_pos >= devpriv->act_chanlist_len)
 			devpriv->act_chanlist_pos = 0;
-		}
+
 		s->async->cur_chan++;
 		if (s->async->cur_chan >= devpriv->ai_n_chan) {
 			s->async->cur_chan = 0;
@@ -805,11 +803,10 @@ static irqreturn_t interrupt_pcl818_ai_mode13_fifo(int irq, void *d)
 		return IRQ_HANDLED;
 	}
 
-	if (lo & 2) {
+	if (lo & 2)
 		len = 512;
-	} else {
+	else
 		len = 0;
-	}
 
 	for (i = 0; i < len; i++) {
 		lo = inb(dev->iobase + PCL818_FI_DATALO);
@@ -827,9 +824,9 @@ static irqreturn_t interrupt_pcl818_ai_mode13_fifo(int irq, void *d)
 		comedi_buf_put(s->async, (lo >> 4) | (inb(dev->iobase + PCL818_FI_DATAHI) << 4));	/*  get one sample */
 
 		devpriv->act_chanlist_pos++;
-		if (devpriv->act_chanlist_pos >= devpriv->act_chanlist_len) {
+		if (devpriv->act_chanlist_pos >= devpriv->act_chanlist_len)
 			devpriv->act_chanlist_pos = 0;
-		}
+
 		s->async->cur_chan++;
 		if (s->async->cur_chan >= devpriv->ai_n_chan) {
 			s->async->cur_chan = 0;
@@ -1009,7 +1006,7 @@ static int pcl818_ai_cmd_mode(int mode, struct comedi_device *dev,
 	int divisor1 = 0, divisor2 = 0;
 	unsigned int seglen;
 
-	printk("pcl818_ai_cmd_mode()\n");
+	dev_dbg(dev->hw_dev, "pcl818_ai_cmd_mode()\n");
 	if ((!dev->irq) && (!devpriv->dma_rtc)) {
 		comedi_error(dev, "IRQ not defined!");
 		return -EINVAL;
@@ -1112,7 +1109,7 @@ static int pcl818_ai_cmd_mode(int mode, struct comedi_device *dev,
 		break;
 	}
 #endif
-	printk("pcl818_ai_cmd_mode() end\n");
+	dev_dbg(dev->hw_dev, "pcl818_ai_cmd_mode() end\n");
 	return 0;
 }
 
@@ -1309,11 +1306,9 @@ static void setup_channel_list(struct comedi_device *dev,
 */
 static int check_single_ended(unsigned int port)
 {
-	if (inb(port + PCL818_STATUS) & 0x20) {
+	if (inb(port + PCL818_STATUS) & 0x20)
 		return 1;
-	} else {
-		return 0;
-	}
+	return 0;
 }
 
 /*
@@ -1352,9 +1347,8 @@ static int ai_cmdtest(struct comedi_device *dev, struct comedi_subdevice *s,
 	if (!cmd->stop_src || tmp != cmd->stop_src)
 		err++;
 
-	if (err) {
+	if (err)
 		return 1;
-	}
 
 	/* step 2: make sure trigger sources are unique and mutually compatible */
 
@@ -1377,9 +1371,8 @@ static int ai_cmdtest(struct comedi_device *dev, struct comedi_subdevice *s,
 	if (cmd->stop_src != TRIG_NONE && cmd->stop_src != TRIG_COUNT)
 		err++;
 
-	if (err) {
+	if (err)
 		return 2;
-	}
 
 	/* step 3: make sure arguments are trivially compatible */
 
@@ -1421,9 +1414,8 @@ static int ai_cmdtest(struct comedi_device *dev, struct comedi_subdevice *s,
 		}
 	}
 
-	if (err) {
+	if (err)
 		return 3;
-	}
 
 	/* step 4: fix up any arguments */
 
@@ -1438,9 +1430,8 @@ static int ai_cmdtest(struct comedi_device *dev, struct comedi_subdevice *s,
 			err++;
 	}
 
-	if (err) {
+	if (err)
 		return 4;
-	}
 
 	/* step 5: complain about special chanlist considerations */
 
@@ -1461,7 +1452,7 @@ static int ai_cmd(struct comedi_device *dev, struct comedi_subdevice *s)
 	struct comedi_cmd *cmd = &s->async->cmd;
 	int retval;
 
-	printk("pcl818_ai_cmd()\n");
+	dev_dbg(dev->hw_dev, "pcl818_ai_cmd()\n");
 	devpriv->ai_n_chan = cmd->chanlist_len;
 	devpriv->ai_chanlist = cmd->chanlist;
 	devpriv->ai_flags = cmd->flags;
@@ -1470,17 +1461,16 @@ static int ai_cmd(struct comedi_device *dev, struct comedi_subdevice *s)
 	devpriv->ai_timer1 = 0;
 	devpriv->ai_timer2 = 0;
 
-	if (cmd->stop_src == TRIG_COUNT) {
+	if (cmd->stop_src == TRIG_COUNT)
 		devpriv->ai_scans = cmd->stop_arg;
-	} else {
+	else
 		devpriv->ai_scans = 0;
-	}
 
 	if (cmd->scan_begin_src == TRIG_FOLLOW) {	/*  mode 1, 3 */
 		if (cmd->convert_src == TRIG_TIMER) {	/*  mode 1 */
 			devpriv->ai_timer1 = cmd->convert_arg;
 			retval = pcl818_ai_cmd_mode(1, dev, s);
-			printk("pcl818_ai_cmd() end\n");
+			dev_dbg(dev->hw_dev, "pcl818_ai_cmd() end\n");
 			return retval;
 		}
 		if (cmd->convert_src == TRIG_EXT) {	/*  mode 3 */
@@ -1499,7 +1489,7 @@ static int pcl818_ai_cancel(struct comedi_device *dev,
 			    struct comedi_subdevice *s)
 {
 	if (devpriv->irq_blocked > 0) {
-		printk("pcl818_ai_cancel()\n");
+		dev_dbg(dev->hw_dev, "pcl818_ai_cancel()\n");
 		devpriv->irq_was_now_closed = 1;
 
 		switch (devpriv->ai_mode) {
@@ -1549,7 +1539,7 @@ static int pcl818_ai_cancel(struct comedi_device *dev,
 	}
 
 end:
-	printk("pcl818_ai_cancel() end\n");
+	dev_dbg(dev->hw_dev, "pcl818_ai_cancel() end\n");
 	return 0;
 }
 
@@ -1633,11 +1623,11 @@ static int set_rtc_irq_bit(unsigned char bit)
 	save_flags(flags);
 	cli();
 	val = CMOS_READ(RTC_CONTROL);
-	if (bit) {
+	if (bit)
 		val |= RTC_PIE;
-	} else {
+	else
 		val &= ~RTC_PIE;
-	}
+
 	CMOS_WRITE(val, RTC_CONTROL);
 	CMOS_READ(RTC_INTR_FLAGS);
 	restore_flags(flags);
@@ -1754,7 +1744,7 @@ static int pcl818_attach(struct comedi_device *dev, struct comedi_devconfig *it)
 
 	/* claim our I/O space */
 	iobase = it->options[0];
-	printk("comedi%d: pcl818:  board=%s, ioport=0x%03lx",
+	printk(KERN_INFO "comedi%d: pcl818:  board=%s, ioport=0x%03lx",
 	       dev->minor, this_board->name, iobase);
 	devpriv->io_range = this_board->io_range;
 	if ((this_board->fifo) && (it->options[2] == -1)) {	/*  we've board with FIFO and we want to use FIFO */
@@ -1762,17 +1752,19 @@ static int pcl818_attach(struct comedi_device *dev, struct comedi_devconfig *it)
 		devpriv->usefifo = 1;
 	}
 	if (!request_region(iobase, devpriv->io_range, "pcl818")) {
-		printk("I/O port conflict\n");
+		printk(KERN_ERR "I/O port conflict\n");
 		return -EIO;
 	}
 
+	printk(KERN_CONT ", I/O port detected\n");
 	dev->iobase = iobase;
 
 	if (pcl818_check(iobase)) {
-		printk(", I can't detect board. FAIL!\n");
+		printk(KERN_WARNING "I can't detect board. FAIL!\n");
 		return -EIO;
 	}
 
+	printk(KERN_CONT ", Board is detected successfully\n");
 	/* set up some name stuff */
 	dev->board_name = this_board->name;
 	/* grab our IRQ */
@@ -1782,30 +1774,29 @@ static int pcl818_attach(struct comedi_device *dev, struct comedi_devconfig *it)
 		if (irq) {	/* we want to use IRQ */
 			if (((1 << irq) & this_board->IRQbits) == 0) {
 				printk
-				    (", IRQ %u is out of allowed range, DISABLING IT",
+				printk(KERN_CONT ", IRQ %u is out of allowed range,"
+				"DISABLING IT",	irq);
 				irq = 0;	/* Bad IRQ */
 			} else {
 				if (request_irq
 				    (irq, interrupt_pcl818, 0, "pcl818", dev)) {
 					printk
-					    (", unable to allocate IRQ %u, DISABLING IT",
+					printk(KERN_CONT ", unable to allocate IRQ %u,"
+					"DISABLING IT", irq);
 					irq = 0;	/* Can't use IRQ */
 				} else {
-					printk(", irq=%u", irq);
+					printk(KERN_CONT ", irq=%u", irq);
 				}
 			}
 		}
 	}
 
 	dev->irq = irq;
-	if (irq) {
-		devpriv->irq_free = 1;
-	} /* 1=we have allocated irq */
-	else {
+	if (irq)
+		devpriv->irq_free = 1;   /* 1=we have allocated irq */
+	else
 		devpriv->irq_free = 0;
-	}
+
 	devpriv->irq_blocked = 0;	/* number of subdevice which use IRQ */
 	devpriv->ai_mode = 0;	/* mode of irq */
 
@@ -1825,7 +1816,7 @@ static int pcl818_attach(struct comedi_device *dev, struct comedi_devconfig *it)
 				 "pcl818 DMA (RTC)", dev)) {
 			devpriv->dma_rtc = 1;
 			devpriv->rtc_irq = RTC_IRQ;
-			printk(", dma_irq=%u", devpriv->rtc_irq);
+			printk(KERN_CONT ", dma_irq=%u", devpriv->rtc_irq);
 		} else {
 			RTC_lock--;
 			if (RTC_lock == 0) {
@@ -1850,23 +1841,26 @@ no_rtc:
 		if (dma < 1)
 			goto no_dma;	/* DMA disabled */
 		if (((1 << dma) & this_board->DMAbits) == 0) {
-			printk(", DMA is out of allowed range, FAIL!\n");
+			printk(KERN_WARNING "DMA is out of allowed range, FAIL!\n");
 			return -EINVAL;	/* Bad DMA */
 		}
+		printk(KERN_CONT ", DMA is in allowed range\n");
 		ret = request_dma(dma, "pcl818");
 		if (ret) {
-			printk(", unable to allocate DMA %u, FAIL!\n", dma);
+			printk(KERN_WARNNG "unable to allocate DMA %u,"
+			"FAIL!\n", dma);
 			return -EBUSY;	/* DMA isn't free */
 		}
+		printk(KERN_CONT ", DMA allocated successfully\n");
 		devpriv->dma = dma;
-		printk(", dma=%u", dma);
+		printk(KERN_CONT ", dma=%u", dma);
 		pages = 2;	/* we need 16KB */
 		devpriv->dmabuf[0] = __get_dma_pages(GFP_KERNEL, pages);
 		if (!devpriv->dmabuf[0]) {
-			printk(", unable to allocate DMA buffer, FAIL!\n");
+			printk(KERN_WARNING "unable to allocate DMA buffer, FAIL!\n");
 			/* maybe experiment with try_to_free_pages() will help .... */
 			return -EBUSY;	/* no buffer :-( */
 		}
+		printk(KERN_CONT ", DMA buffer allocated successfully\n");
 		devpriv->dmapages[0] = pages;
 		devpriv->hwdmaptr[0] = virt_to_bus((void *)devpriv->dmabuf[0]);
 		devpriv->hwdmasize[0] = (1 << pages) * PAGE_SIZE;
@@ -1874,10 +1868,11 @@ no_rtc:
 		if (devpriv->dma_rtc == 0) {	/*  we must do duble buff :-( */
 			devpriv->dmabuf[1] = __get_dma_pages(GFP_KERNEL, pages);
 			if (!devpriv->dmabuf[1]) {
-				printk
-				    (", unable to allocate DMA buffer, FAIL!\n");
+				printk(KERN_WARNING
+				"unable to allocate DMA buffer, FAIL!\n");
 				return -EBUSY;
 			}
+			printk(KERN_CONT ", DMA buffer allocated successfully\n");
 			devpriv->dmapages[1] = pages;
 			devpriv->hwdmaptr[1] =
 			    virt_to_bus((void *)devpriv->dmabuf[1]);
@@ -1901,11 +1896,11 @@ no_dma:
 		if (check_single_ended(dev->iobase)) {
 			s->n_chan = this_board->n_aichan_se;
 			s->subdev_flags |= SDF_COMMON | SDF_GROUND;
-			printk(", %dchans S.E. DAC", s->n_chan);
+			printk(KERN_CONT ", %dchans S.E. DAC", s->n_chan);
 		} else {
 			s->n_chan = this_board->n_aichan_diff;
 			s->subdev_flags |= SDF_DIFF;
-			printk(", %dchans DIFF DAC", s->n_chan);
+			printk(KERN_CONT ", %dchans DIFF DAC", s->n_chan);
 		}
 		s->maxdata = this_board->ai_maxdata;
 		s->len_chanlist = s->n_chan;
@@ -2017,11 +2012,10 @@ no_dma:
 	}
 
 	/* select 1/10MHz oscilator */
-	if ((it->options[3] == 0) || (it->options[3] == 10)) {
+	if ((it->options[3] == 0) || (it->options[3] == 10))
 		devpriv->i8253_osc_base = 100;
-	} else {
+	else
 		devpriv->i8253_osc_base = 1000;
-	}
 
 	/* max sampling speed */
 	devpriv->ns_min = this_board->ns_min;
@@ -2033,7 +2027,7 @@ no_dma:
 
 	pcl818_reset(dev);
 
-	printk("\n");
+	printk(KERN_CONT "\n");
 
 	return 0;
 }
-- 
1.6.5.2

Re: [PATCH v2] Staging: comedi: fix warning issue in pcl818.c

[Dan Carpenter]

You're still putting KERNEL_WARNING labels in the middle of the
printed output instead of only at the start of the line.

regards,
dan carpenter

[PATCH v2] Staging: comedi: fix warning issue in pcl818.c

[Ravishankar]

From: Ravishankar <ravi.shankar@greenturtles.in>

This is a patch to the pcl818.c file that fixes up a printk(),brace coding style and initilise statics to NULL warning found by the checkpatch.pl tool

Signed-off-by: Ravishankar <ravishankarkm32@gmail.com>
---
 drivers/staging/comedi/drivers/pcl818.c |  120 +++++++++++++++++----------------
 1 files changed, 58 insertions(+), 62 deletions(-)

diff --git a/drivers/staging/comedi/drivers/pcl818.c b/drivers/staging/comedi/drivers/pcl818.c
index b45a9bd..18d318c 100644
--- a/drivers/staging/comedi/drivers/pcl818.c
+++ b/drivers/staging/comedi/drivers/pcl818.c
@@ -252,8 +252,8 @@ static int pcl818_attach(struct comedi_device *dev,
 static int pcl818_detach(struct comedi_device *dev);
 
 #ifdef unused
-static int RTC_lock = 0;	/* RTC lock */
-static int RTC_timer_lock = 0;	/* RTC int lock */
+static int RTC_lock;	/* RTC lock */
+static int RTC_timer_lock;	/* RTC int lock */
 #endif
 
 struct pcl818_board {
@@ -463,10 +463,8 @@ static int pcl818_ao_insn_read(struct comedi_device *dev,
 	int n;
 	int chan = CR_CHAN(insn->chanspec);
 
-	for (n = 0; n < insn->n; n++) {
+	for (n = 0; n < insn->n; n++)
 		data[n] = devpriv->ao_readback[chan];
-	}
-
 	return n;
 }
 
@@ -571,9 +569,9 @@ conv_finish:
 		return IRQ_HANDLED;
 	}
 	devpriv->act_chanlist_pos++;
-	if (devpriv->act_chanlist_pos >= devpriv->act_chanlist_len) {
+	if (devpriv->act_chanlist_pos >= devpriv->act_chanlist_len)
 		devpriv->act_chanlist_pos = 0;
-	}
+
 	s->async->cur_chan++;
 	if (s->async->cur_chan >= devpriv->ai_n_chan) {
 		/*  printk("E"); */
@@ -645,9 +643,9 @@ static irqreturn_t interrupt_pcl818_ai_mode13_dma(int irq, void *d)
 		comedi_buf_put(s->async, ptr[bufptr++] >> 4);	/*  get one sample */
 
 		devpriv->act_chanlist_pos++;
-		if (devpriv->act_chanlist_pos >= devpriv->act_chanlist_len) {
+		if (devpriv->act_chanlist_pos >= devpriv->act_chanlist_len)
 			devpriv->act_chanlist_pos = 0;
-		}
+
 		s->async->cur_chan++;
 		if (s->async->cur_chan >= devpriv->ai_n_chan) {
 			s->async->cur_chan = 0;
@@ -805,11 +803,10 @@ static irqreturn_t interrupt_pcl818_ai_mode13_fifo(int irq, void *d)
 		return IRQ_HANDLED;
 	}
 
-	if (lo & 2) {
+	if (lo & 2)
 		len = 512;
-	} else {
+	else
 		len = 0;
-	}
 
 	for (i = 0; i < len; i++) {
 		lo = inb(dev->iobase + PCL818_FI_DATALO);
@@ -827,9 +824,9 @@ static irqreturn_t interrupt_pcl818_ai_mode13_fifo(int irq, void *d)
 		comedi_buf_put(s->async, (lo >> 4) | (inb(dev->iobase + PCL818_FI_DATAHI) << 4));	/*  get one sample */
 
 		devpriv->act_chanlist_pos++;
-		if (devpriv->act_chanlist_pos >= devpriv->act_chanlist_len) {
+		if (devpriv->act_chanlist_pos >= devpriv->act_chanlist_len)
 			devpriv->act_chanlist_pos = 0;
-		}
+
 		s->async->cur_chan++;
 		if (s->async->cur_chan >= devpriv->ai_n_chan) {
 			s->async->cur_chan = 0;
@@ -1009,7 +1006,7 @@ static int pcl818_ai_cmd_mode(int mode, struct comedi_device *dev,
 	int divisor1 = 0, divisor2 = 0;
 	unsigned int seglen;
 
-	printk("pcl818_ai_cmd_mode()\n");
+	dev_dbg(dev->hw_dev, "pcl818_ai_cmd_mode()\n");
 	if ((!dev->irq) && (!devpriv->dma_rtc)) {
 		comedi_error(dev, "IRQ not defined!");
 		return -EINVAL;
@@ -1112,7 +1109,7 @@ static int pcl818_ai_cmd_mode(int mode, struct comedi_device *dev,
 		break;
 	}
 #endif
-	printk("pcl818_ai_cmd_mode() end\n");
+	dev_dbg(dev->hw_dev, "pcl818_ai_cmd_mode() end\n");
 	return 0;
 }
 
@@ -1309,11 +1306,9 @@ static void setup_channel_list(struct comedi_device *dev,
 */
 static int check_single_ended(unsigned int port)
 {
-	if (inb(port + PCL818_STATUS) & 0x20) {
+	if (inb(port + PCL818_STATUS) & 0x20)
 		return 1;
-	} else {
-		return 0;
-	}
+	return 0;
 }
 
 /*
@@ -1352,9 +1347,8 @@ static int ai_cmdtest(struct comedi_device *dev, struct comedi_subdevice *s,
 	if (!cmd->stop_src || tmp != cmd->stop_src)
 		err++;
 
-	if (err) {
+	if (err)
 		return 1;
-	}
 
 	/* step 2: make sure trigger sources are unique and mutually compatible */
 
@@ -1377,9 +1371,8 @@ static int ai_cmdtest(struct comedi_device *dev, struct comedi_subdevice *s,
 	if (cmd->stop_src != TRIG_NONE && cmd->stop_src != TRIG_COUNT)
 		err++;
 
-	if (err) {
+	if (err)
 		return 2;
-	}
 
 	/* step 3: make sure arguments are trivially compatible */
 
@@ -1421,9 +1414,8 @@ static int ai_cmdtest(struct comedi_device *dev, struct comedi_subdevice *s,
 		}
 	}
 
-	if (err) {
+	if (err)
 		return 3;
-	}
 
 	/* step 4: fix up any arguments */
 
@@ -1438,9 +1430,8 @@ static int ai_cmdtest(struct comedi_device *dev, struct comedi_subdevice *s,
 			err++;
 	}
 
-	if (err) {
+	if (err)
 		return 4;
-	}
 
 	/* step 5: complain about special chanlist considerations */
 
@@ -1461,7 +1452,7 @@ static int ai_cmd(struct comedi_device *dev, struct comedi_subdevice *s)
 	struct comedi_cmd *cmd = &s->async->cmd;
 	int retval;
 
-	printk("pcl818_ai_cmd()\n");
+	dev_dbg(dev->hw_dev, "pcl818_ai_cmd()\n");
 	devpriv->ai_n_chan = cmd->chanlist_len;
 	devpriv->ai_chanlist = cmd->chanlist;
 	devpriv->ai_flags = cmd->flags;
@@ -1470,17 +1461,16 @@ static int ai_cmd(struct comedi_device *dev, struct comedi_subdevice *s)
 	devpriv->ai_timer1 = 0;
 	devpriv->ai_timer2 = 0;
 
-	if (cmd->stop_src == TRIG_COUNT) {
+	if (cmd->stop_src == TRIG_COUNT)
 		devpriv->ai_scans = cmd->stop_arg;
-	} else {
+	else
 		devpriv->ai_scans = 0;
-	}
 
 	if (cmd->scan_begin_src == TRIG_FOLLOW) {	/*  mode 1, 3 */
 		if (cmd->convert_src == TRIG_TIMER) {	/*  mode 1 */
 			devpriv->ai_timer1 = cmd->convert_arg;
 			retval = pcl818_ai_cmd_mode(1, dev, s);
-			printk("pcl818_ai_cmd() end\n");
+			dev_dbg(dev->hw_dev, "pcl818_ai_cmd() end\n");
 			return retval;
 		}
 		if (cmd->convert_src == TRIG_EXT) {	/*  mode 3 */
@@ -1499,7 +1489,7 @@ static int pcl818_ai_cancel(struct comedi_device *dev,
 			    struct comedi_subdevice *s)
 {
 	if (devpriv->irq_blocked > 0) {
-		printk("pcl818_ai_cancel()\n");
+		dev_dbg(dev->hw_dev, "pcl818_ai_cancel()\n");
 		devpriv->irq_was_now_closed = 1;
 
 		switch (devpriv->ai_mode) {
@@ -1549,7 +1539,7 @@ static int pcl818_ai_cancel(struct comedi_device *dev,
 	}
 
 end:
-	printk("pcl818_ai_cancel() end\n");
+	dev_dbg(dev->hw_dev, "pcl818_ai_cancel() end\n");
 	return 0;
 }
 
@@ -1633,11 +1623,11 @@ static int set_rtc_irq_bit(unsigned char bit)
 	save_flags(flags);
 	cli();
 	val = CMOS_READ(RTC_CONTROL);
-	if (bit) {
+	if (bit)
 		val |= RTC_PIE;
-	} else {
+	else
 		val &= ~RTC_PIE;
-	}
+
 	CMOS_WRITE(val, RTC_CONTROL);
 	CMOS_READ(RTC_INTR_FLAGS);
 	restore_flags(flags);
@@ -1754,7 +1744,7 @@ static int pcl818_attach(struct comedi_device *dev, struct comedi_devconfig *it)
 
 	/* claim our I/O space */
 	iobase = it->options[0];
-	printk("comedi%d: pcl818:  board=%s, ioport=0x%03lx",
+	printk(KERN_INFO "comedi%d: pcl818:  board=%s, ioport=0x%03lx",
 	       dev->minor, this_board->name, iobase);
 	devpriv->io_range = this_board->io_range;
 	if ((this_board->fifo) && (it->options[2] == -1)) {	/*  we've board with FIFO and we want to use FIFO */
@@ -1762,17 +1752,19 @@ static int pcl818_attach(struct comedi_device *dev, struct comedi_devconfig *it)
 		devpriv->usefifo = 1;
 	}
 	if (!request_region(iobase, devpriv->io_range, "pcl818")) {
-		printk("I/O port conflict\n");
+		printk(KERN_ERR "I/O port conflict\n");
 		return -EIO;
 	}
 
+	printk(KERN_CONT ", I/O port detected\n");
 	dev->iobase = iobase;
 
 	if (pcl818_check(iobase)) {
-		printk(", I can't detect board. FAIL!\n");
+		printk(KERN_WARNING "I can't detect board. FAIL!\n");
 		return -EIO;
 	}
 
+	printk(KERN_CONT ", Board is detected successfully\n");
 	/* set up some name stuff */
 	dev->board_name = this_board->name;
 	/* grab our IRQ */
@@ -1782,30 +1774,29 @@ static int pcl818_attach(struct comedi_device *dev, struct comedi_devconfig *it)
 		if (irq) {	/* we want to use IRQ */
 			if (((1 << irq) & this_board->IRQbits) == 0) {
 				printk
-				    (", IRQ %u is out of allowed range, DISABLING IT",
+				printk(KERN_CONT ", IRQ %u is out of allowed range,"
+				"DISABLING IT",	irq);
 				irq = 0;	/* Bad IRQ */
 			} else {
 				if (request_irq
 				    (irq, interrupt_pcl818, 0, "pcl818", dev)) {
 					printk
-					    (", unable to allocate IRQ %u, DISABLING IT",
+					printk(KERN_CONT ", unable to allocate IRQ %u,"
+					"DISABLING IT", irq);
 					irq = 0;	/* Can't use IRQ */
 				} else {
-					printk(", irq=%u", irq);
+					printk(KERN_CONT ", irq=%u", irq);
 				}
 			}
 		}
 	}
 
 	dev->irq = irq;
-	if (irq) {
-		devpriv->irq_free = 1;
-	} /* 1=we have allocated irq */
-	else {
+	if (irq)
+		devpriv->irq_free = 1;   /* 1=we have allocated irq */
+	else
 		devpriv->irq_free = 0;
-	}
+
 	devpriv->irq_blocked = 0;	/* number of subdevice which use IRQ */
 	devpriv->ai_mode = 0;	/* mode of irq */
 
@@ -1825,7 +1816,7 @@ static int pcl818_attach(struct comedi_device *dev, struct comedi_devconfig *it)
 				 "pcl818 DMA (RTC)", dev)) {
 			devpriv->dma_rtc = 1;
 			devpriv->rtc_irq = RTC_IRQ;
-			printk(", dma_irq=%u", devpriv->rtc_irq);
+			printk(KERN_CONT ", dma_irq=%u", devpriv->rtc_irq);
 		} else {
 			RTC_lock--;
 			if (RTC_lock == 0) {
@@ -1850,23 +1841,26 @@ no_rtc:
 		if (dma < 1)
 			goto no_dma;	/* DMA disabled */
 		if (((1 << dma) & this_board->DMAbits) == 0) {
-			printk(", DMA is out of allowed range, FAIL!\n");
+			printk(KERN_WARNING "DMA is out of allowed range, FAIL!\n");
 			return -EINVAL;	/* Bad DMA */
 		}
+		printk(KERN_CONT ", DMA is in allowed range\n");
 		ret = request_dma(dma, "pcl818");
 		if (ret) {
-			printk(", unable to allocate DMA %u, FAIL!\n", dma);
+			printk(KERN_WARNING "unable to allocate DMA %u,"
+			"FAIL!\n", dma);
 			return -EBUSY;	/* DMA isn't free */
 		}
+		printk(KERN_CONT ", DMA allocated successfully\n");
 		devpriv->dma = dma;
-		printk(", dma=%u", dma);
+		printk(KERN_CONT ", dma=%u", dma);
 		pages = 2;	/* we need 16KB */
 		devpriv->dmabuf[0] = __get_dma_pages(GFP_KERNEL, pages);
 		if (!devpriv->dmabuf[0]) {
-			printk(", unable to allocate DMA buffer, FAIL!\n");
+			printk(KERN_WARNING "unable to allocate DMA buffer, FAIL!\n");
 			/* maybe experiment with try_to_free_pages() will help .... */
 			return -EBUSY;	/* no buffer :-( */
 		}
+		printk(KERN_CONT ", DMA buffer allocated successfully\n");
 		devpriv->dmapages[0] = pages;
 		devpriv->hwdmaptr[0] = virt_to_bus((void *)devpriv->dmabuf[0]);
 		devpriv->hwdmasize[0] = (1 << pages) * PAGE_SIZE;
@@ -1874,10 +1868,11 @@ no_rtc:
 		if (devpriv->dma_rtc == 0) {	/*  we must do duble buff :-( */
 			devpriv->dmabuf[1] = __get_dma_pages(GFP_KERNEL, pages);
 			if (!devpriv->dmabuf[1]) {
-				printk
-				    (", unable to allocate DMA buffer, FAIL!\n");
+				printk(KERN_WARNING "unable to allocate DMA buffer, FAIL!\n");
 				return -EBUSY;
 			}
+			printk(KERN_CONT ", DMA buffer allocated successfully\n");
 			devpriv->dmapages[1] = pages;
 			devpriv->hwdmaptr[1] =
 			    virt_to_bus((void *)devpriv->dmabuf[1]);
@@ -1901,11 +1896,11 @@ no_dma:
 		if (check_single_ended(dev->iobase)) {
 			s->n_chan = this_board->n_aichan_se;
 			s->subdev_flags |= SDF_COMMON | SDF_GROUND;
-			printk(", %dchans S.E. DAC", s->n_chan);
+			printk(KERN_CONT ", %dchans S.E. DAC", s->n_chan);
 		} else {
 			s->n_chan = this_board->n_aichan_diff;
 			s->subdev_flags |= SDF_DIFF;
-			printk(", %dchans DIFF DAC", s->n_chan);
+			printk(KERN_CONT ", %dchans DIFF DAC", s->n_chan);
 		}
 		s->maxdata = this_board->ai_maxdata;
 		s->len_chanlist = s->n_chan;
@@ -2017,11 +2012,10 @@ no_dma:
 	}
 
 	/* select 1/10MHz oscilator */
-	if ((it->options[3] == 0) || (it->options[3] == 10)) {
+	if ((it->options[3] == 0) || (it->options[3] == 10))
 		devpriv->i8253_osc_base = 100;
-	} else {
+	else
 		devpriv->i8253_osc_base = 1000;
-	}
 
 	/* max sampling speed */
 	devpriv->ns_min = this_board->ns_min;
@@ -2033,7 +2027,7 @@ no_dma:
 
 	pcl818_reset(dev);
 
-	printk("\n");
+	printk(KERN_CONT "\n");
 
 	return 0;
 }
-- 
1.6.5.2

Re: [PATCH v2] Staging: comedi: fix warning issue in pcl818.c

[Ryan Mallon]

On 03/08/11 23:49, Ravishankar wrote:
> From: Ravishankar<ravi.shankar@greenturtles.in>
>
> This is a patch to the pcl818.c file that fixes up a printk(),brace coding style and initilise statics to NULL warning found by the checkpatch.pl tool
>
> Signed-off-by: Ravishankar<ravishankarkm32@gmail.com>

There are still a multitude of issues with these patches, including:
- Arbitary mix of dev_printk functions and printk functions.
- Introducing split printk lines
- Busted indentation on said split lines
- KERN_CONT problems (still!)
- Introducing additional printks (which the changelog does not mention)

I don't think you are understanding the code which you are attempting to 
edit, and you clearly aren't testing your changes (or even compiling them!).

You have posted dozens of patches now for what should be trivial clean 
ups, and all of them are unacceptable. I suspect that the people who may 
have been in a place to merge these patches have long since dropped you 
into their killfile to avoid the noise :-/.

~Ryan

> ---
>   drivers/staging/comedi/drivers/pcl818.c |  120 +++++++++++++++++----------------
>   1 files changed, 58 insertions(+), 62 deletions(-)
>
> diff --git a/drivers/staging/comedi/drivers/pcl818.c b/drivers/staging/comedi/drivers/pcl818.c
> index b45a9bd..18d318c 100644
> --- a/drivers/staging/comedi/drivers/pcl818.c
> +++ b/drivers/staging/comedi/drivers/pcl818.c
> @@ -252,8 +252,8 @@ static int pcl818_attach(struct comedi_device *dev,
>   static int pcl818_detach(struct comedi_device *dev);
>
>   #ifdef unused
> -static int RTC_lock = 0;	/* RTC lock */
> -static int RTC_timer_lock = 0;	/* RTC int lock */
> +static int RTC_lock;	/* RTC lock */
> +static int RTC_timer_lock;	/* RTC int lock */
>   #endif
>
>   struct pcl818_board {
> @@ -463,10 +463,8 @@ static int pcl818_ao_insn_read(struct comedi_device *dev,
>   	int n;
>   	int chan = CR_CHAN(insn->chanspec);
>
> -	for (n = 0; n<  insn->n; n++) {
> +	for (n = 0; n<  insn->n; n++)
>   		data[n] = devpriv->ao_readback[chan];
> -	}
> -
>   	return n;
>   }
>
> @@ -571,9 +569,9 @@ conv_finish:
>   		return IRQ_HANDLED;
>   	}
>   	devpriv->act_chanlist_pos++;
> -	if (devpriv->act_chanlist_pos>= devpriv->act_chanlist_len) {
> +	if (devpriv->act_chanlist_pos>= devpriv->act_chanlist_len)
>   		devpriv->act_chanlist_pos = 0;
> -	}
> +
>   	s->async->cur_chan++;
>   	if (s->async->cur_chan>= devpriv->ai_n_chan) {
>   		/*  printk("E"); */
> @@ -645,9 +643,9 @@ static irqreturn_t interrupt_pcl818_ai_mode13_dma(int irq, void *d)
>   		comedi_buf_put(s->async, ptr[bufptr++]>>  4);	/*  get one sample */
>
>   		devpriv->act_chanlist_pos++;
> -		if (devpriv->act_chanlist_pos>= devpriv->act_chanlist_len) {
> +		if (devpriv->act_chanlist_pos>= devpriv->act_chanlist_len)
>   			devpriv->act_chanlist_pos = 0;
> -		}
> +
>   		s->async->cur_chan++;
>   		if (s->async->cur_chan>= devpriv->ai_n_chan) {
>   			s->async->cur_chan = 0;
> @@ -805,11 +803,10 @@ static irqreturn_t interrupt_pcl818_ai_mode13_fifo(int irq, void *d)
>   		return IRQ_HANDLED;
>   	}
>
> -	if (lo&  2) {
> +	if (lo&  2)
>   		len = 512;
> -	} else {
> +	else
>   		len = 0;
> -	}
>
>   	for (i = 0; i<  len; i++) {
>   		lo = inb(dev->iobase + PCL818_FI_DATALO);
> @@ -827,9 +824,9 @@ static irqreturn_t interrupt_pcl818_ai_mode13_fifo(int irq, void *d)
>   		comedi_buf_put(s->async, (lo>>  4) | (inb(dev->iobase + PCL818_FI_DATAHI)<<  4));	/*  get one sample */
>
>   		devpriv->act_chanlist_pos++;
> -		if (devpriv->act_chanlist_pos>= devpriv->act_chanlist_len) {
> +		if (devpriv->act_chanlist_pos>= devpriv->act_chanlist_len)
>   			devpriv->act_chanlist_pos = 0;
> -		}
> +
>   		s->async->cur_chan++;
>   		if (s->async->cur_chan>= devpriv->ai_n_chan) {
>   			s->async->cur_chan = 0;
> @@ -1009,7 +1006,7 @@ static int pcl818_ai_cmd_mode(int mode, struct comedi_device *dev,
>   	int divisor1 = 0, divisor2 = 0;
>   	unsigned int seglen;
>
> -	printk("pcl818_ai_cmd_mode()\n");
> +	dev_dbg(dev->hw_dev, "pcl818_ai_cmd_mode()\n");
>   	if ((!dev->irq)&&  (!devpriv->dma_rtc)) {
>   		comedi_error(dev, "IRQ not defined!");
>   		return -EINVAL;
> @@ -1112,7 +1109,7 @@ static int pcl818_ai_cmd_mode(int mode, struct comedi_device *dev,
>   		break;
>   	}
>   #endif
> -	printk("pcl818_ai_cmd_mode() end\n");
> +	dev_dbg(dev->hw_dev, "pcl818_ai_cmd_mode() end\n");
>   	return 0;
>   }
>
> @@ -1309,11 +1306,9 @@ static void setup_channel_list(struct comedi_device *dev,
>   */
>   static int check_single_ended(unsigned int port)
>   {
> -	if (inb(port + PCL818_STATUS)&  0x20) {
> +	if (inb(port + PCL818_STATUS)&  0x20)
>   		return 1;
> -	} else {
> -		return 0;
> -	}
> +	return 0;
>   }
>
>   /*
> @@ -1352,9 +1347,8 @@ static int ai_cmdtest(struct comedi_device *dev, struct comedi_subdevice *s,
>   	if (!cmd->stop_src || tmp != cmd->stop_src)
>   		err++;
>
> -	if (err) {
> +	if (err)
>   		return 1;
> -	}
>
>   	/* step 2: make sure trigger sources are unique and mutually compatible */
>
> @@ -1377,9 +1371,8 @@ static int ai_cmdtest(struct comedi_device *dev, struct comedi_subdevice *s,
>   	if (cmd->stop_src != TRIG_NONE&&  cmd->stop_src != TRIG_COUNT)
>   		err++;
>
> -	if (err) {
> +	if (err)
>   		return 2;
> -	}
>
>   	/* step 3: make sure arguments are trivially compatible */
>
> @@ -1421,9 +1414,8 @@ static int ai_cmdtest(struct comedi_device *dev, struct comedi_subdevice *s,
>   		}
>   	}
>
> -	if (err) {
> +	if (err)
>   		return 3;
> -	}
>
>   	/* step 4: fix up any arguments */
>
> @@ -1438,9 +1430,8 @@ static int ai_cmdtest(struct comedi_device *dev, struct comedi_subdevice *s,
>   			err++;
>   	}
>
> -	if (err) {
> +	if (err)
>   		return 4;
> -	}
>
>   	/* step 5: complain about special chanlist considerations */
>
> @@ -1461,7 +1452,7 @@ static int ai_cmd(struct comedi_device *dev, struct comedi_subdevice *s)
>   	struct comedi_cmd *cmd =&s->async->cmd;
>   	int retval;
>
> -	printk("pcl818_ai_cmd()\n");
> +	dev_dbg(dev->hw_dev, "pcl818_ai_cmd()\n");
>   	devpriv->ai_n_chan = cmd->chanlist_len;
>   	devpriv->ai_chanlist = cmd->chanlist;
>   	devpriv->ai_flags = cmd->flags;
> @@ -1470,17 +1461,16 @@ static int ai_cmd(struct comedi_device *dev, struct comedi_subdevice *s)
>   	devpriv->ai_timer1 = 0;
>   	devpriv->ai_timer2 = 0;
>
> -	if (cmd->stop_src == TRIG_COUNT) {
> +	if (cmd->stop_src == TRIG_COUNT)
>   		devpriv->ai_scans = cmd->stop_arg;
> -	} else {
> +	else
>   		devpriv->ai_scans = 0;
> -	}
>
>   	if (cmd->scan_begin_src == TRIG_FOLLOW) {	/*  mode 1, 3 */
>   		if (cmd->convert_src == TRIG_TIMER) {	/*  mode 1 */
>   			devpriv->ai_timer1 = cmd->convert_arg;
>   			retval = pcl818_ai_cmd_mode(1, dev, s);
> -			printk("pcl818_ai_cmd() end\n");
> +			dev_dbg(dev->hw_dev, "pcl818_ai_cmd() end\n");
>   			return retval;
>   		}
>   		if (cmd->convert_src == TRIG_EXT) {	/*  mode 3 */
> @@ -1499,7 +1489,7 @@ static int pcl818_ai_cancel(struct comedi_device *dev,
>   			    struct comedi_subdevice *s)
>   {
>   	if (devpriv->irq_blocked>  0) {
> -		printk("pcl818_ai_cancel()\n");
> +		dev_dbg(dev->hw_dev, "pcl818_ai_cancel()\n");
>   		devpriv->irq_was_now_closed = 1;
>
>   		switch (devpriv->ai_mode) {
> @@ -1549,7 +1539,7 @@ static int pcl818_ai_cancel(struct comedi_device *dev,
>   	}
>
>   end:
> -	printk("pcl818_ai_cancel() end\n");
> +	dev_dbg(dev->hw_dev, "pcl818_ai_cancel() end\n");
>   	return 0;
>   }
>
> @@ -1633,11 +1623,11 @@ static int set_rtc_irq_bit(unsigned char bit)
>   	save_flags(flags);
>   	cli();
>   	val = CMOS_READ(RTC_CONTROL);
> -	if (bit) {
> +	if (bit)
>   		val |= RTC_PIE;
> -	} else {
> +	else
>   		val&= ~RTC_PIE;
> -	}
> +
>   	CMOS_WRITE(val, RTC_CONTROL);
>   	CMOS_READ(RTC_INTR_FLAGS);
>   	restore_flags(flags);
> @@ -1754,7 +1744,7 @@ static int pcl818_attach(struct comedi_device *dev, struct comedi_devconfig *it)
>
>   	/* claim our I/O space */
>   	iobase = it->options[0];
> -	printk("comedi%d: pcl818:  board=%s, ioport=0x%03lx",
> +	printk(KERN_INFO "comedi%d: pcl818:  board=%s, ioport=0x%03lx",
>   	       dev->minor, this_board->name, iobase);
>   	devpriv->io_range = this_board->io_range;
>   	if ((this_board->fifo)&&  (it->options[2] == -1)) {	/*  we've board with FIFO and we want to use FIFO */
> @@ -1762,17 +1752,19 @@ static int pcl818_attach(struct comedi_device *dev, struct comedi_devconfig *it)
>   		devpriv->usefifo = 1;
>   	}
>   	if (!request_region(iobase, devpriv->io_range, "pcl818")) {
> -		printk("I/O port conflict\n");
> +		printk(KERN_ERR "I/O port conflict\n");
>   		return -EIO;
>   	}
>
> +	printk(KERN_CONT ", I/O port detected\n");
>   	dev->iobase = iobase;
>
>   	if (pcl818_check(iobase)) {
> -		printk(", I can't detect board. FAIL!\n");
> +		printk(KERN_WARNING "I can't detect board. FAIL!\n");
>   		return -EIO;
>   	}
>
> +	printk(KERN_CONT ", Board is detected successfully\n");
>   	/* set up some name stuff */
>   	dev->board_name = this_board->name;
>   	/* grab our IRQ */
> @@ -1782,30 +1774,29 @@ static int pcl818_attach(struct comedi_device *dev, struct comedi_devconfig *it)
>   		if (irq) {	/* we want to use IRQ */
>   			if (((1<<  irq)&  this_board->IRQbits) == 0) {
>   				printk
> -				    (", IRQ %u is out of allowed range, DISABLING IT",
> +				printk(KERN_CONT ", IRQ %u is out of allowed range,"
> +				"DISABLING IT",	irq);
>   				irq = 0;	/* Bad IRQ */
>   			} else {
>   				if (request_irq
>   				    (irq, interrupt_pcl818, 0, "pcl818", dev)) {
>   					printk
> -					    (", unable to allocate IRQ %u, DISABLING IT",
> +					printk(KERN_CONT ", unable to allocate IRQ %u,"
> +					"DISABLING IT", irq);
>   					irq = 0;	/* Can't use IRQ */
>   				} else {
> -					printk(", irq=%u", irq);
> +					printk(KERN_CONT ", irq=%u", irq);
>   				}
>   			}
>   		}
>   	}
>
>   	dev->irq = irq;
> -	if (irq) {
> -		devpriv->irq_free = 1;
> -	} /* 1=we have allocated irq */
> -	else {
> +	if (irq)
> +		devpriv->irq_free = 1;   /* 1=we have allocated irq */
> +	else
>   		devpriv->irq_free = 0;
> -	}
> +
>   	devpriv->irq_blocked = 0;	/* number of subdevice which use IRQ */
>   	devpriv->ai_mode = 0;	/* mode of irq */
>
> @@ -1825,7 +1816,7 @@ static int pcl818_attach(struct comedi_device *dev, struct comedi_devconfig *it)
>   				 "pcl818 DMA (RTC)", dev)) {
>   			devpriv->dma_rtc = 1;
>   			devpriv->rtc_irq = RTC_IRQ;
> -			printk(", dma_irq=%u", devpriv->rtc_irq);
> +			printk(KERN_CONT ", dma_irq=%u", devpriv->rtc_irq);
>   		} else {
>   			RTC_lock--;
>   			if (RTC_lock == 0) {
> @@ -1850,23 +1841,26 @@ no_rtc:
>   		if (dma<  1)
>   			goto no_dma;	/* DMA disabled */
>   		if (((1<<  dma)&  this_board->DMAbits) == 0) {
> -			printk(", DMA is out of allowed range, FAIL!\n");
> +			printk(KERN_WARNING "DMA is out of allowed range, FAIL!\n");
>   			return -EINVAL;	/* Bad DMA */
>   		}
> +		printk(KERN_CONT ", DMA is in allowed range\n");
>   		ret = request_dma(dma, "pcl818");
>   		if (ret) {
> -			printk(", unable to allocate DMA %u, FAIL!\n", dma);
> +			printk(KERN_WARNING "unable to allocate DMA %u,"
> +			"FAIL!\n", dma);
>   			return -EBUSY;	/* DMA isn't free */
>   		}
> +		printk(KERN_CONT ", DMA allocated successfully\n");
>   		devpriv->dma = dma;
> -		printk(", dma=%u", dma);
> +		printk(KERN_CONT ", dma=%u", dma);
>   		pages = 2;	/* we need 16KB */
>   		devpriv->dmabuf[0] = __get_dma_pages(GFP_KERNEL, pages);
>   		if (!devpriv->dmabuf[0]) {
> -			printk(", unable to allocate DMA buffer, FAIL!\n");
> +			printk(KERN_WARNING "unable to allocate DMA buffer, FAIL!\n");
>   			/* maybe experiment with try_to_free_pages() will help .... */
>   			return -EBUSY;	/* no buffer :-( */
>   		}
> +		printk(KERN_CONT ", DMA buffer allocated successfully\n");
>   		devpriv->dmapages[0] = pages;
>   		devpriv->hwdmaptr[0] = virt_to_bus((void *)devpriv->dmabuf[0]);
>   		devpriv->hwdmasize[0] = (1<<  pages) * PAGE_SIZE;
> @@ -1874,10 +1868,11 @@ no_rtc:
>   		if (devpriv->dma_rtc == 0) {	/*  we must do duble buff :-( */
>   			devpriv->dmabuf[1] = __get_dma_pages(GFP_KERNEL, pages);
>   			if (!devpriv->dmabuf[1]) {
> -				printk
> -				    (", unable to allocate DMA buffer, FAIL!\n");
> +				printk(KERN_WARNING "unable to allocate DMA buffer, FAIL!\n");
>   				return -EBUSY;
>   			}
> +			printk(KERN_CONT ", DMA buffer allocated successfully\n");
>   			devpriv->dmapages[1] = pages;
>   			devpriv->hwdmaptr[1] =
>   			    virt_to_bus((void *)devpriv->dmabuf[1]);
> @@ -1901,11 +1896,11 @@ no_dma:
>   		if (check_single_ended(dev->iobase)) {
>   			s->n_chan = this_board->n_aichan_se;
>   			s->subdev_flags |= SDF_COMMON | SDF_GROUND;
> -			printk(", %dchans S.E. DAC", s->n_chan);
> +			printk(KERN_CONT ", %dchans S.E. DAC", s->n_chan);
>   		} else {
>   			s->n_chan = this_board->n_aichan_diff;
>   			s->subdev_flags |= SDF_DIFF;
> -			printk(", %dchans DIFF DAC", s->n_chan);
> +			printk(KERN_CONT ", %dchans DIFF DAC", s->n_chan);
>   		}
>   		s->maxdata = this_board->ai_maxdata;
>   		s->len_chanlist = s->n_chan;
> @@ -2017,11 +2012,10 @@ no_dma:
>   	}
>
>   	/* select 1/10MHz oscilator */
> -	if ((it->options[3] == 0) || (it->options[3] == 10)) {
> +	if ((it->options[3] == 0) || (it->options[3] == 10))
>   		devpriv->i8253_osc_base = 100;
> -	} else {
> +	else
>   		devpriv->i8253_osc_base = 1000;
> -	}
>
>   	/* max sampling speed */
>   	devpriv->ns_min = this_board->ns_min;
> @@ -2033,7 +2027,7 @@ no_dma:
>
>   	pcl818_reset(dev);
>
> -	printk("\n");
> +	printk(KERN_CONT "\n");
>
>   	return 0;
>   }

[PATCH v2] Staging: comedi: fix warning issue in pcl818.c

[Ravishankar]

From: Ravishankar <ravi.shankar@greenturtles.in>

This is a patch to the pcl818.c file that fixes up a printk,brace coding style and NULL initilization warning found by the checkpatch.pl tool

Signed-off-by: Ravishankar <ravishankarkm32@gmail.com>
---
 Arbitary mix of dev_printk functions and printk functions
 Introducing split printk lines
 KERN_CONT problems are fixed.		
 drivers/staging/comedi/drivers/pcl818.c |  126 ++++++++++++++----------------
 1 files changed, 59 insertions(+), 67 deletions(-)

diff --git a/drivers/staging/comedi/drivers/pcl818.c b/drivers/staging/comedi/drivers/pcl818.c
index b45a9bd..636e6fb 100644
--- a/drivers/staging/comedi/drivers/pcl818.c
+++ b/drivers/staging/comedi/drivers/pcl818.c
@@ -252,8 +252,8 @@ static int pcl818_attach(struct comedi_device *dev,
 static int pcl818_detach(struct comedi_device *dev);
 
 #ifdef unused
-static int RTC_lock = 0;	/* RTC lock */
-static int RTC_timer_lock = 0;	/* RTC int lock */
+static int RTC_lock;	/* RTC lock */
+static int RTC_timer_lock;	/* RTC int lock */
 #endif
 
 struct pcl818_board {
@@ -463,10 +463,8 @@ static int pcl818_ao_insn_read(struct comedi_device *dev,
 	int n;
 	int chan = CR_CHAN(insn->chanspec);
 
-	for (n = 0; n < insn->n; n++) {
+	for (n = 0; n < insn->n; n++)
 		data[n] = devpriv->ao_readback[chan];
-	}
-
 	return n;
 }
 
@@ -571,9 +569,8 @@ conv_finish:
 		return IRQ_HANDLED;
 	}
 	devpriv->act_chanlist_pos++;
-	if (devpriv->act_chanlist_pos >= devpriv->act_chanlist_len) {
+	 if (devpriv->act_chanlist_pos >= devpriv->act_chanlist_len)
 		devpriv->act_chanlist_pos = 0;
-	}
 	s->async->cur_chan++;
 	if (s->async->cur_chan >= devpriv->ai_n_chan) {
 		/*  printk("E"); */
@@ -645,9 +642,8 @@ static irqreturn_t interrupt_pcl818_ai_mode13_dma(int irq, void *d)
 		comedi_buf_put(s->async, ptr[bufptr++] >> 4);	/*  get one sample */
 
 		devpriv->act_chanlist_pos++;
-		if (devpriv->act_chanlist_pos >= devpriv->act_chanlist_len) {
+		if (devpriv->act_chanlist_pos >= devpriv->act_chanlist_len)
 			devpriv->act_chanlist_pos = 0;
-		}
 		s->async->cur_chan++;
 		if (s->async->cur_chan >= devpriv->ai_n_chan) {
 			s->async->cur_chan = 0;
@@ -805,11 +801,10 @@ static irqreturn_t interrupt_pcl818_ai_mode13_fifo(int irq, void *d)
 		return IRQ_HANDLED;
 	}
 
-	if (lo & 2) {
+	if (lo & 2)
 		len = 512;
-	} else {
+	else
 		len = 0;
-	}
 
 	for (i = 0; i < len; i++) {
 		lo = inb(dev->iobase + PCL818_FI_DATALO);
@@ -827,9 +822,8 @@ static irqreturn_t interrupt_pcl818_ai_mode13_fifo(int irq, void *d)
 		comedi_buf_put(s->async, (lo >> 4) | (inb(dev->iobase + PCL818_FI_DATAHI) << 4));	/*  get one sample */
 
 		devpriv->act_chanlist_pos++;
-		if (devpriv->act_chanlist_pos >= devpriv->act_chanlist_len) {
+		if (devpriv->act_chanlist_pos >= devpriv->act_chanlist_len)
 			devpriv->act_chanlist_pos = 0;
-		}
 		s->async->cur_chan++;
 		if (s->async->cur_chan >= devpriv->ai_n_chan) {
 			s->async->cur_chan = 0;
@@ -1009,7 +1003,7 @@ static int pcl818_ai_cmd_mode(int mode, struct comedi_device *dev,
 	int divisor1 = 0, divisor2 = 0;
 	unsigned int seglen;
 
-	printk("pcl818_ai_cmd_mode()\n");
+	dev_printk(KERN_DEBUG, dev->hw_dev, "pcl818_ai_cmd_mode()\n");
 	if ((!dev->irq) && (!devpriv->dma_rtc)) {
 		comedi_error(dev, "IRQ not defined!");
 		return -EINVAL;
@@ -1112,7 +1106,7 @@ static int pcl818_ai_cmd_mode(int mode, struct comedi_device *dev,
 		break;
 	}
 #endif
-	printk("pcl818_ai_cmd_mode() end\n");
+	dev_printk(KERN_DEBUG, dev->hw_dev, "pcl818_ai_cmd_mode() end\n");
 	return 0;
 }
 
@@ -1309,11 +1303,9 @@ static void setup_channel_list(struct comedi_device *dev,
 */
 static int check_single_ended(unsigned int port)
 {
-	if (inb(port + PCL818_STATUS) & 0x20) {
+	if (inb(port + PCL818_STATUS) & 0x20)
 		return 1;
-	} else {
-		return 0;
-	}
+	return 0;
 }
 
 /*
@@ -1352,9 +1344,8 @@ static int ai_cmdtest(struct comedi_device *dev, struct comedi_subdevice *s,
 	if (!cmd->stop_src || tmp != cmd->stop_src)
 		err++;
 
-	if (err) {
+	if (err)
 		return 1;
-	}
 
 	/* step 2: make sure trigger sources are unique and mutually compatible */
 
@@ -1377,9 +1368,8 @@ static int ai_cmdtest(struct comedi_device *dev, struct comedi_subdevice *s,
 	if (cmd->stop_src != TRIG_NONE && cmd->stop_src != TRIG_COUNT)
 		err++;
 
-	if (err) {
+	if (err)
 		return 2;
-	}
 
 	/* step 3: make sure arguments are trivially compatible */
 
@@ -1421,9 +1411,8 @@ static int ai_cmdtest(struct comedi_device *dev, struct comedi_subdevice *s,
 		}
 	}
 
-	if (err) {
+	if (err)
 		return 3;
-	}
 
 	/* step 4: fix up any arguments */
 
@@ -1438,9 +1427,8 @@ static int ai_cmdtest(struct comedi_device *dev, struct comedi_subdevice *s,
 			err++;
 	}
 
-	if (err) {
+	if (err)
 		return 4;
-	}
 
 	/* step 5: complain about special chanlist considerations */
 
@@ -1461,7 +1449,7 @@ static int ai_cmd(struct comedi_device *dev, struct comedi_subdevice *s)
 	struct comedi_cmd *cmd = &s->async->cmd;
 	int retval;
 
-	printk("pcl818_ai_cmd()\n");
+	dev_printk(KERN_DEBUG, dev->hw_dev, "pcl818_ai_cmd()\n");
 	devpriv->ai_n_chan = cmd->chanlist_len;
 	devpriv->ai_chanlist = cmd->chanlist;
 	devpriv->ai_flags = cmd->flags;
@@ -1470,17 +1458,16 @@ static int ai_cmd(struct comedi_device *dev, struct comedi_subdevice *s)
 	devpriv->ai_timer1 = 0;
 	devpriv->ai_timer2 = 0;
 
-	if (cmd->stop_src == TRIG_COUNT) {
+	if (cmd->stop_src == TRIG_COUNT)
 		devpriv->ai_scans = cmd->stop_arg;
-	} else {
+	else
 		devpriv->ai_scans = 0;
-	}
 
 	if (cmd->scan_begin_src == TRIG_FOLLOW) {	/*  mode 1, 3 */
 		if (cmd->convert_src == TRIG_TIMER) {	/*  mode 1 */
 			devpriv->ai_timer1 = cmd->convert_arg;
 			retval = pcl818_ai_cmd_mode(1, dev, s);
-			printk("pcl818_ai_cmd() end\n");
+			dev_printk(KERN_DEBUG, dev->hw_dev, "pcl818_ai_cmd() end\n");
 			return retval;
 		}
 		if (cmd->convert_src == TRIG_EXT) {	/*  mode 3 */
@@ -1499,7 +1486,7 @@ static int pcl818_ai_cancel(struct comedi_device *dev,
 			    struct comedi_subdevice *s)
 {
 	if (devpriv->irq_blocked > 0) {
-		printk("pcl818_ai_cancel()\n");
+		dev_printk(KERN_DEBUG, dev->hw_dev, "pcl818_ai_cancel()\n");
 		devpriv->irq_was_now_closed = 1;
 
 		switch (devpriv->ai_mode) {
@@ -1549,7 +1536,7 @@ static int pcl818_ai_cancel(struct comedi_device *dev,
 	}
 
 end:
-	printk("pcl818_ai_cancel() end\n");
+	dev_printk(KERN_DEBUG, dev->hw_dev, "pcl818_ai_cancel() end\n");
 	return 0;
 }
 
@@ -1633,11 +1620,10 @@ static int set_rtc_irq_bit(unsigned char bit)
 	save_flags(flags);
 	cli();
 	val = CMOS_READ(RTC_CONTROL);
-	if (bit) {
+	if (bit)
 		val |= RTC_PIE;
-	} else {
+	else
 		val &= ~RTC_PIE;
-	}
 	CMOS_WRITE(val, RTC_CONTROL);
 	CMOS_READ(RTC_INTR_FLAGS);
 	restore_flags(flags);
@@ -1754,22 +1740,23 @@ static int pcl818_attach(struct comedi_device *dev, struct comedi_devconfig *it)
 
 	/* claim our I/O space */
 	iobase = it->options[0];
-	printk("comedi%d: pcl818:  board=%s, ioport=0x%03lx",
-	       dev->minor, this_board->name, iobase);
+	dev_printk(KERN_INFO, dev->hw_dev,
+			"comedi%d: pcl818:  board=%s, ioport=0x%03lx",
+			dev->minor, this_board->name, iobase);
 	devpriv->io_range = this_board->io_range;
 	if ((this_board->fifo) && (it->options[2] == -1)) {	/*  we've board with FIFO and we want to use FIFO */
 		devpriv->io_range = PCLx1xFIFO_RANGE;
 		devpriv->usefifo = 1;
 	}
 	if (!request_region(iobase, devpriv->io_range, "pcl818")) {
-		printk("I/O port conflict\n");
+		dev_printk(KERN_ERR, dev->hw_dev, "I/O port conflict\n");
 		return -EIO;
 	}
 
 	dev->iobase = iobase;
 
 	if (pcl818_check(iobase)) {
-		printk(", I can't detect board. FAIL!\n");
+		dev_printk(KERN_ERR, dev->hw_dev, "I can't detect board. FAIL!\n");
 		return -EIO;
 	}
 
@@ -1781,31 +1768,30 @@ static int pcl818_attach(struct comedi_device *dev, struct comedi_devconfig *it)
 		irq = it->options[1];
 		if (irq) {	/* we want to use IRQ */
 			if (((1 << irq) & this_board->IRQbits) == 0) {
-				printk
-				    (", IRQ %u is out of allowed range, DISABLING IT",
-				     irq);
+				dev_printk(KERN_INFO, dev->hw_dev,
+						"IRQ %u is out of allowed range, DISABLING IT",
+						irq);
 				irq = 0;	/* Bad IRQ */
 			} else {
 				if (request_irq
 				    (irq, interrupt_pcl818, 0, "pcl818", dev)) {
-					printk
-					    (", unable to allocate IRQ %u, DISABLING IT",
-					     irq);
+					dev_printk(KERN_INFO, dev->hw_dev,
+							"unable to allocate IRQ %u, DISABLING IT",
+							irq);
 					irq = 0;	/* Can't use IRQ */
 				} else {
-					printk(", irq=%u", irq);
+					dev_printk(KERN_CONT, dev->hw_dev,
+								"irq=%u", irq);
 				}
 			}
 		}
 	}
 
 	dev->irq = irq;
-	if (irq) {
-		devpriv->irq_free = 1;
-	} /* 1=we have allocated irq */
-	else {
+	if (irq)
+		devpriv->irq_free = 1;   /* 1=we have allocated irq */
+	else
 		devpriv->irq_free = 0;
-	}
 	devpriv->irq_blocked = 0;	/* number of subdevice which use IRQ */
 	devpriv->ai_mode = 0;	/* mode of irq */
 
@@ -1825,7 +1811,8 @@ static int pcl818_attach(struct comedi_device *dev, struct comedi_devconfig *it)
 				 "pcl818 DMA (RTC)", dev)) {
 			devpriv->dma_rtc = 1;
 			devpriv->rtc_irq = RTC_IRQ;
-			printk(", dma_irq=%u", devpriv->rtc_irq);
+			dev_printk(KERN_DEBUG, dev->he_dev,
+					"dma_irq=%u", devpriv->rtc_irq);
 		} else {
 			RTC_lock--;
 			if (RTC_lock == 0) {
@@ -1850,20 +1837,24 @@ no_rtc:
 		if (dma < 1)
 			goto no_dma;	/* DMA disabled */
 		if (((1 << dma) & this_board->DMAbits) == 0) {
-			printk(", DMA is out of allowed range, FAIL!\n");
+			dev_printk(KERN_WARNING, dev->hw_dev,
+					"DMA is out of allowed range, FAIL!\n");
 			return -EINVAL;	/* Bad DMA */
 		}
 		ret = request_dma(dma, "pcl818");
 		if (ret) {
-			printk(", unable to allocate DMA %u, FAIL!\n", dma);
+			dev_printk(KERN_WARNING, dev->hw_dev,
+					"unable to allocate DMA %u, FAIL!\n",
+					dma);
 			return -EBUSY;	/* DMA isn't free */
 		}
 		devpriv->dma = dma;
-		printk(", dma=%u", dma);
+		dev_printk(KERN_DEBUG, dev->hw_dev, "dma=%u", dma);
 		pages = 2;	/* we need 16KB */
 		devpriv->dmabuf[0] = __get_dma_pages(GFP_KERNEL, pages);
 		if (!devpriv->dmabuf[0]) {
-			printk(", unable to allocate DMA buffer, FAIL!\n");
+			dev_printk(KERN_WARNING, dev->hw_dev,
+					"unable to allocate DMA buffer, FAIL!\n");
 			/* maybe experiment with try_to_free_pages() will help .... */
 			return -EBUSY;	/* no buffer :-( */
 		}
@@ -1874,8 +1865,8 @@ no_rtc:
 		if (devpriv->dma_rtc == 0) {	/*  we must do duble buff :-( */
 			devpriv->dmabuf[1] = __get_dma_pages(GFP_KERNEL, pages);
 			if (!devpriv->dmabuf[1]) {
-				printk
-				    (", unable to allocate DMA buffer, FAIL!\n");
+				dev_printk(KERN_WARNING, dev->hw_dev,
+						"unable to allocate DMA buffer, FAIL!\n");
 				return -EBUSY;
 			}
 			devpriv->dmapages[1] = pages;
@@ -1901,11 +1892,13 @@ no_dma:
 		if (check_single_ended(dev->iobase)) {
 			s->n_chan = this_board->n_aichan_se;
 			s->subdev_flags |= SDF_COMMON | SDF_GROUND;
-			printk(", %dchans S.E. DAC", s->n_chan);
+			dev_printk(KERN_DEBUG, dev->hw_dev,
+					"%dchans S.E. DAC", s->n_chan);
 		} else {
 			s->n_chan = this_board->n_aichan_diff;
 			s->subdev_flags |= SDF_DIFF;
-			printk(", %dchans DIFF DAC", s->n_chan);
+			dev_printk(KERN_DEBUG, dev->hw_dev,
+					"%dchans DIFF DAC", s->n_chan);
 		}
 		s->maxdata = this_board->ai_maxdata;
 		s->len_chanlist = s->n_chan;
@@ -2017,11 +2010,10 @@ no_dma:
 	}
 
 	/* select 1/10MHz oscilator */
-	if ((it->options[3] == 0) || (it->options[3] == 10)) {
+	if ((it->options[3] == 0) || (it->options[3] == 10))
 		devpriv->i8253_osc_base = 100;
-	} else {
+	else
 		devpriv->i8253_osc_base = 1000;
-	}
 
 	/* max sampling speed */
 	devpriv->ns_min = this_board->ns_min;
@@ -2033,7 +2025,7 @@ no_dma:
 
 	pcl818_reset(dev);
 
-	printk("\n");
+	printk(KERN_CONT "\n");
 
 	return 0;
 }
-- 
1.6.5.2