diff options
| author | H Hartley Sweeten <hsweeten@visionengravers.com> | 2013-11-26 18:41:36 -0500 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-12-03 12:20:19 -0500 |
| commit | 5e1503aea7f46dcd5ebc2b929d7368a2f175bfb8 (patch) | |
| tree | 3cfaa5b2b9b8a32f0ce4050989410c1488b5ff11 | |
| parent | 5adbc856a7e2ac5c710492afcfdfb48325d81653 (diff) | |
staging: comedi: pcl816: convert some printk's to dev_dbg()
Convert these warning messages to dev_dbg() to minimize the noise.
Also, tidy them up a bit.
Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | drivers/staging/comedi/drivers/pcl816.c | 28 |
1 files changed, 12 insertions, 16 deletions
diff --git a/drivers/staging/comedi/drivers/pcl816.c b/drivers/staging/comedi/drivers/pcl816.c index 7b92aa50aa7f..8776c596bad2 100644 --- a/drivers/staging/comedi/drivers/pcl816.c +++ b/drivers/staging/comedi/drivers/pcl816.c | |||
| @@ -800,12 +800,10 @@ check_channel_list(struct comedi_device *dev, | |||
| 800 | (CR_CHAN(chansegment[i - 1]) + 1) % chanlen; | 800 | (CR_CHAN(chansegment[i - 1]) + 1) % chanlen; |
| 801 | if (nowmustbechan != CR_CHAN(chanlist[i])) { | 801 | if (nowmustbechan != CR_CHAN(chanlist[i])) { |
| 802 | /* channel list isn't continuous :-( */ | 802 | /* channel list isn't continuous :-( */ |
| 803 | printk(KERN_WARNING | 803 | dev_dbg(dev->class_dev, |
| 804 | "comedi%d: pcl816: channel list must " | 804 | "channel list must be continuous! chanlist[%i]=%d but must be %d or %d!\n", |
| 805 | "be continuous! chanlist[%i]=%d but " | 805 | i, CR_CHAN(chanlist[i]), nowmustbechan, |
| 806 | "must be %d or %d!\n", dev->minor, | 806 | CR_CHAN(chanlist[0])); |
| 807 | i, CR_CHAN(chanlist[i]), nowmustbechan, | ||
| 808 | CR_CHAN(chanlist[0])); | ||
| 809 | return 0; | 807 | return 0; |
| 810 | } | 808 | } |
| 811 | /* well, this is next correct channel in list */ | 809 | /* well, this is next correct channel in list */ |
| @@ -815,16 +813,14 @@ check_channel_list(struct comedi_device *dev, | |||
| 815 | /* check whole chanlist */ | 813 | /* check whole chanlist */ |
| 816 | for (i = 0, segpos = 0; i < chanlen; i++) { | 814 | for (i = 0, segpos = 0; i < chanlen; i++) { |
| 817 | if (chanlist[i] != chansegment[i % seglen]) { | 815 | if (chanlist[i] != chansegment[i % seglen]) { |
| 818 | printk(KERN_WARNING | 816 | dev_dbg(dev->class_dev, |
| 819 | "comedi%d: pcl816: bad channel or range" | 817 | "bad channel or range number! chanlist[%i]=%d,%d,%d and not %d,%d,%d!\n", |
| 820 | " number! chanlist[%i]=%d,%d,%d and not" | 818 | i, CR_CHAN(chansegment[i]), |
| 821 | " %d,%d,%d!\n", dev->minor, i, | 819 | CR_RANGE(chansegment[i]), |
| 822 | CR_CHAN(chansegment[i]), | 820 | CR_AREF(chansegment[i]), |
| 823 | CR_RANGE(chansegment[i]), | 821 | CR_CHAN(chanlist[i % seglen]), |
| 824 | CR_AREF(chansegment[i]), | 822 | CR_RANGE(chanlist[i % seglen]), |
| 825 | CR_CHAN(chanlist[i % seglen]), | 823 | CR_AREF(chansegment[i % seglen])); |
| 826 | CR_RANGE(chanlist[i % seglen]), | ||
| 827 | CR_AREF(chansegment[i % seglen])); | ||
| 828 | return 0; /* chan/gain list is strange */ | 824 | return 0; /* chan/gain list is strange */ |
| 829 | } | 825 | } |
| 830 | } | 826 | } |