[FFmpeg-devel] [PATCH] avcodec/dxva2_av1: fix superres_denom parameter

[Hendrik Leppkes]

On Wed, Sep 8, 2021 at 7:21 AM Tong Wu <tong1.wu at intel.com> wrote:
>
> Defined in spec 5.9.8. When superres is enabled, SuperresDenom equals
> "coded_denom + SUPERRES_DENOM_MIN" instead of coded_denom.
>
> Signed-off-by: Tong Wu <tong1.wu at intel.com>
> ---
>  libavcodec/dxva2_av1.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavcodec/dxva2_av1.c b/libavcodec/dxva2_av1.c
> index aa14e473df..c30b57799c 100644
> --- a/libavcodec/dxva2_av1.c
> +++ b/libavcodec/dxva2_av1.c
> @@ -73,7 +73,7 @@ static int fill_picture_parameters(const AVCodecContext *avctx, AVDXVAContext *c
>      pp->max_height = seq->max_frame_height_minus_1 + 1;
>
>      pp->CurrPicTextureIndex = ff_dxva2_get_surface_index(avctx, ctx, h->cur_frame.tf.f);
> -    pp->superres_denom      = frame_header->use_superres ? frame_header->coded_denom : AV1_SUPERRES_NUM;
> +    pp->superres_denom      = frame_header->use_superres ? frame_header->coded_denom + AV1_SUPERRES_DENOM_MIN : AV1_SUPERRES_NUM;
>      pp->bitdepth            = get_bit_depth_from_seq(seq);
>      pp->seq_profile         = seq->seq_profile;
>
> --
> 2.16.1.windows.4
>

LGTM. Confirmed in the spec. Will apply soon.