James Almer: > On 8/3/2023 3:01 AM, fei.w.wang-at-intel....@ffmpeg.org wrote: >> From: Fei Wang<fei.w.w...@intel.com> >> >> Signed-off-by: Fei Wang<fei.w.w...@intel.com> >> --- >> update: >> 1. Rename libavcodec/av1_levels*. >> 2. Use array instead of handle for AV1LevelDescriptor.name. >> 3. Compile libavcodec/av1_levels* only when enable vaapi av1 encoder. >> >> libavcodec/Makefile | 1 + >> libavcodec/av1_levels.c | 92 +++++++++++++++++++++++++ >> libavcodec/av1_levels.h | 58 ++++++++++++++++ >> libavcodec/tests/.gitignore | 1 + >> libavcodec/tests/av1_levels.c | 124 ++++++++++++++++++++++++++++++++++ >> tests/fate/libavcodec.mak | 5 ++ >> 6 files changed, 281 insertions(+) >> create mode 100644 libavcodec/av1_levels.c >> create mode 100644 libavcodec/av1_levels.h >> create mode 100644 libavcodec/tests/av1_levels.c >> >> diff --git a/libavcodec/Makefile b/libavcodec/Makefile >> index 3c16b51462..a6b2ecbb22 100644 >> --- a/libavcodec/Makefile >> +++ b/libavcodec/Makefile >> @@ -1319,6 +1319,7 @@ TESTPROGS = >> avcodec \ >> >> jpeg2000dwt \ >> >> mathops \ >> +TESTPROGS-$(CONFIG_AV1_VAAPI_ENCODER) += av1_levels > > This encoder does not exist at the point this patch would be applied. > It's introduced in patch 6/6. > > Squash libavcodec/av1_levels.c into patch 6/6 (Its first user), and then > add the test in a new patch 7.
Alternatively, add the file, but without the Makefile parts, which are then to be added 6/6. - Andreas _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
