> From: ffmpeg-devel <[email protected]> On Behalf Of Andreas
> Rheinhardt
> Sent: 2023年10月12日 22:01
> To: [email protected]
> Subject: Re: [FFmpeg-devel] [PATCH 1/2] avcodec/videotoolboxenc: add
> hw_configs
>
> Zhao Zhili:
> > From: Zhao Zhili <[email protected]>
> >
> > Will be used in the following patches.
>
> It doesn't seem to be used for 2/2, so it should be applied with the
> patches that actually need it.
It's used in an nonobvious way. With hw_config we can get avctx->hw_frames_ctx,
and with avctx->hw_frames_ctx we get sw_pix_fmt. Otherwise sw_pix_fmt is none.
I need sw_pix_fmt before get the first frame.
>
> > ---
> > libavcodec/videotoolboxenc.c | 9 +++++++++
> > 1 file changed, 9 insertions(+)
> >
> > diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c
> > index 1d1595329a..b685bf4cc4 100644
> > --- a/libavcodec/videotoolboxenc.c
> > +++ b/libavcodec/videotoolboxenc.c
> > @@ -37,6 +37,7 @@
> > #include "encode.h"
> > #include "h264.h"
> > #include "h264_sei.h"
> > +#include "hwconfig.h"
> > #include <dlfcn.h>
> >
> > #if !HAVE_KCMVIDEOCODECTYPE_HEVC
> > @@ -2830,6 +2831,11 @@ static const enum AVPixelFormat prores_pix_fmts[] = {
> > "Sets the maximum number of reference frames. This only has an
> > effect when the value is less than the maximum allowed
> by the profile/level.", \
> > OFFSET(max_ref_frames), AV_OPT_TYPE_INT, { .i64 = 0 }, 0, INT_MAX,
> > VE },
> >
> > +static const AVCodecHWConfigInternal *const vt_encode_hw_configs[] = {
> > + HW_CONFIG_ENCODER_FRAMES(VIDEOTOOLBOX, VIDEOTOOLBOX),
> > + NULL,
> > +};
> > +
> > #define OFFSET(x) offsetof(VTEncContext, x)
> > static const AVOption h264_options[] = {
> > { "profile", "Profile", OFFSET(profile), AV_OPT_TYPE_INT, { .i64 =
> > AV_PROFILE_UNKNOWN }, AV_PROFILE_UNKNOWN, INT_MAX,
> VE, "profile" },
> > @@ -2886,6 +2892,7 @@ const FFCodec ff_h264_videotoolbox_encoder = {
> > .close = vtenc_close,
> > .p.priv_class = &h264_videotoolbox_class,
> > .caps_internal = FF_CODEC_CAP_INIT_CLEANUP,
> > + .hw_configs = vt_encode_hw_configs,
> > };
> >
> > static const AVOption hevc_options[] = {
> > @@ -2923,6 +2930,7 @@ const FFCodec ff_hevc_videotoolbox_encoder = {
> > .p.priv_class = &hevc_videotoolbox_class,
> > .caps_internal = FF_CODEC_CAP_INIT_CLEANUP,
> > .p.wrapper_name = "videotoolbox",
> > + .hw_configs = vt_encode_hw_configs,
> > };
> >
> > static const AVOption prores_options[] = {
> > @@ -2961,4 +2969,5 @@ const FFCodec ff_prores_videotoolbox_encoder = {
> > .p.priv_class = &prores_videotoolbox_class,
> > .caps_internal = FF_CODEC_CAP_INIT_CLEANUP,
> > .p.wrapper_name = "videotoolbox",
> > + .hw_configs = vt_encode_hw_configs,
> > };
>
> _______________________________________________
> ffmpeg-devel mailing list
> [email protected]
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> [email protected] with subject "unsubscribe".
_______________________________________________
ffmpeg-devel mailing list
[email protected]
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
[email protected] with subject "unsubscribe".
