diff options
| author |   Colin Ian King <colin.king@canonical.com> | 2017-01-11 14:24:33 -0200 |
|---|---|---|
| committer |   Mauro Carvalho Chehab <mchehab@s-opensource.com> | 2017-02-03 11:18:33 -0200 |
| commit | c099b73a29278c22a768a8c203c584e803388490 (patch) | |
| tree | c7bfea8302362ebc53f816791a9a6891bd754351 | |
| parent | [media] tc358743: ctrl_detect_tx_5v should always be updated (diff) | |
| download | linux-dev-c099b73a29278c22a768a8c203c584e803388490.tar.xz linux-dev-c099b73a29278c22a768a8c203c584e803388490.zip | |
[media] gspca_stv06xx: remove redundant check for err < 0
The comparison of err < 0 is redundant as err has been previously
been assigned to 0 and has not changed. Remove the redundant check.
Fixes CoverityScan CID#703363 ("Logically dead code")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
| -rw-r--r-- | drivers/media/usb/gspca/stv06xx/stv06xx_vv6410.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/media/usb/gspca/stv06xx/stv06xx_vv6410.c b/drivers/media/usb/gspca/stv06xx/stv06xx_vv6410.c index 9f490f55d367..d265e6b00994 100644 --- a/drivers/media/usb/gspca/stv06xx/stv06xx_vv6410.c +++ b/drivers/media/usb/gspca/stv06xx/stv06xx_vv6410.c @@ -116,9 +116,6 @@ static int vv6410_init(struct sd *sd) for (i = 0; i < ARRAY_SIZE(stv_bridge_init); i++) stv06xx_write_bridge(sd, stv_bridge_init[i].addr, stv_bridge_init[i].data); - if (err < 0) - return err; - err = stv06xx_write_sensor_bytes(sd, (u8 *) vv6410_sensor_init, ARRAY_SIZE(vv6410_sensor_init)); return (err < 0) ? err : 0; |