[MPlayer-dev-eng] [PATCH]Support PCM in Bluray streams

[Reimar Döffinger]

On Thu, Aug 27, 2009 at 09:59:21PM +0200, Carl Eugen Hoyos wrote:
> On Thursday 27 August 2009 09:57:08 Nico Sabbi wrote:
> > > >+audiocodec lpcm
> > > >+  info "Bluray LPCM"
> > > >+  status working
> > > >+  fourcc HDMV
> > > >+  driver ffmpeg
> > > >+  dll pcm_bluray
> 
> [...]
> 
> > not  to me: HDMV is a descriptor present both in the program and in
> > some or all the streams (audio, video, subs). Who is the genius that
> > used it as a fourcc?
> 
> What do you think about attached patch?
> 
> Carl Eugen

> Index: libmpdemux/mp_taglists.c
> ===================================================================
> --- libmpdemux/mp_taglists.c	(revision 29554)
> +++ libmpdemux/mp_taglists.c	(working copy)
> @@ -63,6 +63,7 @@
>      { CODEC_ID_PCM_S24LE,         1},
>      { CODEC_ID_PCM_S32BE,         MKTAG('i', 'n', '3', '2')},
>      { CODEC_ID_PCM_S32LE,         1},
> +    { CODEC_ID_PCM_BLURAY,        MKTAG('L', 'P', 'C', 'M')},
>      { 0, 0 },
>  };
>  
> Index: etc/codecs.conf
> ===================================================================
> --- etc/codecs.conf	(revision 29554)
> +++ etc/codecs.conf	(working copy)
> @@ -3786,6 +3797,13 @@
>    format 0x10001
>    driver dvdpcm
>  
> +audiocodec lpcm
> +  info "Bluray LPCM"
> +  status working
> +  fourcc LPCM
> +  driver ffmpeg
> +  dll pcm_bluray

We already have DVD LPCM, maybe BRPC or BPCM is a better 4cc.