Staging: Comedi adv_pci1710: Cleaned up comments
authorTomas Melin <tomas.melin@iki.fi>
Mon, 7 May 2012 19:49:48 +0000 (22:49 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 9 May 2012 20:23:40 +0000 (13:23 -0700)
Removed useless comments and cleaned up text.

Signed-off-by: Tomas Melin <tomas.melin@iki.fi>
Acked-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/comedi/drivers/adv_pci1710.c

index ff098c0bb8db26035afe72994c8e0f814348c515..6dd4f4a486637e630bcc861172428336e1133ef3 100644 (file)
@@ -1164,33 +1164,31 @@ static int check_channel_list(struct comedi_device *dev,
        if (n_chan == 1)
                return 1; /* seglen=1 */
 
-       chansegment[0] = chanlist[0];   /*  first channel is every time ok */
-       for (i = 1, seglen = 1; i < n_chan; i++, seglen++) {    /*  build part of chanlist */
-               /*  printk("%d. %d %d\n",i,CR_CHAN(chanlist[i]),CR_RANGE(chanlist[i])); */
+       chansegment[0] = chanlist[0]; /*  first channel is every time ok */
+       for (i = 1, seglen = 1; i < n_chan; i++, seglen++) {
                if (chanlist[0] == chanlist[i])
-                       break;  /*  we detect loop, this must by finish */
-               if (CR_CHAN(chanlist[i]) & 1)   /*  odd channel cann't by differencial */
+                       break;  /*  we detected a loop, stop */
+               if (CR_CHAN(chanlist[i]) & 1)
                        if (CR_AREF(chanlist[i]) == AREF_DIFF) {
                                comedi_error(dev,
-                                            "Odd channel can't be differential input!\n");
+                                            "Odd channel cannot be differential input!\n");
                                return 0;
                        }
                nowmustbechan =
                        (CR_CHAN(chansegment[i - 1]) + 1) % s->n_chan;
                if (CR_AREF(chansegment[i - 1]) == AREF_DIFF)
                        nowmustbechan = (nowmustbechan + 1) % s->n_chan;
-               if (nowmustbechan != CR_CHAN(chanlist[i])) {    /*  channel list isn't continuous :-( */
+               if (nowmustbechan != CR_CHAN(chanlist[i])) {
                        printk
                                ("channel list must be continuous! chanlist[%i]=%d but must be %d or %d!\n",
                                 i, CR_CHAN(chanlist[i]), nowmustbechan,
                                 CR_CHAN(chanlist[0]));
                        return 0;
                }
-               chansegment[i] = chanlist[i];   /*  well, this is next correct channel in list */
+               chansegment[i] = chanlist[i]; /* next correct channel in list */
        }
 
-       for (i = 0, segpos = 0; i < n_chan; i++) {      /*  check whole chanlist */
-               /* printk("%d %d=%d %d\n",CR_CHAN(chansegment[i%seglen]),CR_RANGE(chansegment[i%seglen]),CR_CHAN(chanlist[i]),CR_RANGE(chanlist[i])); */
+       for (i = 0, segpos = 0; i < n_chan; i++) {
                if (chanlist[i] != chansegment[i % seglen]) {
                        printk
                                ("bad channel, reference or range number! chanlist[%i]=%d,%d,%d and not %d,%d,%d!\n",
@@ -1200,7 +1198,7 @@ static int check_channel_list(struct comedi_device *dev,
                                 CR_CHAN(chanlist[i % seglen]),
                                 CR_RANGE(chanlist[i % seglen]),
                                 CR_AREF(chansegment[i % seglen]));
-                       return 0;       /*  chan/gain list is strange */
+                       return 0; /*  chan/gain list is strange */
                }
        }
        return seglen;