linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Gianluca Gennari <gennarone@gmail.com>
To: Jim Darby <uberscubajim@gmail.com>
Cc: Mauro Carvalho Chehab <mchehab@redhat.com>, linux-media@vger.kernel.org
Subject: Re: Possible regression in 3.2 kernel with PCTV Nanostick T2 (em28xx, cxd2820r and tda18271)
Date: Fri, 13 Jan 2012 15:24:47 +0100	[thread overview]
Message-ID: <4F103EAF.1010302@gmail.com> (raw)
In-Reply-To: <4F102D0F.3010408@gmail.com>

Il 13/01/2012 14:09, Jim Darby ha scritto:
> On 13/01/12 11:21, Mauro Carvalho Chehab wrote:
> 
>> Hmm... this patch shouldn't be causing troubles for an application that
>> only uses DVBv3 call. Is Kaffeine filling the DTV_DELIVERY_SYSTEM with
>> SYS_UNDEFINED (0)?
> 
> I think this is perhaps where (some of) our problems are starting. I
> just looked at the kaffeine source code and, as far as I can make out,
> DTV_DELIVERY_SYSTEM is filled out by performing a FE_SET_PROPERTY ioctl
> with a key/value pair of something like DTV_DELIVERY_SYSTEM/SYS_DVBS2
> (amongst other parameters).
> 
> The issue here is that kaffeine *only* performs any sort of
> FE_SET_PROPERTY ioctl for DVB-S2. It certainly doesn't for any form of
> DVB-T (2 or original).
> 
> It would therefore appear that kaffeine is committing a sin of omission
> in not setting the front-end properties and hence we have this problem.

Hi Jim,
that's because Kaffeine is using the new DVBv5 API to interface with
DVB-S2 hardware (as this is the only API supported), while it is using
the old DVBv3 API to interface to DVB-T/C hardware. It's not a real
problem, as in dvb-core there is some emulation logic that takes care of
supporting DVBv3 applications.

> Mauro, if you can confirm that this is the case and that with the latest
> linux-media drivers performing the FE_SET_PROPERTY ioctl is mandatory
> then I can work with the kaffeine developers and get this fixed.
> 
> For reference, the existing kaffeine works with the stock 3.2.0 kernel.
> It's just the linux-media from linux-tv.org that breaks it.

Mauro already fixed the bug I reported. But if you can work with the
Kaffeine developers to adopt DVBv5 API also for DVB-T/C tuners, it would
be a nice contribution.

Best regards,
Gianluca


  reply	other threads:[~2012-01-13 14:24 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-10 13:28 Possible regression in 3.2 kernel with PCTV Nanostick T2 (em28xx, cxd2820r and tda18271) Jim Darby
2012-01-10 13:54 ` Steven Toth
2012-01-10 23:22   ` Jim Darby
2012-01-11  0:01     ` Steven Toth
2012-01-11  0:34       ` Andy Walls
2012-01-11  1:05 ` Antti Palosaari
2012-01-11 11:30   ` Jim Darby
2012-01-11 19:19   ` Jim Darby
2012-01-12 16:22     ` Gianluca Gennari
2012-01-12 16:35       ` Jim Darby
2012-01-12 17:13         ` Simon Jones
2012-01-12 17:34         ` Gianluca Gennari
2012-01-13 11:21       ` Mauro Carvalho Chehab
2012-01-13 11:45         ` Gianluca Gennari
2012-01-13 13:50           ` [PATCH] [media] dvb-core: preserve the delivery system at cache clear Mauro Carvalho Chehab
2012-01-13 16:04             ` Gianluca Gennari
2012-01-14  0:00             ` Jim Darby
2012-01-14 14:51               ` Mauro Carvalho Chehab
2012-01-13 13:09         ` Possible regression in 3.2 kernel with PCTV Nanostick T2 (em28xx, cxd2820r and tda18271) Jim Darby
2012-01-13 14:24           ` Gianluca Gennari [this message]
2012-01-12 14:29 ` Simon Jones

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=4F103EAF.1010302@gmail.com \
    --to=gennarone@gmail.com \
    --cc=linux-media@vger.kernel.org \
    --cc=mchehab@redhat.com \
    --cc=uberscubajim@gmail.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).