Staging: comedi: fix brace coding style issue in pcl818.c
authorRavishankar karkala Mallikarjunayya <ravishankar.km@greenturtles.in>
Sat, 19 Nov 2011 04:36:37 +0000 (10:06 +0530)
committerGreg Kroah-Hartman <gregkh@suse.de>
Sun, 27 Nov 2011 02:45:18 +0000 (18:45 -0800)
This is a patch to the pcl818.c file that fixes up a brace
coding style warning found by the checkpatch.pl tool.

Signed-off-by: Ravishankar Karkala Mallikarjunayya <ravishankar.km@greenturtles.in>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/comedi/drivers/pcl818.c

index b45a9bd8b489add24cb13a230b757ed09be78503..ca033de4a4e28b52d5a11d2acaa2b74f049d1d8d 100644 (file)
@@ -463,9 +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 +570,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 +644,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 +804,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 +825,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;
@@ -1309,11 +1307,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 +1348,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 +1372,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 +1415,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 +1431,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 */
 
@@ -1470,11 +1462,10 @@ 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 */
@@ -1633,11 +1624,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);
@@ -1800,12 +1791,11 @@ static int pcl818_attach(struct comedi_device *dev, struct comedi_devconfig *it)
        }
 
        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 */
 
@@ -2017,11 +2007,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;