From: Gianluca Gennari <gennarone@gmail.com>
To: linux-media@vger.kernel.org, mchehab@redhat.com
Cc: crope@iki.fi, Gianluca Gennari <gennarone@gmail.com>
Subject: [PATCH 0/3] cxd2820r: tweak search algorithm, enable LNA in DVB-T mode
Date: Thu, 15 Mar 2012 18:33:46 +0100 [thread overview]
Message-ID: <1331832829-4580-1-git-send-email-gennarone@gmail.com> (raw)
The PCTV 290e had several issues on my mipsel-based STB (powered by a
Broadcom 7405 SoC), running a Linux 3.1 kernel and the Enigma2 OS.
The most annoying one was that the 290e was able to tune the lone DVB-T2
frequency existing in my area, but was not able to tune any DVB-T channel.
Following a suggestion of the original author of the driver, I tried to
tweak the wait time in the lock loop. In fact, increasing the wait time
from 50 to 200ms in the tuning loop was enough to get the lock on most
channels.
But channel change was quite slow and sometimes, doing an automatic scan,
some frequency was not locked.
So instead of playing with the timings I changed the behavior of the
search algorithm as explained in the patch 1, with very good results.
With this modification, the automatic scan is 100% reliable and zapping
is quite fast (on the STB). There is no noticeable difference when using
Kaffeine on the PC.
But there was a further issue: a few weak channels were affected by high
BER and badly corrupted pictures. The same channels were working fine on
an Avermedia A867 stick (as well as other sticks).
The driver has an option to enable a "Low Noise Amplifier" (LNA) before the
demodulator. Enabling it, the reception of weak channels improved a lot,
as reported in the description of patch 2.
Finally, patch 3 is a trivial clean-up.
Best regards,
Gianluca Gennari
Gianluca Gennari (3):
cxd2820r: tweak search algorithm behavior
em28xx-dvb: enable LNA for cxd2820r in DVB-T mode
cxd2820r: delete unused function cxd2820r_init_t2
drivers/media/dvb/frontends/cxd2820r_core.c | 4 ++--
drivers/media/dvb/frontends/cxd2820r_priv.h | 2 --
drivers/media/video/em28xx/em28xx-dvb.c | 3 ++-
3 files changed, 4 insertions(+), 5 deletions(-)
--
1.7.5.4
next reply other threads:[~2012-03-15 17:34 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-03-15 17:33 Gianluca Gennari [this message]
2012-03-15 17:33 ` [PATCH 1/3] cxd2820r: tweak search algorithm behavior Gianluca Gennari
2012-03-23 12:49 ` Antti Palosaari
2012-03-15 17:33 ` [PATCH 2/3] em28xx-dvb: enable LNA for cxd2820r in DVB-T mode Gianluca Gennari
2012-03-23 12:55 ` Antti Palosaari
[not found] ` <CACOeW9MNoRZOs5yruTSEqcj_576ih6cnpW-j0HzKCs0Qyy=P4w@mail.gmail.com>
2012-03-23 14:45 ` Mauro Carvalho Chehab
2012-03-15 17:33 ` [PATCH 3/3] cxd2820r: delete unused function cxd2820r_init_t2 Gianluca Gennari
2012-03-23 12:50 ` Antti Palosaari
2012-03-15 17:41 ` [PATCH 0/3] cxd2820r: tweak search algorithm, enable LNA in DVB-T mode Mauro Carvalho Chehab
2012-03-15 17:49 ` Antti Palosaari
2012-03-15 18:06 ` Mauro Carvalho Chehab
2012-03-15 18:07 ` Gianluca Gennari
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1331832829-4580-1-git-send-email-gennarone@gmail.com \
--to=gennarone@gmail.com \
--cc=crope@iki.fi \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.