From 3153e7f46d414e032756d79ab138cd622cd74c67 Mon Sep 17 00:00:00 2001 From: Seungha Yang Date: Sun, 9 Apr 2023 00:36:22 +0900 Subject: [PATCH] amfav1enc: Workaround driver bug with bt601 color matrix If bt601 is specified, driver generates sequence header with identity color matrix Part-of: --- subprojects/gst-plugins-bad/sys/amfcodec/gstamfav1enc.cpp | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/subprojects/gst-plugins-bad/sys/amfcodec/gstamfav1enc.cpp b/subprojects/gst-plugins-bad/sys/amfcodec/gstamfav1enc.cpp index 4a8ae6d0a7..12be1a68fe 100644 --- a/subprojects/gst-plugins-bad/sys/amfcodec/gstamfav1enc.cpp +++ b/subprojects/gst-plugins-bad/sys/amfcodec/gstamfav1enc.cpp @@ -1021,6 +1021,9 @@ gst_amf_av1_enc_set_format (GstAmfEncoder * encoder, } color_profile = AMF_VIDEO_CONVERTER_COLOR_PROFILE_UNKNOWN; switch (cinfo->matrix) { + /* XXX: There's a driver bug that BT601 color matrix here results in + * identity matrix specified */ +#if 0 case GST_VIDEO_COLOR_MATRIX_BT601: if (cinfo->range == GST_VIDEO_COLOR_RANGE_0_255) { color_profile = AMF_VIDEO_CONVERTER_COLOR_PROFILE_FULL_601; @@ -1028,6 +1031,7 @@ gst_amf_av1_enc_set_format (GstAmfEncoder * encoder, color_profile = AMF_VIDEO_CONVERTER_COLOR_PROFILE_601; } break; +#endif case GST_VIDEO_COLOR_MATRIX_BT709: if (cinfo->range == GST_VIDEO_COLOR_RANGE_0_255) { color_profile = AMF_VIDEO_CONVERTER_COLOR_PROFILE_FULL_709;