On Sun, May 03, 2020 at 10:01:31AM +0200, Marton Balint wrote:
>
>
> On Fri, 15 Nov 2019, Limin Wang wrote:
>
> >
> > ping, the patchset is for HDR/HLG color support for prores.
> >
> > On Mon, Nov 04, 2019 at 07:10:00PM +0800, lance.lmw...@gmail.com wrote:
> > > From: Limin Wang <lance.lmw...@gmail.com>
> > >
> > > It's based on the following specs:
> > > RDD 36:2015 - SMPTE Registered Disclosure Doc - Apple ProRes Bitstream
> > > Syntax and Decoding Process
> > >
> > > Signed-off-by: Limin Wang <lance.lmw...@gmail.com>
> > > ---
> > > doc/bitstream_filters.texi | 2 ++
> > > libavcodec/prores_metadata_bsf.c | 3 ++-
> > > 2 files changed, 4 insertions(+), 1 deletion(-)
> > >
> > > diff --git a/doc/bitstream_filters.texi b/doc/bitstream_filters.texi
> > > index 8fe5b3ad75..e2de4fd4ed 100644
> > > --- a/doc/bitstream_filters.texi
> > > +++ b/doc/bitstream_filters.texi
> > > @@ -589,6 +589,8 @@ Keep the same transfer characteristics property
> > > (default).
> > > @item unknown
> > > @item bt709
> > > BT 601, BT 709, BT 2020
> > > +@item smpte2084
> > > +SMPTE ST 2084
> > > @end table
> > >
> > >
> > > diff --git a/libavcodec/prores_metadata_bsf.c
> > > b/libavcodec/prores_metadata_bsf.c
> > > index 0510d3520a..24615e9305 100644
> > > --- a/libavcodec/prores_metadata_bsf.c
> > > +++ b/libavcodec/prores_metadata_bsf.c
> > > @@ -140,10 +140,11 @@ static const AVOption options[] = {
> > > {"smpte431", NULL, 0, AV_OPT_TYPE_CONST,
> > > {.i64=AVCOL_PRI_SMPTE431}, INT_MIN, INT_MAX, FLAGS,
> > > "color_primaries"},
> > > {"smpte432", NULL, 0, AV_OPT_TYPE_CONST,
> > > {.i64=AVCOL_PRI_SMPTE432}, INT_MIN, INT_MAX, FLAGS,
> > > "color_primaries"},
> > >
> > > - {"color_trc", "select color transfer",
> > > OFFSET(transfer_characteristics), AV_OPT_TYPE_INT, {.i64=-1}, -1,
> > > AVCOL_TRC_BT709, FLAGS, "color_trc"},
> > > + {"color_trc", "select color transfer",
> > > OFFSET(transfer_characteristics), AV_OPT_TYPE_INT, {.i64=-1}, -1,
> > > AVCOL_TRC_NB, FLAGS, "color_trc"},
>
> NB-1.
>
> > > {"auto", "keep the same color transfer", 0, AV_OPT_TYPE_CONST,
> > > {.i64=-1}, INT_MIN, INT_MAX, FLAGS,
> > > "color_trc"},
> > > {"unknown", NULL, 0, AV_OPT_TYPE_CONST,
> > > {.i64=0}, INT_MIN, INT_MAX, FLAGS,
> > > "color_trc"},
> > > {"bt709", NULL, 0, AV_OPT_TYPE_CONST,
> > > {.i64=AVCOL_TRC_BT709}, INT_MIN, INT_MAX, FLAGS,
> > > "color_trc"},
> > > + {"smpte2084", NULL, 0, AV_OPT_TYPE_CONST,
> > > {.i64=AVCOL_TRC_SMPTE2084}, INT_MIN, INT_MAX, FLAGS,
> > > "color_trc"},
> > >
> > > {"colorspace", "select colorspace", OFFSET(matrix_coefficients),
> > > AV_OPT_TYPE_INT, {.i64=-1}, -1, AVCOL_SPC_BT2020_NCL, FLAGS,
> > > "colorspace"},
> > > {"auto", "keep the same colorspace", 0, AV_OPT_TYPE_CONST,
> > > {.i64=-1}, INT_MIN, INT_MAX, FLAGS,
> > > "colorspace"},
>
> Yet again, please, do not apply patches which have not been LGTM-ed and you
> have not pinged them recently.
Sorry, I think I have ping before. Next time I'll try to ping again for the old
patches
or resubmit.
>
> Thanks,
> Marton
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
--
Thanks,
Limin Wang
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
