From c1f5108ac363b05639970e3d1529b206666251c2 Mon Sep 17 00:00:00 2001 From: Markus Fritsche Date: Sat, 25 Apr 2026 21:13:11 +0000 Subject: [PATCH] include/h264-ctrls.h, src/config.c: rename to V4L2_PIX_FMT_H264_SLICE MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Kernel mainline renamed V4L2_PIX_FMT_H264_SLICE_RAW → V4L2_PIX_FMT_H264_SLICE some time after this library went dormant (the 'S264' fourcc value is unchanged; only the C identifier moved). The two definitions have the same value so this is purely cosmetic at runtime, but keeping the name aligned with linux/videodev2.h matches downstream patches (e.g. bootlin PR #38) and avoids confusion when reading kernel + library side-by-side. Co-Authored-By: Claude Opus 4.7 (1M context) --- include/h264-ctrls.h | 2 +- src/config.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/include/h264-ctrls.h b/include/h264-ctrls.h index e1404d7..6160a69 100644 --- a/include/h264-ctrls.h +++ b/include/h264-ctrls.h @@ -14,7 +14,7 @@ #include /* Our pixel format isn't stable at the moment */ -#define V4L2_PIX_FMT_H264_SLICE_RAW v4l2_fourcc('S', '2', '6', '4') /* H264 parsed slices */ +#define V4L2_PIX_FMT_H264_SLICE v4l2_fourcc('S', '2', '6', '4') /* H264 parsed slices */ /* * This is put insanely high to avoid conflicting with controls that diff --git a/src/config.c b/src/config.c index e396268..8c08148 100644 --- a/src/config.c +++ b/src/config.c @@ -128,7 +128,7 @@ VAStatus RequestQueryConfigProfiles(VADriverContextP context, found = v4l2_find_format(driver_data->video_fd, V4L2_BUF_TYPE_VIDEO_OUTPUT, - V4L2_PIX_FMT_H264_SLICE_RAW); + V4L2_PIX_FMT_H264_SLICE); if (found && index < (V4L2_REQUEST_MAX_CONFIG_ATTRIBUTES - 5)) { profiles[index++] = VAProfileH264Main; profiles[index++] = VAProfileH264High;