All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@redhat.com>
To: Lars Hanisch <dvb@flensrocker.de>
Cc: Devin Heitmueller <dheitmueller@kernellabs.com>,
	Linux Media Mailing List <linux-media@vger.kernel.org>
Subject: Re: [PATCH 0/5] Fix dvb-core set_delivery_system and port drxk to one frontend
Date: Thu, 05 Jan 2012 22:05:18 -0200	[thread overview]
Message-ID: <4F063ABE.3020806@redhat.com> (raw)
In-Reply-To: <4F05EE24.50209@flensrocker.de>

On 05-01-2012 16:38, Lars Hanisch wrote:
> Hi,
> 
>  First: I'm no driver but an application developer.
> 
> Am 05.01.2012 17:40, schrieb Devin Heitmueller:
>> On Thu, Jan 5, 2012 at 10:37 AM, Mauro Carvalho Chehab
>> <mchehab@redhat.com>  wrote:
>>> With all these series applied, it is now possible to use frontend 0
>>> for all delivery systems. As the current tools don't support changing
>>> the delivery system, the dvb-fe-tool (on my experimental tree[1]) can now
>>> be used to change between them:
>>
>> Hi Mauro,
>>
>> While from a functional standpoint I think this is a good change (and
>> we probably should have done it this way all along), is there not
>> concern that this could be interpreted by regular users as a
>> regression?  Right now they get two frontends, and they can use all
>> their existing tools.  We're moving to a model where if users upgraded
>> their kernel they would now require some new userland tool to do
>> something that the kernel was allowing them to do previously.
>>
>> Sure, it's not "ABI breakage" in the classic sense but the net effect
>> is the same - stuff that used to work stops working and now they need
>> new tools or to recompile their existing tools to include new
>> functionality (which as you mentioned, none of those tools have
>> today).
> 
> Since now there isn't any consistent behaviour of hybrid multifrontend devices.
> Some create multiple frontends but only one demux/dvr (like drx-k), others create 
> all devices for every delivery system (HVR 4000). But they all could only be opened 
> mutually exclusive. In case of vdr (my favourite app) you have to trick with udev,
> symlinks, "remove unwanted frontends" etc. to get the devices in a shape so the 
> application can use it. I don't know how mythtv is handling such devices, but I 
> think there will be something like driver-dependend code in the one or other way.
> 
> The spec isn't really meaningful for hybrid devices. Maybe we should start 
> there and claim something the driver developer can follow.

We had some discussions about that at the KS workshop. All people there
agreed that the better is to use one frontend by physical device.

So, in the end of the day, all drivers should behave like what those DRX-K
patches are doing.

>> Perhaps you would consider some sort of module option that would let
>> users fall back to the old behavior?
> 
>  That would be fine but better would be a module option that will 
>  initialize frontend0 to the "connected" delivery system. In case of DVB-C/-T
>  you don't switch frequently from one to the other. You would need extra hardware 
>  like a splitter which switches inputs if there are e.g. 5V for an active antenna
>  (which means: switch to the dvb-t-input). Is there any DVB-T card which can supply
>  such voltage? And is it controllable via an ioctl (like LNB power supply in DVB-S)?

This is called LNA. A proper LNA support is missing. I think we should 
add a DVBv5 property to control it, on the devices that supports this feature.

The DRX-K chips described at the drxk_hard.c don't support such feature, but
there are other devices that supports it.

Btw, there are some DRX-K devices with two separate demods and two separate
frontends. A driver option won't work on such devices, as one frontend may
be connected to DVB-C, and the other one to DVB-T.

Also, the user may have more than one device of the same type (I have 3 sticks here
with em28xx/drx-k) that could be used simultaneously. Again, a modprobe
parameter won't fit, if the user wants to use some devices for one type, and
the other ones for the second type.

> 
>  Anyway, I think, if there's finally a solution so all drivers behave the same,
>  the tools and applications will handle this new model in the near future.

Yes, that's what we expect ;)

The DTV_ENUM_DELSYS and DTV_DELIVERY_SYSTEM properties are enough to
properly support such devices.
> 
>  Please do something... :-)
> 
> Regards,
> Lars.
> 
>>
>> Devin
>>
> -- 
> 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


  reply	other threads:[~2012-01-06  0:05 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-05 15:37 [PATCH 0/5] Fix dvb-core set_delivery_system and port drxk to one frontend Mauro Carvalho Chehab
2012-01-05 15:37 ` [PATCH 1/5] [media] drxk: remove ops.info.frequency_stepsize from DVB-C Mauro Carvalho Chehab
2012-01-05 15:37 ` [PATCH 2/5] [media] drxk: create only one frontend for both DVB-C and DVB-T Mauro Carvalho Chehab
2012-01-05 15:37 ` [PATCH 3/5] [media] drxk_hard: fix locking issues when changing the delsys Mauro Carvalho Chehab
2012-01-05 15:37 ` [PATCH 4/5] [media] dvb_frontend: regression fix: add a missing inc inside the loop Mauro Carvalho Chehab
2012-01-05 15:37 ` [PATCH 5/5] [media] dvb_frontend: Update ops.info.type earlier Mauro Carvalho Chehab
2012-01-05 18:47   ` [PATCH v2] [media] dvb_frontend: Update the dynamic info->type Mauro Carvalho Chehab
2012-01-05 16:40 ` [PATCH 0/5] Fix dvb-core set_delivery_system and port drxk to one frontend Devin Heitmueller
2012-01-05 18:35   ` Mauro Carvalho Chehab
2012-01-05 18:38   ` Lars Hanisch
2012-01-06  0:05     ` Mauro Carvalho Chehab [this message]
2012-01-06  1:02   ` Oliver Endriss
2012-01-10 21:45 ` Antti Palosaari
2012-01-10 22:22   ` 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=4F063ABE.3020806@redhat.com \
    --to=mchehab@redhat.com \
    --cc=dheitmueller@kernellabs.com \
    --cc=dvb@flensrocker.de \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.