public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Oliver Endriss <o.endriss@gmx.de>
To: linux-dvb@linuxtv.org
Cc: Andrea <mariofutire@googlemail.com>
Subject: Re: [linux-dvb] [PATCH] Synchronize dvb-apps with v4l-dvb
Date: Sat, 3 May 2008 14:35:59 +0200	[thread overview]
Message-ID: <200805031435.59693@orion.escape-edv.de> (raw)
In-Reply-To: <481A3D61.7040508@googlemail.com>

Andrea wrote:
> Hi,
> 
> This patch (posted for the 3rd times) keeps dvb-apps in line with recent (and not so recent) changes 
> in v4l-dvb:
> 
> 1) for 2 years it has not been possible to open the dvr more than once for read only.
> After this change http://linuxtv.org/hg/v4l-dvb/rev/64edfcc88eeb the dvr can only be opened once.
> The patch fixes a comment to the function dvbdemux_open_dvr
> 
> 2) this changeset http://linuxtv.org/hg/v4l-dvb/rev/65699e8bc6f7 added the option 
> DMX_OUT_TSDEMUX_TAP to send the TS to the demux.
> The patch enhances dvbdemux.h to use the DMX_OUT_TSDEMUX_TAP
> 
> 3) a recent changest http://linuxtv.org/hg/v4l-dvb/rev/8389fb4e774c implemented DMX_SET_BUFFER for 
> the dvr.
> tzap used to set the dvr buffer size to 1MB (this ioctl call used to be ignored) while the default 
> size is actually double (~2MB). I think the aim of the code was to make the buffer bigger and not to 
> shrink it. That buffer in my opinion should stay as it is.

Committed to dvb-apps. Thanks.

Btw, I also committed this patch to make dvbdemux.c compile against
current kernel headers:

+++ b/lib/libdvbapi/dvbdemux.c	Sat May 03 14:26:59 2008 +0200
@@ -128,9 +128,11 @@ int dvbdemux_set_pes_filter(int fd, int 
 		filter.output = DMX_OUT_TS_TAP;
 		break;
 
+#ifdef DMX_OUT_TSDEMUX_TAP
 	case DVBDEMUX_OUTPUT_TS_DEMUX:
 		filter.output = DMX_OUT_TSDEMUX_TAP;
 		break;
+#endif
 
 	default:
 		return -EINVAL;
@@ -205,9 +207,11 @@ int dvbdemux_set_pid_filter(int fd, int 
 		filter.output = DMX_OUT_TS_TAP;
 		break;
 
+#ifdef DMX_OUT_TSDEMUX_TAP
 	case DVBDEMUX_OUTPUT_TS_DEMUX:
 		filter.output = DMX_OUT_TSDEMUX_TAP;
 		break;
+#endif
 
 	default:
 		return -EINVAL;

> I've posted this patch 3 times already with 0 (i.e. ZERO) replies.
> Is there anybody taking care of dvb-apps?

Sorry, dvb-apps is more or less unmaintained.

CU
Oliver

-- 
----------------------------------------------------------------
VDR Remote Plugin 0.4.0: http://www.escape-edv.de/endriss/vdr/
----------------------------------------------------------------

_______________________________________________
linux-dvb mailing list
linux-dvb@linuxtv.org
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb

  reply	other threads:[~2008-05-03 12:36 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-05-01 22:00 [linux-dvb] [PATCH] Synchronize dvb-apps with v4l-dvb Andrea
2008-05-03 12:35 ` Oliver Endriss [this message]
2008-05-03 20:46   ` Andrea

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=200805031435.59693@orion.escape-edv.de \
    --to=o.endriss@gmx.de \
    --cc=linux-dvb@linuxtv.org \
    --cc=mariofutire@googlemail.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