diff options
author | Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2014-11-05 06:58:07 -0500 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2014-11-05 06:58:07 -0500 |
commit | 6deaca2bc3224aa138c93447000677ba5731007e (patch) | |
tree | c8cdc19e84236748af466809f895a3ad54684e2a | |
parent | 5eb2829212890d2db9883562310b916cc86d72ac (diff) |
[media] cx23110: Fix return code for cx24110_set_fec()
When a parameter is invalid, the right return code is
-EINVAL.
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
-rw-r--r-- | drivers/media/dvb-frontends/cx24110.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/media/dvb-frontends/cx24110.c b/drivers/media/dvb-frontends/cx24110.c index 4f5c992afe67..5a31b3f59306 100644 --- a/drivers/media/dvb-frontends/cx24110.c +++ b/drivers/media/dvb-frontends/cx24110.c | |||
@@ -217,8 +217,7 @@ static int cx24110_set_fec (struct cx24110_state* state, fe_code_rate_t fec) | |||
217 | cx24110_writereg(state, 0x1b, g2[fec]); | 217 | cx24110_writereg(state, 0x1b, g2[fec]); |
218 | /* not sure if this is the right way: I always used AutoAcq mode */ | 218 | /* not sure if this is the right way: I always used AutoAcq mode */ |
219 | } else | 219 | } else |
220 | return -EOPNOTSUPP; | 220 | return -EINVAL; |
221 | /* fixme (low): which is the correct return code? */ | ||
222 | } | 221 | } |
223 | return 0; | 222 | return 0; |
224 | } | 223 | } |