Staging: comedi: range: tidy up comedi_check_chanlist()
authorFred Akers <knivey@botops.net>
Thu, 6 Mar 2014 06:57:18 +0000 (01:57 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 9 Mar 2014 02:59:48 +0000 (18:59 -0800)
Refactor this function to remove an extra indent level

Signed-off-by: Fred Akers <knivey@botops.net>
Reviewed-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/comedi/range.c

index 46b3da686384eac270221784c0ce3372b1a36d56..c4bb13bab934bbb8c32c607575f284589858c2ed 100644 (file)
@@ -143,29 +143,28 @@ int comedi_check_chanlist(struct comedi_subdevice *s, int n,
        unsigned int chanspec;
        int chan, range_len, i;
 
-       if (s->range_table || s->range_table_list) {
-               for (i = 0; i < n; i++) {
-                       chanspec = chanlist[i];
-                       chan = CR_CHAN(chanspec);
-                       if (s->range_table)
-                               range_len = s->range_table->length;
-                       else if (s->range_table_list && chan < s->n_chan)
-                               range_len = s->range_table_list[chan]->length;
-                       else
-                               range_len = 0;
-                       if (chan >= s->n_chan ||
-                           CR_RANGE(chanspec) >= range_len ||
-                           aref_invalid(s, chanspec)) {
-                               dev_warn(dev->class_dev,
-                                        "bad chanlist[%d]=0x%08x chan=%d range length=%d\n",
-                                        i, chanspec, chan, range_len);
-                               return -EINVAL;
-                       }
-               }
-       } else {
+       if (!s->range_table && !s->range_table_list) {
                dev_err(dev->class_dev, "(bug) no range type list!\n");
                return -EINVAL;
        }
+       for (i = 0; i < n; i++) {
+               chanspec = chanlist[i];
+               chan = CR_CHAN(chanspec);
+               if (s->range_table)
+                       range_len = s->range_table->length;
+               else if (s->range_table_list && chan < s->n_chan)
+                       range_len = s->range_table_list[chan]->length;
+               else
+                       range_len = 0;
+               if (chan >= s->n_chan ||
+                   CR_RANGE(chanspec) >= range_len ||
+                   aref_invalid(s, chanspec)) {
+                       dev_warn(dev->class_dev,
+                                "bad chanlist[%d]=0x%08x chan=%d range length=%d\n",
+                                i, chanspec, chan, range_len);
+                       return -EINVAL;
+               }
+       }
        return 0;
 }
 EXPORT_SYMBOL_GPL(comedi_check_chanlist);