radeonsi/vcn: Fix setting color range in AV1 bitstream
authorDavid Rosca <nowrep@gmail.com>
Mon, 17 Jul 2023 10:57:34 +0000 (12:57 +0200)
committerMarge Bot <emma+marge@anholt.net>
Wed, 19 Jul 2023 18:51:48 +0000 (18:51 +0000)
As per spec color_range and chroma_sample_position parameters
are always set, not conditional on color_description_present_flag.

Reviewed-by: Ruijing Dong <ruijing.dong@amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/24179>

src/gallium/drivers/radeonsi/radeon_vcn_enc.c

index cb4d9e3..3cdbc4a 100644 (file)
@@ -542,9 +542,9 @@ static void radeon_vcn_enc_av1_color_description(struct radeon_encoder *enc,
       enc->enc_pic.av1_color_description.color_primaries = pic->seq.color_config.color_primaries;
       enc->enc_pic.av1_color_description.transfer_characteristics = pic->seq.color_config.transfer_characteristics;
       enc->enc_pic.av1_color_description.maxtrix_coefficients = pic->seq.color_config.matrix_coefficients;
-      enc->enc_pic.av1_color_description.color_range = pic->seq.color_config.color_range;
-      enc->enc_pic.av1_color_description.chroma_sample_position = pic->seq.color_config.chroma_sample_position;
    }
+   enc->enc_pic.av1_color_description.color_range = pic->seq.color_config.color_range;
+   enc->enc_pic.av1_color_description.chroma_sample_position = pic->seq.color_config.chroma_sample_position;
 }
 
 static void radeon_vcn_enc_av1_get_rc_param(struct radeon_encoder *enc,