public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Andreas Oberritter <obi@linuxtv.org>
To: Hamad Kadmany <hkadmany@codeaurora.org>
Cc: linux-media@vger.kernel.org
Subject: Re: Support for multiple section feeds with same PIDs
Date: Thu, 01 Dec 2011 17:30:56 +0100	[thread overview]
Message-ID: <4ED7ABC0.4010003@linuxtv.org> (raw)
In-Reply-To: <000501ccb041$f3f08210$dbd18630$@org>

On 01.12.2011 16:57, Hamad Kadmany wrote:
> Hello Andreas
>  
>> On 01.12.2011 16:30, Andreas Oberritter wrote:
> 
>> Yes. Feel free to enhance the demux API to your needs in order to fully
>> support the features of your hardware.
> 
> I have another question in that regard: Actually, multiple filter with same
> PID is assumed to be possible in case we have one filter for TS packets (for
> DVR device) and another for video PES (for playback). So it seems there's
> such assumption in this regard but not for sections. Is my understanding
> correct?

The in-kernel software demux ("dvb_demux") receives TS packets and
strips off headers as requested in order to convert TS to PES or
sections. If you're implementing a driver, you can choose to either
extend the software demux by overwriting its
{allocate,release}_{ts,section}_feed function pointers or you can
replace the software demux completely, using your own functions.

That said, the software demux supports multiple filters of any type
simultaneously on the same PID, so whether it will work for you will
eventually depend on your implementation.

It's definitely easier to extend the sofware demux, but replacing it may
possibly allow you use your hardware more efficiently.

I think the av7110 driver also implements section filters, as a further
example.

Regards,
Andreas

  reply	other threads:[~2011-12-01 16:31 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-29  8:05 Support for multiple section feeds with same PIDs Hamad Kadmany
2011-12-01  6:23 ` Hamad Kadmany
2011-12-01 11:48   ` Mauro Carvalho Chehab
2011-12-01 13:36 ` Andreas Oberritter
2011-12-01 13:55   ` Hamad Kadmany
2011-12-01 13:57     ` Andreas Oberritter
2011-12-01 14:22       ` Hamad Kadmany
2011-12-01 14:30         ` Andreas Oberritter
2011-12-01 15:57           ` Hamad Kadmany
2011-12-01 16:30             ` Andreas Oberritter [this message]
2011-12-07 11:30 ` [dvb] Problem registering demux0 device Hamad Kadmany
2011-12-07 11:49   ` Mauro Carvalho Chehab
2011-12-07 13:27     ` Hamad Kadmany

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=4ED7ABC0.4010003@linuxtv.org \
    --to=obi@linuxtv.org \
    --cc=hkadmany@codeaurora.org \
    --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