public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: e9hack <e9hack@googlemail.com>
To: linux-dvb@linuxtv.org
Subject: Re: [linux-dvb] [PATCH]Fix a bug in scan, which outputs the wrong frequency if the current tuned transponder is scanned only
Date: Tue, 02 Dec 2008 20:22:41 +0100	[thread overview]
Message-ID: <49358B01.2010701@googlemail.com> (raw)
In-Reply-To: <19a3b7a80812020834t265f2cc0vcf485b05b23b6724@mail.gmail.com>

Christoph Pfister schrieb:
> 2008/11/17 e9hack <e9hack@googlemail.com>:
>> Hi,
>>
>> if the current tuned transponder is scanned only and the output needs the frequency of the
>> transponder, it is used the last frequency, which is found during the NIT scanning. This
>> is wrong. The attached patch will fix this problem.
> 
> Any opinion about this patch? It seems ok from a quick look, so I'll
> apply it soon if nobody objects.

After some more investigation, my patch has some problems:

1)Usually the patch is only necessary for transponders, which transmits NIT data of other
transponders (DVB-C only?).

2)It will not work with DVB-S. On DVB-S the returned frequency from frontend is the
LNB-IF. It must be convert to the transponder frequency depend on the LNB configuration.
Scan can't read the LNB configuration from frontend. It must trust the given parameters.

3)On DVB-S, the reported frequency from frontend contains the offset from zigzag scan. If
the offset is greater than 2kHz, scan can't find the correct frequency within the NIT data.

3)The reported frequency from many frontends contains a measured offset. If the offset is
  greater than 2kHz, see above.

Depend on this 4 points, scan should not report any transponder infos, if the current
transponder is scanned only (scan does not tune itself) and if the output format must
contain the frequency of the transponder.

-Hartmut

_______________________________________________
linux-dvb mailing list
linux-dvb@linuxtv.org
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb

      parent reply	other threads:[~2008-12-02 19:22 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-11-17 12:51 [linux-dvb] [PATCH]Fix a bug in scan, which outputs the wrong frequency if the current tuned transponder is scanned only e9hack
2008-12-02 16:34 ` Christoph Pfister
2008-12-02 16:49   ` Alex Betis
2008-12-02 19:43     ` e9hack
2008-12-02 20:05       ` Alex Betis
2008-12-02 22:07         ` e9hack
2008-12-03  7:23           ` Alex Betis
2008-12-03 17:13             ` e9hack
2008-12-03 19:04               ` e9hack
2008-12-03 21:28                 ` Alex Betis
2008-12-08 20:21                   ` e9hack
2008-12-08 20:36                     ` Alex Betis
2008-12-08 21:00                       ` e9hack
2008-12-02 19:22   ` e9hack [this message]

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=49358B01.2010701@googlemail.com \
    --to=e9hack@googlemail.com \
    --cc=linux-dvb@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