diff options
| author | Roel Kluin <roel.kluin@gmail.com> | 2009-10-23 18:03:08 +0400 |
|---|---|---|
| committer | Takashi Iwai <tiwai@suse.de> | 2009-10-30 14:25:07 +0300 |
| commit | 84ed1a1942e8c28fb4c23a6235ec48672fc43e49 (patch) | |
| tree | 67b0d7a9c639cddd9cb6319ed67e0c0d6a680dd7 /sound/pci/ctxfi | |
| parent | a688e4885c1aa6b88ab5ffa64655bacc01749c9e (diff) | |
| download | linux-84ed1a1942e8c28fb4c23a6235ec48672fc43e49.tar.xz | |
ALSA: Cleanup redundant tests on unsigned
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>
Diffstat (limited to 'sound/pci/ctxfi')
| -rw-r--r-- | sound/pci/ctxfi/ctatc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/pci/ctxfi/ctatc.c b/sound/pci/ctxfi/ctatc.c index b1b3a644f738..6bfce99b42a2 100644 --- a/sound/pci/ctxfi/ctatc.c +++ b/sound/pci/ctxfi/ctatc.c @@ -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 { |