summaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorMauro Carvalho Chehab <mchehab@osg.samsung.com>2014-11-05 12:58:07 +0100
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>2014-11-05 12:58:07 +0100
commit6deaca2bc3224aa138c93447000677ba5731007e (patch)
treec8cdc19e84236748af466809f895a3ad54684e2a /drivers
parent[media] cx24110: Fix whitespaces at cx24110_set_fec() (diff)
downloadlinux-6deaca2bc3224aa138c93447000677ba5731007e.tar.xz
linux-6deaca2bc3224aa138c93447000677ba5731007e.zip
[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>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/media/dvb-frontends/cx24110.c3
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)
cx24110_writereg(state, 0x1b, g2[fec]);
/* not sure if this is the right way: I always used AutoAcq mode */
} else
- return -EOPNOTSUPP;
-/* fixme (low): which is the correct return code? */
+ return -EINVAL;
}
return 0;
}