anv: Fix to set CDEF flter flag correctly for AV1 decoding

and relevant tiny clean-up.

Fixes: 8432b8b282 ("anv: add initial support for AV1 decoding")

Signed-off-by: Hyunjun Ko <zzoon@igalia.com>
Acked-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/33316>
(cherry picked from commit 52d9edbf05)
This commit is contained in:
Hyunjun Ko 2025-01-31 16:18:41 +09:00 committed by Eric Engestrom
parent efdd9452fe
commit cd4ffc319f
2 changed files with 3 additions and 3 deletions

View file

@ -1224,7 +1224,7 @@
"description": "anv: Fix to set CDEF flter flag correctly for AV1 decoding",
"nominated": true,
"nomination_type": 2,
"resolution": 0,
"resolution": 1,
"main_sha": null,
"because_sha": "8432b8b2828bca6fa19793b00b3a6e61a06ff650",
"notes": null

View file

@ -2098,7 +2098,7 @@ anv_av1_decode_video_tile(struct anv_cmd_buffer *cmd_buffer,
pic.AllowScreenContentToolsFlag = std_pic_info->flags.allow_screen_content_tools;
pic.ForceIntegerMVFlag = std_pic_info->flags.force_integer_mv;
pic.AllowWarpedMotionFlag = std_pic_info->flags.allow_warped_motion;
pic.UseCDEFFilterFlag = seq_hdr->flags.enable_cdef;
pic.UseCDEFFilterFlag = !frame_lossless && seq_hdr->flags.enable_cdef;
pic.UseSuperResFlag = std_pic_info->flags.use_superres;
pic.FrameLevelLoopRestorationFilterEnable = frame_restoration_type[0] || frame_restoration_type[1] || frame_restoration_type[2];
pic.FrameType = std_pic_info->frame_type;
@ -2113,7 +2113,7 @@ anv_av1_decode_video_tile(struct anv_cmd_buffer *cmd_buffer,
pic.LastActiveSegmentSegmentID = last_active_segid;
pic.DeltaQPresentFlag = std_pic_info->flags.delta_q_present;
pic.DeltaQRes = std_pic_info->delta_q_res;
pic.FrameCodedLosslessMode = frame_lossless; /* TODO */
pic.FrameCodedLosslessMode = frame_lossless;
pic.SegmentMapisZeroFlag = 0; /* TODO */
pic.SegmentIDBufferStreamInEnableFlag = 0; /* TODO */
pic.SegmentIDBufferStreamOutEnableFlag = 0; /* TODO */