From: Antti Palosaari <crope@iki.fi>
To: Mauro Carvalho Chehab <mchehab@redhat.com>
Cc: Linux Media Mailing List <linux-media@vger.kernel.org>
Subject: Re: [ANNOUNCE] DVBv5 tools version 0.0.1
Date: Sun, 15 Jan 2012 23:47:10 +0200 [thread overview]
Message-ID: <4F13495E.8030106@iki.fi> (raw)
In-Reply-To: <4F13404D.2020001@redhat.com>
On 01/15/2012 11:08 PM, Mauro Carvalho Chehab wrote:
> There was a bug at the error code handling on dvb-fe-tool: basically, if it can't open
> a device, it were using a NULL pointer. It was likely fixed by this commit:
>
> http://git.linuxtv.org/v4l-utils.git/commit/1f669eed5433d17df4d8fb1fa43d2886f99d3991
That bug was fixed as I tested.
But could you tell why dvb-fe-tool --set-delsys=DVBC/ANNEX_A calls
get_frontent() ?
That will cause this kind of calls in demod driver:
init()
get_frontend()
get_frontend()
sleep()
My guess is that it resolves current delivery system. But as demod is
usually sleeping (not tuned) at that phase it does not know frontend
settings asked, like modulation etc. In case of cxd2820r those are
available after set_frontend() call. I think I will add check and return
-EINVAL in that case.
regards
Antti
--
http://palosaari.fi/
next prev parent reply other threads:[~2012-01-15 21:47 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-01-07 12:19 [ANNOUNCE] DVBv5 tools version 0.0.1 Mauro Carvalho Chehab
2012-01-07 17:29 ` Honza Petrouš
2012-01-07 20:05 ` Mauro Carvalho Chehab
2012-01-08 1:52 ` Mauro Carvalho Chehab
2012-01-08 12:57 ` dvbv5-tools update - was: " Mauro Carvalho Chehab
2012-01-10 21:36 ` Antti Palosaari
2012-01-10 22:00 ` Mauro Carvalho Chehab
2012-01-15 18:37 ` Antti Palosaari
2012-01-15 20:03 ` Antti Palosaari
2012-01-15 21:08 ` Mauro Carvalho Chehab
2012-01-15 21:14 ` Antti Palosaari
2012-01-15 21:47 ` Antti Palosaari [this message]
2012-01-16 0:16 ` Mauro Carvalho Chehab
2012-01-16 2:57 ` Mauro Carvalho Chehab
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=4F13495E.8030106@iki.fi \
--to=crope@iki.fi \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@redhat.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;
as well as URLs for NNTP newsgroup(s).