Staging: line6: ffzb returns an unsigned integer
authorFrederik Deweerdt <frederik.deweerdt@xprog.eu>
Mon, 14 Sep 2009 08:52:37 +0000 (08:52 +0000)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 15 Sep 2009 19:02:34 +0000 (12:02 -0700)
find_first_zero_bit returns a positive value, use it accordingly.

Signed-off-by: Frederik Deweerdt <frederik.deweerdt@xprog.eu>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/line6/capture.c

index 8393e25..ea2060b 100644 (file)
@@ -26,7 +26,7 @@
 */
 static int submit_audio_in_urb(struct snd_pcm_substream *substream)
 {
-       int index;
+       unsigned int index;
        unsigned long flags;
        struct snd_line6_pcm *line6pcm = snd_pcm_substream_chip(substream);
        int i, urb_size;
@@ -35,7 +35,7 @@ static int submit_audio_in_urb(struct snd_pcm_substream *substream)
        spin_lock_irqsave(&line6pcm->lock_audio_in, flags);
        index = find_first_zero_bit(&line6pcm->active_urb_in, LINE6_ISO_BUFFERS);
 
-       if (index < 0 || index >= LINE6_ISO_BUFFERS) {
+       if (index >= LINE6_ISO_BUFFERS) {
                spin_unlock_irqrestore(&line6pcm->lock_audio_in, flags);
                dev_err(s2m(substream), "no free URB found\n");
                return -EINVAL;