[FFmpeg-devel] [PATCH] avcodec: Pass the HDR10+ metadata to the packet side data in VP9 encoder

[James Zern]

On Tue, Jun 22, 2021 at 1:46 PM James Zern <jzern at google.com> wrote:
>
> On Thu, Jun 17, 2021 at 10:21 PM Mohammad Izadi
> <izadi-at-google.com at ffmpeg.org> wrote:
> >
> > HDR10+ metadata is stored in the bit stream for HEVC. The story is different for VP9 and cannot store the metadata in the bit stream. HDR10+ should be passed to packet side data an stored in the container (mkv) for VP9.
> >
> > This CL is taking HDR10+ from AVFrame side data in libvpxenc and is passing it to the AVPacket side data.
> > ---
> >  doc/APIchanges         |  2 +
> >  libavcodec/avpacket.c  |  1 +
> >  libavcodec/decode.c    |  1 +
> >  libavcodec/libvpxenc.c | 91 ++++++++++++++++++++++++++++++++++++++++++
> >  libavcodec/packet.h    |  8 ++++
> >  libavcodec/version.h   |  4 +-
> >  6 files changed, 105 insertions(+), 2 deletions(-)
> >
>
> lgtm with updates for version.h and doc/APIchanges locally. I'll
> submit this soon if there aren't any additional comments.
>

applied. thanks for the patch.

> > diff --git a/doc/APIchanges b/doc/APIchanges
> > index 06493763b3..ee7881e1e9 100644
> > --- a/doc/APIchanges
> > +++ b/doc/APIchanges
> > @@ -13,6 +13,8 @@ libavutil:     2021-04-27
> >
> >
> >  API changes, most recent first:
> > +2021-05-25 - 8c88a66d3c - lavc 59.2.100 - packet.h
>
> 8c88a66d3c should be xxxxxxxxxx to start with since the commit hash will change.