diff options
| author | Stefan Richter <stefanr@s5r6.in-berlin.de> | 2008-09-11 19:22:53 +0200 |
|---|---|---|
| committer | Stefan Richter <stefanr@s5r6.in-berlin.de> | 2008-10-15 22:21:09 +0200 |
| commit | d98562d12f71284593b3a5db020d6f2655061efe (patch) | |
| tree | b6ebacb0acfbc6805ce7cd2cb17d061c2d0c8e23 /drivers/ieee1394/dv1394.c | |
| parent | f22e52b89e036fd12b9374212da8b5d4a447bd1e (diff) | |
| download | lwn-d98562d12f71284593b3a5db020d6f2655061efe.tar.gz lwn-d98562d12f71284593b3a5db020d6f2655061efe.zip | |
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) /* default SYT offset is 3 cycles */ init->syt_offset = 3; - if ( (init->channel > 63) || (init->channel < 0) ) + if (init->channel > 63) init->channel = 63; chan_mask = (u64)1 << init->channel; |