diff options
author | Christophe Thommeret <hftom@free.fr> | 2008-10-15 19:01:32 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2008-10-17 16:29:46 -0400 |
commit | 745632147d54cd4db6812045351510ebd3981a24 (patch) | |
tree | cc1e0d8dada7b2ce4bad2851c3ffea320bd61322 /drivers/media | |
parent | 430189da042f8cc3305b8fbbce18ea103501fb90 (diff) |
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.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/media/dvb/frontends/cx24116.c b/drivers/media/dvb/frontends/cx24116.c index 8afdfbec551b..fe62f92f6601 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, | |||
1248 | 1248 | ||
1249 | switch (c->pilot) { | 1249 | switch (c->pilot) { |
1250 | case PILOT_AUTO: /* Not supported but emulated */ | 1250 | case PILOT_AUTO: /* Not supported but emulated */ |
1251 | retune = 2; /* Fall-through */ | 1251 | state->dnxt.pilot_val = (c->modulation == QPSK) |
1252 | ? CX24116_PILOT_OFF : CX24116_PILOT_ON; | ||
1253 | retune = 2; | ||
1254 | break; | ||
1252 | case PILOT_OFF: | 1255 | case PILOT_OFF: |
1253 | state->dnxt.pilot_val = CX24116_PILOT_OFF; | 1256 | state->dnxt.pilot_val = CX24116_PILOT_OFF; |
1254 | break; | 1257 | break; |