public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@redhat.com>
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 09:48:28 -0200	[thread overview]
Message-ID: <4ED7698C.4070602@redhat.com> (raw)
In-Reply-To: <000001ccaff1$cb1cc060$61564120$@org>

On 01-12-2011 04:23, Hamad Kadmany wrote:
> Hi,
>
> Sorry to repeat the question, anyone has an idea on this? I appreciate your
> feedback.
>
> Thank you
> Hamad
>
> -----Original Message-----
> From: linux-media-owner@vger.kernel.org
> [mailto:linux-media-owner@vger.kernel.org] On Behalf Of Hamad Kadmany
> Sent: Tuesday, November 29, 2011 10:05 AM
> To: linux-media@vger.kernel.org
> Subject: Support for multiple section feeds with same PIDs
>
> Hello
>
> Question on the current behavior of dvb_dmxdev_filter_start (dmxdev.c)
>
> In case of DMXDEV_TYPE_SEC, the code restricts of having multiple sections
> feeds allocated (allocate_section_feed) with same PID. From my experience,
> applications might request allocating several section feeds using same PID
> but with different filters (for example, in DVB standard, SDT and BAT tables
> have same PID).
>
> The current implementation only supports of having multiple filters on the
> same section feed.
>
> Any special reason why it was implemented this way?

Eventually, you might be able to find some reason by digging into the patches
that introduced such feature. If this went before 2.6.12-rc2, you'll need to
take a look at the Linux history git trees or at the mercurial tree.

In any case, if you need to extend it to support multiple sections, just
propose a patch extending it, yet maintaining backward compatibility with the
existing behavior.

Regards,
Mauro.

>
> Thank you
> Hamad
>
> --
> 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
>
> --
> 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:[~2011-12-01 11:48 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 [this message]
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
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=4ED7698C.4070602@redhat.com \
    --to=mchehab@redhat.com \
    --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