[FFmpeg-devel] [PATCH] FireWire DV/HDV input device using libiec61883

[Stefano Sabatini]

On date Wednesday 2012-04-25 23:09:45 +0200, Georg Lippitsch encoded:
> From 01857f7b075e62e0ecbcd32bc7bbe8f0b3c4ad50 Mon Sep 17 00:00:00 2001
> From: Georg Lippitsch <georg.lippitsch at gmx.at>
> Date: Wed, 25 Apr 2012 23:08:09 +0200
> Subject: [PATCH 3/3] iec61883 input device documentation
> 
> ---
>  doc/indevs.texi |   41 +++++++++++++++++++++++++++++++++++++++++
>  1 files changed, 41 insertions(+), 0 deletions(-)
> 
> diff --git a/doc/indevs.texi b/doc/indevs.texi
> index e699e11..ddf21c1 100644
> --- a/doc/indevs.texi
> +++ b/doc/indevs.texi
> @@ -179,6 +179,47 @@ ffmpeg -f fbdev -frames:v 1 -r 1 -i /dev/fb0 screenshot.jpeg
>  
>  See also @url{http://linux-fbdev.sourceforge.net/}, and fbset(1).
>  
> + at section iec61883
> +
> +FireWire DV/HDV input device using libiec61883

Missing final dot.

> +
> +The iec61883 capture device supports capturing from a video device
> +connected via IEEE1394 (FireWire), using libiec61883 and the new Linux
> +FireWire stack (juju). This is the default DV/HDV input method in Linux
> +Kernel 2.6.37 and later, since the old FireWire stack was removed.
> +

> +Specify the FireWire port to be used as input file, or use any non-numeric
> +value (like "auto") to choose the first port connected.

> +
> + at subsection Options
> +
> + at table @option
> +
> + at item dvtype
> +Override autodetection of DV/HDV. This should only be used if auto
> +detection does not work, or if usage of a different device type
> +should be prohibted. Treating a DV device as HDV (or vice versa) will

typo: prohibited

> +not work and result in undefined behavior.
> +The values @option{auto}, @option{dv} and @option{hdv} are supported.
> +
> + at item hdvbuffer
> +When capturing HDV (mpeg2), libiec61883 buffers transport stream packets.
> +This sets the number of packets buffered by libiec61883.
> +
> + at end table
> +
> + at subsection Examples
> +Grab and show the input of a FireWire DV/HDV device.
> + at example
> +ffplay -f iec61883 -i auto
> + at end example
> +
> +Grab and record the input of a FireWire DV/HDV device,
> +using a packet buffer of 100000 packets if the source is HDV.
> + at example
> +ffmpeg -f iec61883 -i auto -hdvbuffer 100000 out.mpg
> + at end example

More beautiful:

@subsection Examples

@itemize

@item
Grab and show the input of a FireWire DV/HDV device.
@example
ffplay -f iec61883 -i auto
@end example

@item
...

Also please merge this with the previous patch.

Looks good otherwise.
-- 
FFmpeg = Fierce Freak Monstrous Powered Enlightened Guru