From: Antti Palosaari <crope@iki.fi>
To: "Miroslav Slugeň" <thunder.mmm@gmail.com>
Cc: linux-media@vger.kernel.org
Subject: Re: Add tuner_type to zl10353 config and use it for reporting signal directly from tuner.
Date: Thu, 22 Dec 2011 18:58:14 +0200 [thread overview]
Message-ID: <4EF361A6.7080305@iki.fi> (raw)
In-Reply-To: <CAEN_-SAuS1UTfLcJUpVP-WYeLVVj4-ycF0NyaEi=iQ0AnVbZEQ@mail.gmail.com>
On 12/21/2011 11:07 PM, Miroslav Slugeň wrote:
> XC4000 based cards are not using AGC control in normal way, so it is
> not possible to get signal level from AGC registres of zl10353
> demodulator, instead of this i send previous patch to implement signal
> level directly in xc4000 tuner and now sending patch for zl10353 to
> implement this future for digital mode. Signal reporting is very
> accurate and was well tested on 3 different Leadtek XC4000 cards.
I don't like that patch at all. My opinion is that you should put hacks
like to the interface driver. Override demod .read_signal_strength()
callback and route it to the tuner callback. No any changes for the
demod driver should be done.
Estimation of the signal strength is a little bit hard when looking
demod point of view. Demod gets IF as input signal and thus have mainly
idea of IF AGC values. Estimating RF strength is thus very inaccurate
from the IF AGC gain. And those IF AGC values are tuner/demod
combination dependent too. Sometimes there is also RF AGC available for
the demod. With both IF and RF AGC you could estimate more better - but
still very inaccurate.
regards
Antti
--
http://palosaari.fi/
next prev parent reply other threads:[~2011-12-22 16:58 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-12-21 21:07 Add tuner_type to zl10353 config and use it for reporting signal directly from tuner Miroslav Slugeň
2011-12-21 21:29 ` Devin Heitmueller
2011-12-21 23:50 ` Miroslav Slugeň
2011-12-21 23:55 ` Miroslav Slugeň
2011-12-22 16:58 ` Antti Palosaari [this message]
2011-12-22 23:45 ` Miroslav Slugeň
2011-12-23 8:14 ` Antti Palosaari
2011-12-23 8:19 ` Antti Palosaari
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=4EF361A6.7080305@iki.fi \
--to=crope@iki.fi \
--cc=linux-media@vger.kernel.org \
--cc=thunder.mmm@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox