diff mbox series

[FFmpeg-devel] avformat/mkv: add mkv tags for AVS2 and AVS3 codecs.

Message ID BYAPR03MB4536C048FBD9A10A29C617A5B8269@BYAPR03MB4536.namprd03.prod.outlook.com
State New
Headers show
Series [FFmpeg-devel] avformat/mkv: add mkv tags for AVS2 and AVS3 codecs. | expand

Checks

Context Check Description
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished
andriy/make_ppc success Make finished
andriy/make_fate_ppc success Make fate finished
andriy/make_aarch64_jetson success Make finished
andriy/make_fate_aarch64_jetson success Make fate finished
andriy/make_armv7_RPi4 success Make finished
andriy/make_fate_armv7_RPi4 success Make fate finished

Commit Message

Ze Yuan Feb. 1, 2022, 4:35 a.m. UTC
From babcceafbd30eff677b2366a0c470d31c503bed1 Mon Sep 17 00:00:00 2001
From: TianBo Zheng <naturalwalker@hotmail.com>
Date: Mon, 10 Jan 2022 11:18:56 +0000
Subject: [PATCH] avformat/mkv: add mkv tags for AVS2 and AVS3 codecs.

MKV codec mappings:
  V_AVS2 and V_AVS3 (https://github.com/ietf-wg-cellar/matroska-specification/blob/master/codec_specs.md)

Encoding tool:
  Ffmpeg with AVS2/AVS3 enabled: https://github.com/xatabhk/FFmpeg-avs2-avs3/releases
  Command line:

             ffmpeg -i xxxx.mp4 -vcodec avs2 -acodec copy xxxx_avs2.mkv
             ffmpeg -i xxxx.mp4 -vcodec avs2 -speed_level 4 -acodec copy xxxx_avs2.mkv`

Players:
  (1) Ffmpeg with avs2/avs3 enabled: (https://github.com/xatabhk/FFmpeg-avs2-avs3/releases):
    Command line:

        ffplay xxxx_avs2.mkv
    ffplay xxxx_avs3.mkv
  (2) VLC 3.0.x with AVS2/AVS3 enabled: https://github.com/xatabhk/vlc-3.0-avs2-avs3/releases
  (3) Mpc-hc 1.9.x with AVS2/AVS3 enabled: https://gitee.com/zhengtianbo/cavs-avs2-avs3_decoder_added_to_mpc_hc/releases

AVS2/AVS3 MKV samples:
  https://github.com/xatabhk/avs2-avs3-video-samples
Signed-off-by: TianBo Zheng naturalwalker@hotmail.com
---
libavformat/matroska.c | 2 ++
1 file changed, 2 insertions(+)

--
2.19.1

Comments

Andreas Rheinhardt Feb. 1, 2022, 4:43 a.m. UTC | #1
Ze Yuan:
> From babcceafbd30eff677b2366a0c470d31c503bed1 Mon Sep 17 00:00:00 2001
> From: TianBo Zheng <naturalwalker@hotmail.com>
> Date: Mon, 10 Jan 2022 11:18:56 +0000
> Subject: [PATCH] avformat/mkv: add mkv tags for AVS2 and AVS3 codecs.
> 
> MKV codec mappings:
>   V_AVS2 and V_AVS3 (https://github.com/ietf-wg-cellar/matroska-specification/blob/master/codec_specs.md)
> 
> Encoding tool:
>   Ffmpeg with AVS2/AVS3 enabled: https://github.com/xatabhk/FFmpeg-avs2-avs3/releases
>   Command line:
> 
>              ffmpeg -i xxxx.mp4 -vcodec avs2 -acodec copy xxxx_avs2.mkv
>              ffmpeg -i xxxx.mp4 -vcodec avs2 -speed_level 4 -acodec copy xxxx_avs2.mkv`
> 
> Players:
>   (1) Ffmpeg with avs2/avs3 enabled: (https://github.com/xatabhk/FFmpeg-avs2-avs3/releases):
>     Command line:
> 
>         ffplay xxxx_avs2.mkv
>     ffplay xxxx_avs3.mkv
>   (2) VLC 3.0.x with AVS2/AVS3 enabled: https://github.com/xatabhk/vlc-3.0-avs2-avs3/releases
>   (3) Mpc-hc 1.9.x with AVS2/AVS3 enabled: https://gitee.com/zhengtianbo/cavs-avs2-avs3_decoder_added_to_mpc_hc/releases
> 
> AVS2/AVS3 MKV samples:
>   https://github.com/xatabhk/avs2-avs3-video-samples
> Signed-off-by: TianBo Zheng naturalwalker@hotmail.com
> ---
> libavformat/matroska.c | 2 ++
> 1 file changed, 2 insertions(+)
> 
> diff --git a/libavformat/matroska.c b/libavformat/matroska.c
> index 7c56aba403..90d94b65bf 100644
> --- a/libavformat/matroska.c
> +++ b/libavformat/matroska.c
> @@ -78,6 +78,8 @@ const CodecTags ff_mkv_codec_tags[]={
>      {"S_HDMV/TEXTST"    , AV_CODEC_ID_HDMV_TEXT_SUBTITLE},
> 
>      {"V_AV1"            , AV_CODEC_ID_AV1},
> +    {"V_AVS2"           , AV_CODEC_ID_AVS2},
> +    {"V_AVS3"           , AV_CODEC_ID_AVS3},
>      {"V_DIRAC"          , AV_CODEC_ID_DIRAC},
>      {"V_FFV1"           , AV_CODEC_ID_FFV1},
>      {"V_MJPEG"          , AV_CODEC_ID_MJPEG},
> --
> 2.19.1

Sorry for having ignored this for so long.
I actually wanted to merge your AVS2 patch long ago (see
https://github.com/mkver/FFmpeg/commit/18c52e7493b1c1f11de059fbcbf9d17635e5945d),
but I remember that your mails were weird and I managed to recreate a
Chinese name out of them, but I was unsure if this was the correct one
(namely 郑天波). So I didn't apply it. If it is correct, then please
tell me if you prefer this name or TianBo Zheng.

- Andreas
diff mbox series

Patch

diff --git a/libavformat/matroska.c b/libavformat/matroska.c
index 7c56aba403..90d94b65bf 100644
--- a/libavformat/matroska.c
+++ b/libavformat/matroska.c
@@ -78,6 +78,8 @@  const CodecTags ff_mkv_codec_tags[]={
     {"S_HDMV/TEXTST"    , AV_CODEC_ID_HDMV_TEXT_SUBTITLE},

     {"V_AV1"            , AV_CODEC_ID_AV1},
+    {"V_AVS2"           , AV_CODEC_ID_AVS2},
+    {"V_AVS3"           , AV_CODEC_ID_AVS3},
     {"V_DIRAC"          , AV_CODEC_ID_DIRAC},
     {"V_FFV1"           , AV_CODEC_ID_FFV1},
     {"V_MJPEG"          , AV_CODEC_ID_MJPEG},