From: Steven Toth <stoth@linuxtv.org>
To: Andreas Oberritter <obi@linuxtv.org>
Cc: Linux-dvb <linux-dvb@linuxtv.org>
Subject: Re: [linux-dvb] Revisiting the SNR/Strength issue
Date: Fri, 17 Oct 2008 09:31:15 -0400 [thread overview]
Message-ID: <48F893A3.4060607@linuxtv.org> (raw)
In-Reply-To: <48F86120.2020203@linuxtv.org>
Andreas Oberritter wrote:
> Steven Toth wrote:
>> Devin Heitmueller wrote:
>>> Certainly I'm in favor of expressing that there is a preferred unit
>>> that new frontends should use (whether that be ESNO or db), but the
>>> solution I'm suggesting would allow the field to become useful *now*.
>>> This would hold us over until all the other frontends are converted to
>>> db (which I have doubts will ever actually happen).
>> I'm not in favour of this.
>>
>> I'd rather see a single unit of measure agreed up, and each respective
>> maintainer go back and perform the necessary code changes. I'm speaking
>> as a developer of eight (?) different demod drivers in the kernel.
>> That's no small task, but I'd happily conform if I could.
>>
>> Lastly, for the sake of this discussion, assuming that db is agreed
>> upon, if the driver cannot successfully delivery SNR in terms of db then
>> the bogus function returning junk should be removed.
>>
>> Those two changes alone would be a better long term approach, I think.
>
> How about adding a new command instead (and a similar one for S2API)?
>
> /* Read SNR in units of dB/100 */
> #define FE_READ_SNR_DB _IOR('o', 74, __u16)
>
> Then it's no problem to slowly migrate the drivers to this interface. The
> old interface can still stay for some time without changes. Applications
> can try this ioctl, and if it returns an error, then it is not implemented
> for the used device.
Devin has offered to review the demods and snr code, to see the differences.
BTW, I like a couple of the ideas mentioned so far.
Many of the recent Linux demods drivers were written from datasheets, so
we have access to real credible numbers. I suspect I can also push NXP
for datasheets on older parts if the maintainers of other demods are
willing to go the extra mile and add add the code. Other vendors, maybe
not - let's see.
You can't really judge good / better / best or db if you don't have
credible esno / db registers, that's should be the first step - to
understand how many demods have issues and how many are fixable.
The user-facing API is only any good after we know the demods are
standardized. I tend to think we can get > 80% of the demods reporting
db or esno, which in turn can easily be abstracted via dvb-core into
good / better / best or a more appropriate user view.
I don't agree to blindly massaging the demod values and trying to add a
fake user facing API is a real solution.
I do like that people are talking again, and I will certainly be willing
to help in fixing demods.
- Steve
_______________________________________________
linux-dvb mailing list
linux-dvb@linuxtv.org
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb
next prev parent reply other threads:[~2008-10-17 13:31 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-10-15 14:24 [linux-dvb] Revisiting the SNR/Strength issue Devin Heitmueller
2008-10-15 14:30 ` Steven Toth
2008-10-15 14:40 ` Devin Heitmueller
2008-10-15 18:18 ` Steven Toth
2008-10-16 14:28 ` Devin Heitmueller
2008-10-17 9:40 ` Christophe Thommeret
2008-10-17 9:55 ` Andreas Oberritter
2008-10-17 13:31 ` Steven Toth [this message]
2008-10-17 14:12 ` Andreas Oberritter
2008-10-17 18:23 ` Steven Toth
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=48F893A3.4060607@linuxtv.org \
--to=stoth@linuxtv.org \
--cc=linux-dvb@linuxtv.org \
--cc=obi@linuxtv.org \
/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