ALSA: Cleanup redundant tests on unsigned
authorRoel Kluin <roel.kluin@gmail.com>
Fri, 23 Oct 2009 14:03:08 +0000 (16:03 +0200)
committerTakashi Iwai <tiwai@suse.de>
Fri, 30 Oct 2009 11:25:07 +0000 (12:25 +0100)
The variables are unsigned so the test `>= 0' is always true,
the `< 0' test always fails. In these cases the other part of
the test catches wrapped values.

In dac_audio_write() there does not occur a test for wrapped
values, but the test appears redundant.

Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/oss/sh_dac_audio.c
sound/pci/ca0106/ca0106_proc.c
sound/pci/ctxfi/ctatc.c
sound/pci/emu10k1/emu10k1x.c
sound/pci/emu10k1/emuproc.c
sound/pci/emu10k1/io.c
sound/soc/codecs/tlv320aic23.c

index b2ed8757542ac50fd49f88557118bbb703c88c04..4153752507e392f936b7c41844559402c06ba03f 100644 (file)
@@ -164,9 +164,6 @@ static ssize_t dac_audio_write(struct file *file, const char *buf, size_t count,
        int free;
        int nbytes;
 
-       if (count < 0)
-               return -EINVAL;
-
        if (!count) {
                dac_audio_sync();
                return 0;
index c62b7d10ec61644c763beb0be0c1a10073aee2b3..15523e60351cfd720ffc06e355559b4af0fddbb6 100644 (file)
@@ -304,7 +304,7 @@ static void snd_ca0106_proc_reg_write32(struct snd_info_entry *entry,
         while (!snd_info_get_line(buffer, line, sizeof(line))) {
                 if (sscanf(line, "%x %x", &reg, &val) != 2)
                         continue;
-                if ((reg < 0x40) && (reg >=0) && (val <= 0xffffffff) ) {
+               if (reg < 0x40 && val <= 0xffffffff) {
                        spin_lock_irqsave(&emu->emu_lock, flags);
                        outl(val, emu->port + (reg & 0xfffffffc));
                        spin_unlock_irqrestore(&emu->emu_lock, flags);
@@ -405,7 +405,7 @@ static void snd_ca0106_proc_reg_write(struct snd_info_entry *entry,
         while (!snd_info_get_line(buffer, line, sizeof(line))) {
                 if (sscanf(line, "%x %x %x", &reg, &channel_id, &val) != 3)
                         continue;
-                if ((reg < 0x80) && (reg >=0) && (val <= 0xffffffff) && (channel_id >=0) && (channel_id <= 3) )
+               if (reg < 0x80 && val <= 0xffffffff && channel_id <= 3)
                         snd_ca0106_ptr_write(emu, reg, channel_id, val);
         }
 }
index b1b3a644f73804a186263df3241f5c49650c5f48..6bfce99b42a254dbd97dd4d90876c0c6a74708fb 100644 (file)
@@ -240,7 +240,7 @@ static int select_rom(unsigned int pitch)
        } else if (pitch == 0x02000000) {
                /* pitch == 2 */
                return 3;
-       } else if (pitch >= 0x0 && pitch <= 0x08000000) {
+       } else if (pitch <= 0x08000000) {
                /* 0 <= pitch <= 8 */
                return 0;
        } else {
index 36e08bd2b3cc6ff8d2a4aa74e8f51e233bb62477..6b8ae7b5cd54df55a04e00bbbcba00bd30d492b5 100644 (file)
@@ -1040,8 +1040,7 @@ static void snd_emu10k1x_proc_reg_write(struct snd_info_entry *entry,
                if (sscanf(line, "%x %x %x", &reg, &channel_id, &val) != 3)
                        continue;
 
-               if ((reg < 0x49) && (reg >= 0) && (val <= 0xffffffff) 
-                   && (channel_id >= 0) && (channel_id <= 2) )
+               if (reg < 0x49 && val <= 0xffffffff && channel_id <= 2)
                        snd_emu10k1x_ptr_write(emu, reg, channel_id, val);
        }
 }
index 216f9748aff5b00e70ee943a1e4c55e368b76d6d..baa7cd508cd84aab0687e39ef849dc8a77d1d696 100644 (file)
@@ -451,7 +451,7 @@ static void snd_emu_proc_io_reg_write(struct snd_info_entry *entry,
        while (!snd_info_get_line(buffer, line, sizeof(line))) {
                if (sscanf(line, "%x %x", &reg, &val) != 2)
                        continue;
-               if ((reg < 0x40) && (reg >= 0) && (val <= 0xffffffff) ) {
+               if (reg < 0x40 && val <= 0xffffffff) {
                        spin_lock_irqsave(&emu->emu_lock, flags);
                        outl(val, emu->port + (reg & 0xfffffffc));
                        spin_unlock_irqrestore(&emu->emu_lock, flags);
@@ -527,7 +527,7 @@ static void snd_emu_proc_ptr_reg_write(struct snd_info_entry *entry,
        while (!snd_info_get_line(buffer, line, sizeof(line))) {
                if (sscanf(line, "%x %x %x", &reg, &channel_id, &val) != 3)
                        continue;
-               if ((reg < 0xa0) && (reg >= 0) && (val <= 0xffffffff) && (channel_id >= 0) && (channel_id <= 3) )
+               if (reg < 0xa0 && val <= 0xffffffff && channel_id <= 3)
                        snd_ptr_write(emu, iobase, reg, channel_id, val);
        }
 }
index c1a5aa15af8fdb55101928dd6aaaaab2b617a374..5ef7080e14d090c2c83ea5b6edb0705f0e09a249 100644 (file)
@@ -256,7 +256,7 @@ int snd_emu1010_fpga_write(struct snd_emu10k1 * emu, u32 reg, u32 value)
        if (reg > 0x3f)
                return 1;
        reg += 0x40; /* 0x40 upwards are registers. */
-       if (value < 0 || value > 0x3f) /* 0 to 0x3f are values */
+       if (value > 0x3f) /* 0 to 0x3f are values */
                return 1;
        spin_lock_irqsave(&emu->emu_lock, flags);
        outl(reg, emu->port + A_IOCFG);
index 0b8dcb5cd729281bfd80264acdb5a58b9025636e..35606ae60868bc964ded7236ae0890d8098682c8 100644 (file)
@@ -85,7 +85,7 @@ static int tlv320aic23_write(struct snd_soc_codec *codec, unsigned int reg,
         * of data into val
         */
 
-       if ((reg < 0 || reg > 9) && (reg != 15)) {
+       if (reg > 9 && reg != 15) {
                printk(KERN_WARNING "%s Invalid register R%u\n", __func__, reg);
                return -1;
        }