diff options
| author | Manu Abraham <abraham.manu@gmail.com> | 2007-10-10 11:12:13 +0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2008-12-29 20:53:18 +0100 |
| commit | 417dd69de843a2f67a9ff6e21df7de5be5dd8204 (patch) | |
| tree | 6401598cef0d3c1a9c08246de7c2294653201850 /drivers/media/dvb/frontends/stb0899_drv.c | |
| parent | V4L/DVB (9419): Bug: unnecessary large current causes large phasor errors (diff) | |
| download | linux-417dd69de843a2f67a9ff6e21df7de5be5dd8204.tar.xz linux-417dd69de843a2f67a9ff6e21df7de5be5dd8204.zip | |
V4L/DVB (9420): return -EINVAL for invalid parameters
Signed-off-by: Manu Abraham <manu@linuxtv.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media/dvb/frontends/stb0899_drv.c')
| -rw-r--r-- | drivers/media/dvb/frontends/stb0899_drv.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/drivers/media/dvb/frontends/stb0899_drv.c b/drivers/media/dvb/frontends/stb0899_drv.c index cc5db0e8ad06..6e6373002b6c 100644 --- a/drivers/media/dvb/frontends/stb0899_drv.c +++ b/drivers/media/dvb/frontends/stb0899_drv.c @@ -967,7 +967,7 @@ static int stb0899_read_signal_strength(struct dvb_frontend *fe, u16 *strength) break; default: dprintk(verbose, FE_DEBUG, 1, "Unsupported delivery system"); - break; + return -EINVAL; } return 0; @@ -1023,7 +1023,7 @@ static int stb0899_read_snr(struct dvb_frontend *fe, u16 *snr) break; default: dprintk(verbose, FE_DEBUG, 1, "Unsupported delivery system"); - break; + return -EINVAL; } return 0; @@ -1085,7 +1085,7 @@ static int stb0899_read_status(struct dvb_frontend *fe, enum fe_status *status) break; default: dprintk(verbose, FE_DEBUG, 1, "Unsupported delivery system"); - break; + return -EINVAL; } return 0; } @@ -1144,6 +1144,7 @@ static int stb0899_read_ber(struct dvb_frontend *fe, u32 *ber) break; default: dprintk(verbose, FE_DEBUG, 1, "Unsupported delivery system"); + return -EINVAL; } return 0; @@ -1201,7 +1202,7 @@ static int stb0899_set_tone(struct dvb_frontend *fe, fe_sec_tone_mode_t tone) stb0899_write_reg(state, STB0899_DISEQCOCFG, 0x20); break; default: - break; + return -EINVAL; } return 0; } |