diff options
author | Stefan Richter <stefanr@s5r6.in-berlin.de> | 2008-09-11 13:22:53 -0400 |
---|---|---|
committer | Stefan Richter <stefanr@s5r6.in-berlin.de> | 2008-10-15 16:21:09 -0400 |
commit | d98562d12f71284593b3a5db020d6f2655061efe (patch) | |
tree | b6ebacb0acfbc6805ce7cd2cb17d061c2d0c8e23 /drivers/ieee1394/dv1394.c | |
parent | f22e52b89e036fd12b9374212da8b5d4a447bd1e (diff) |
ieee1394: dv1394, video1394: remove unnecessary expressions
init->channel and v.buffer are unsigned and tests for < 0 therefore
always false. gcc knows this and eliminates the code, but anyway...
Reported by Roel Kluin.
Signed-off-by: Stefan Richter <stefanr@s5r6.in-berlin.de>
Diffstat (limited to 'drivers/ieee1394/dv1394.c')
-rw-r--r-- | drivers/ieee1394/dv1394.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/ieee1394/dv1394.c b/drivers/ieee1394/dv1394.c index b6eb2cf25914..df70f51279d8 100644 --- a/drivers/ieee1394/dv1394.c +++ b/drivers/ieee1394/dv1394.c | |||
@@ -918,7 +918,7 @@ static int do_dv1394_init(struct video_card *video, struct dv1394_init *init) | |||
918 | /* default SYT offset is 3 cycles */ | 918 | /* default SYT offset is 3 cycles */ |
919 | init->syt_offset = 3; | 919 | init->syt_offset = 3; |
920 | 920 | ||
921 | if ( (init->channel > 63) || (init->channel < 0) ) | 921 | if (init->channel > 63) |
922 | init->channel = 63; | 922 | init->channel = 63; |
923 | 923 | ||
924 | chan_mask = (u64)1 << init->channel; | 924 | chan_mask = (u64)1 << init->channel; |