ALSA: pcm: Use __ffs() instead of ffs() in snd_mask_min()
authorLars-Peter Clausen <lars@metafoo.de>
Mon, 29 Dec 2014 18:41:44 +0000 (19:41 +0100)
committerTakashi Iwai <tiwai@suse.de>
Tue, 30 Dec 2014 15:41:55 +0000 (16:41 +0100)
The difference between __ffs and ffs is that ffs will return a one based
index whereas __ffs will return a zero based index. Furthermore ffs will
check if the passed value is zero and return zero in that case, whereas
__ffs behavior is undefined if the passed parameter is 0.

Since we already check if the mask is 0 before calling ffs and also subtract
1 from the result __ffs is the better choice.

Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
include/sound/pcm_params.h

index 042049bab0b91edf07ae654d27840ebf64787426..c99e20b0c4c9956ba0615d9ea7d70d1c2c67a393 100644 (file)
@@ -92,7 +92,7 @@ static inline unsigned int snd_mask_min(const struct snd_mask *mask)
        int i;
        for (i = 0; i < SNDRV_MASK_SIZE; i++) {
                if (mask->bits[i])
-                       return ffs(mask->bits[i]) - 1 + (i << 5);
+                       return __ffs(mask->bits[i]) + (i << 5);
        }
        return 0;
 }