diff options
author | Vasiliy Kulikov <segooon@gmail.com> | 2010-09-12 22:56:35 +0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2010-09-16 12:37:34 -0700 |
commit | 2503a26554c7e73c544fc97f47c1d41faa88bd07 (patch) | |
tree | 5859b8aed34415341f9b8e64ceb854bc78f8b447 | |
parent | cae16a1742ad3cc19f15e78b821b94448e330ad9 (diff) | |
download | linux-3.10-2503a26554c7e73c544fc97f47c1d41faa88bd07.tar.gz linux-3.10-2503a26554c7e73c544fc97f47c1d41faa88bd07.tar.bz2 linux-3.10-2503a26554c7e73c544fc97f47c1d41faa88bd07.zip |
staging: comedi: fix signess bug
ai_count and ao_counter are unsigned, check for < 0 doesn't make sense.
Cast them to int.
Signed-off-by: Vasiliy Kulikov <segooon@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-rw-r--r-- | drivers/staging/comedi/drivers/cb_pcidas64.c | 2 | ||||
-rw-r--r-- | drivers/staging/comedi/drivers/usbdux.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/drivers/staging/comedi/drivers/cb_pcidas64.c b/drivers/staging/comedi/drivers/cb_pcidas64.c index 9b1dd3cc212..2583e16cd02 100644 --- a/drivers/staging/comedi/drivers/cb_pcidas64.c +++ b/drivers/staging/comedi/drivers/cb_pcidas64.c @@ -3078,7 +3078,7 @@ static void handle_ai_interrupt(struct comedi_device *dev, spin_unlock_irqrestore(&dev->spinlock, flags); } /* if we are have all the data, then quit */ - if ((cmd->stop_src == TRIG_COUNT && priv(dev)->ai_count <= 0) || + if ((cmd->stop_src == TRIG_COUNT && (int)priv(dev)->ai_count <= 0) || (cmd->stop_src == TRIG_EXT && (status & ADC_STOP_BIT))) { async->events |= COMEDI_CB_EOA; } diff --git a/drivers/staging/comedi/drivers/usbdux.c b/drivers/staging/comedi/drivers/usbdux.c index 4b320b1ff82..6131e2dd059 100644 --- a/drivers/staging/comedi/drivers/usbdux.c +++ b/drivers/staging/comedi/drivers/usbdux.c @@ -649,7 +649,7 @@ static void usbduxsub_ao_IsocIrq(struct urb *urb) /* normal operation: executing a command in this subdevice */ this_usbduxsub->ao_counter--; - if (this_usbduxsub->ao_counter <= 0) { + if ((int)this_usbduxsub->ao_counter <= 0) { /* timer zero */ this_usbduxsub->ao_counter = this_usbduxsub->ao_timer; |