Re: [FFmpeg-devel] [PATCH v1] avcodec/apv_encoder: Updated APV encoder to set the encoder output to OAPV_CFG_VAL_AU_BS_FMT_NONE format (the only AU without bitstream format)

[James Almer <jamrial@gmail.com>]

[-- Attachment #1.1.1: Type: text/plain, Size: 1370 bytes --]

On 5/9/2025 8:52 AM, Dawid Kozinski wrote:
> Signed-off-by: Dawid Kozinski <d.kozinski@samsung.com>
> ---
>   libavcodec/liboapvenc.c | 12 ++++++++++++
>   1 file changed, 12 insertions(+)
> 
> diff --git a/libavcodec/liboapvenc.c b/libavcodec/liboapvenc.c
> index 0c95489bc9..d338b33e2d 100644
> --- a/libavcodec/liboapvenc.c
> +++ b/libavcodec/liboapvenc.c
> @@ -307,6 +307,18 @@ static av_cold int liboapve_init(AVCodecContext *avctx)
>           return AVERROR_EXTERNAL;
>       }
>   
> +    {
> +        int size, value;
> +
> +        value = OAPV_CFG_VAL_AU_BS_FMT_NONE;
> +        size = 4;
> +        ret = oapve_config(apv->id, OAPV_CFG_SET_AU_BS_FMT, &value, &size);
> +        if(OAPV_FAILED(ret)) {
> +            av_log(avctx, AV_LOG_ERROR, "Failed to set config for using encoder output format\n");
> +            return AVERROR_EXTERNAL;
> +        }
> +    }
> +
>       apv->ifrms.frm[FRM_IDX].imgb = apv_imgb_create(avctx);
>       if (apv->ifrms.frm[FRM_IDX].imgb == NULL)
>           return AVERROR(ENOMEM);

Before we start adding support for recent API additions, liboapv needs 
to have versioning. See 
https://github.com/AcademySoftwareFoundation/openapv/issues/86
Otherwise, we have no way to ensure certain symbols are present and 
compilation will fail if you don't have the correct git master snapshot.


[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 495 bytes --]

[-- Attachment #2: Type: text/plain, Size: 251 bytes --]

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".