From: Steve Kerrison <steve@stevekerrison.com>
To: "Bjørn Mork" <bjorn@mork.no>
Cc: linux-media@vger.kernel.org
Subject: Re: [GIT PULL FOR 2.6.40] PCTV nanoStick T2 290e (Sony CXD2820R DVB-T/T2/C)
Date: Fri, 03 Jun 2011 14:33:27 +0100 [thread overview]
Message-ID: <1307108007.2341.2.camel@ares> (raw)
In-Reply-To: <8762onxcuc.fsf@nemi.mork.no>
Here in the UK the mux strengths and parameters are all over the place.
I have had situations in the past (with dib0700 I think) where I've had
to twiddle the force_lna setting one way or the other depending on how
they've decided to reconfigure my local transmitter.
So I'd be cautious about flat-out enabling it the whole time.
--
Steve Kerrison MEng Hons.
http://www.stevekerrison.com/
On Fri, 2011-06-03 at 15:29 +0200, Bjørn Mork wrote:
> Antti Palosaari <crope@iki.fi> writes:
> > On 06/01/2011 04:27 PM, Mauro Carvalho Chehab wrote:
> >> Em 25-05-2011 17:42, Antti Palosaari escreveu:
> >>> Antti Palosaari (7):
> >>> em28xx-dvb: add module param "options" and use it for LNA
> >>
> >> That patch is ugly, for several reasons:
> >>
> >> 1) we don't want a generic "options" parameter, whose meaning changes from
> >> device to devices;
> >
> > I agree it is not proper solution, but in my mind it is better to
> > offer some solution than no solution at all.
> >
> >> 2) what happens if someone has two em28xx devices plugged?
> >
> > It depends depends devices, currently only nanoStick T2 only looks
> > that param, other just ignore. If there is two nanoStics then both
> > have same LNA settings.
> >
> > That's just like same behaviour as for example remote controller
> > polling. Or for example DiBcom driver LNA, since it does have similar
> > module param already. Will you you commit it if I rename it similarly
> > as DiBcom?
> >
> >> 3) the better would be to detect if LNA is needed, or to add a DVBS2API
> >> call to enable/disable LNA.
> >
> > True, but it needs some research. There is many hardware which gets
> > signal input from demod or tuner and makes some fine tune according to
> > that. We need to define some new callbacks for demod and tuner in
> > order to do this kind of actions.
> > Or just add new LNA param to API use it manually.
>
>
> Or option
> 4) just enable the LNA unconditionally.
>
> I did some testing in my environment, and I was unable to tune anything
> on either DVB-T or DVB-C without the LNA enabled. I'm of course aware
> that this depends on your signal, but have you actually seen a real life
> signal where tuning fails with the LNA enabled and works without it?
>
> I do believe that my DVB-C signal at least is pretty strong.
>
>
>
> Bjørn
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2011-06-03 13:33 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-05-25 20:42 [GIT PULL FOR 2.6.40] PCTV nanoStick T2 290e (Sony CXD2820R DVB-T/T2/C) Antti Palosaari
2011-06-01 13:27 ` Mauro Carvalho Chehab
2011-06-03 12:34 ` Antti Palosaari
2011-06-03 13:29 ` Bjørn Mork
2011-06-03 13:33 ` Steve Kerrison [this message]
2011-06-03 14:04 ` Mauro Carvalho Chehab
-- strict thread matches above, loose matches on Subject: below --
2011-05-04 20:54 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=1307108007.2341.2.camel@ares \
--to=steve@stevekerrison.com \
--cc=bjorn@mork.no \
--cc=linux-media@vger.kernel.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