summaryrefslogtreecommitdiffstats
path: root/drivers/media
diff options
context:
space:
mode:
authorChristophe Thommeret <hftom@free.fr>2008-10-15 20:01:32 -0300
committerMauro Carvalho Chehab <mchehab@redhat.com>2008-10-17 17:29:46 -0300
commit745632147d54cd4db6812045351510ebd3981a24 (patch)
treecc1e0d8dada7b2ce4bad2851c3ffea320bd61322 /drivers/media
parent430189da042f8cc3305b8fbbce18ea103501fb90 (diff)
downloadop-kernel-dev-745632147d54cd4db6812045351510ebd3981a24.zip
op-kernel-dev-745632147d54cd4db6812045351510ebd3981a24.tar.gz
V4L/DVB (9270): cx24116: optimise emulated auto-pilot
Christophe divulged the following: "Since 8PSK (and higher mod) signals are very likely to have pilot symbols, pilot_auto should start with pilot_on for 8PSK. And since QPSK signals are unlikely to have pilot, pilot_auto should start with pilot_off for QPSK. Without the patch: QPSK tuning delay: ~100ms 8PSK tuning delay: ~900ms with patch: QPSK tuning delay: ~100ms 8PSK tuning delay: ~100ms " This patch does as Cristophe asks. In emulated auto-pilot we choose pilot-on for tuning PSK_8 and pilot-off for tuning QPSK. Signed-off-by: Darron Broad <darron@kewl.org> Signed-off-by: Steven Toth <stoth@linuxtv.org> Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media')
-rw-r--r--drivers/media/dvb/frontends/cx24116.c5
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/media/dvb/frontends/cx24116.c b/drivers/media/dvb/frontends/cx24116.c
index 8afdfbe..fe62f92 100644
--- a/drivers/media/dvb/frontends/cx24116.c
+++ b/drivers/media/dvb/frontends/cx24116.c
@@ -1248,7 +1248,10 @@ static int cx24116_set_frontend(struct dvb_frontend *fe,
switch (c->pilot) {
case PILOT_AUTO: /* Not supported but emulated */
- retune = 2; /* Fall-through */
+ state->dnxt.pilot_val = (c->modulation == QPSK)
+ ? CX24116_PILOT_OFF : CX24116_PILOT_ON;
+ retune = 2;
+ break;
case PILOT_OFF:
state->dnxt.pilot_val = CX24116_PILOT_OFF;
break;
OpenPOWER on IntegriCloud