* [PATCH v2] Staging: comedi: fix warning issue in pcl818.c [not found] <[PATCH]Staging: Comedi: fix warning issue in pcl818.c> @ 2011-08-03 13:49 ` Ravishankar 2011-08-03 23:16 ` Ryan Mallon 0 siblings, 1 reply; 5+ messages in thread From: Ravishankar @ 2011-08-03 13:49 UTC (permalink / raw) To: gregkh, wfp5p; +Cc: devel, linux-kernel, Ravishankar, 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 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v2] Staging: comedi: fix warning issue in pcl818.c 2011-08-03 13:49 ` [PATCH v2] Staging: comedi: fix warning issue in pcl818.c Ravishankar @ 2011-08-03 23:16 ` Ryan Mallon 0 siblings, 0 replies; 5+ messages in thread From: Ryan Mallon @ 2011-08-03 23:16 UTC (permalink / raw) To: Ravishankar; +Cc: gregkh, wfp5p, devel, linux-kernel, Ravishankar 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; > } ^ permalink raw reply [flat|nested] 5+ messages in thread
[parent not found: <[PATCH]Staging: comedi: fix warning issue in pcl818.c>]
* [PATCH v2] Staging: comedi: fix warning issue in pcl818.c [not found] <[PATCH]Staging: comedi: fix warning issue in pcl818.c> @ 2011-08-08 7:45 ` Ravishankar 0 siblings, 0 replies; 5+ messages in thread From: Ravishankar @ 2011-08-08 7:45 UTC (permalink / raw) To: gregkh, wfp5p; +Cc: devel, linux-kernel, Ravishankar, 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 ^ permalink raw reply related [flat|nested] 5+ messages in thread
[parent not found: <[PATCH]Staging: comedi: fix warning issues in pcl818.c>]
* [PATCH v2] Staging: comedi: fix warning issue in pcl818.c [not found] <[PATCH]Staging: comedi: fix warning issues in pcl818.c> @ 2011-08-03 8:02 ` Ravishnakar 2011-08-03 10:25 ` Dan Carpenter 0 siblings, 1 reply; 5+ messages in thread From: Ravishnakar @ 2011-08-03 8:02 UTC (permalink / raw) To: gregkh, wfp5p; +Cc: devel, linux-kernel, Ravishankar, 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 | 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 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v2] Staging: comedi: fix warning issue in pcl818.c 2011-08-03 8:02 ` Ravishnakar @ 2011-08-03 10:25 ` Dan Carpenter 0 siblings, 0 replies; 5+ messages in thread From: Dan Carpenter @ 2011-08-03 10:25 UTC (permalink / raw) To: Ravishnakar; +Cc: gregkh, wfp5p, devel, linux-kernel, Ravishankar 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 ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2011-08-08 7:46 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <[PATCH]Staging: Comedi: fix warning issue in pcl818.c>
2011-08-03 13:49 ` [PATCH v2] Staging: comedi: fix warning issue in pcl818.c Ravishankar
2011-08-03 23:16 ` Ryan Mallon
[not found] <[PATCH]Staging: comedi: fix warning issue in pcl818.c>
2011-08-08 7:45 ` Ravishankar
[not found] <[PATCH]Staging: comedi: fix warning issues in pcl818.c>
2011-08-03 8:02 ` Ravishnakar
2011-08-03 10:25 ` Dan Carpenter
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox