public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
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

  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