media: uapi: h264: Rename pixel format

The V4L2_PIX_FMT_H264_SLICE_RAW name was originally suggested
because the pixel format would represent H264 slices without any
start code.

However, as we will now introduce a start code menu control,
give the pixel format a more meaningful name, while it's
still early enough to do so.

Signed-off-by: Ezequiel Garcia <ezequiel@collabora.com>
Tested-by: Philipp Zabel <p.zabel@pengutronix.de>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
This commit is contained in:
Ezequiel Garcia 2019-08-16 13:01:23 -03:00 committed by Mauro Carvalho Chehab
parent 4333fb96ca
commit 7bb3c32abd
5 changed files with 8 additions and 8 deletions

View file

@ -52,9 +52,9 @@ Compressed Formats
- ``V4L2_PIX_FMT_H264_MVC`` - ``V4L2_PIX_FMT_H264_MVC``
- 'M264' - 'M264'
- H264 MVC video elementary stream. - H264 MVC video elementary stream.
* .. _V4L2-PIX-FMT-H264-SLICE-RAW: * .. _V4L2-PIX-FMT-H264-SLICE:
- ``V4L2_PIX_FMT_H264_SLICE_RAW`` - ``V4L2_PIX_FMT_H264_SLICE``
- 'S264' - 'S264'
- H264 parsed slice data, without the start code and as - H264 parsed slice data, without the start code and as
extracted from the H264 bitstream. This format is adapted for extracted from the H264 bitstream. This format is adapted for

View file

@ -1343,7 +1343,7 @@ static void v4l_fill_fmtdesc(struct v4l2_fmtdesc *fmt)
case V4L2_PIX_FMT_H264: descr = "H.264"; break; case V4L2_PIX_FMT_H264: descr = "H.264"; break;
case V4L2_PIX_FMT_H264_NO_SC: descr = "H.264 (No Start Codes)"; break; case V4L2_PIX_FMT_H264_NO_SC: descr = "H.264 (No Start Codes)"; break;
case V4L2_PIX_FMT_H264_MVC: descr = "H.264 MVC"; break; case V4L2_PIX_FMT_H264_MVC: descr = "H.264 MVC"; break;
case V4L2_PIX_FMT_H264_SLICE_RAW: descr = "H.264 Parsed Slice Data"; break; case V4L2_PIX_FMT_H264_SLICE: descr = "H.264 Parsed Slice Data"; break;
case V4L2_PIX_FMT_H263: descr = "H.263"; break; case V4L2_PIX_FMT_H263: descr = "H.263"; break;
case V4L2_PIX_FMT_MPEG1: descr = "MPEG-1 ES"; break; case V4L2_PIX_FMT_MPEG1: descr = "MPEG-1 ES"; break;
case V4L2_PIX_FMT_MPEG2: descr = "MPEG-2 ES"; break; case V4L2_PIX_FMT_MPEG2: descr = "MPEG-2 ES"; break;

View file

@ -46,7 +46,7 @@ void cedrus_device_run(void *priv)
V4L2_CID_MPEG_VIDEO_MPEG2_QUANTIZATION); V4L2_CID_MPEG_VIDEO_MPEG2_QUANTIZATION);
break; break;
case V4L2_PIX_FMT_H264_SLICE_RAW: case V4L2_PIX_FMT_H264_SLICE:
run.h264.decode_params = cedrus_find_control_data(ctx, run.h264.decode_params = cedrus_find_control_data(ctx,
V4L2_CID_MPEG_VIDEO_H264_DECODE_PARAMS); V4L2_CID_MPEG_VIDEO_H264_DECODE_PARAMS);
run.h264.pps = cedrus_find_control_data(ctx, run.h264.pps = cedrus_find_control_data(ctx,

View file

@ -38,7 +38,7 @@ static struct cedrus_format cedrus_formats[] = {
.directions = CEDRUS_DECODE_SRC, .directions = CEDRUS_DECODE_SRC,
}, },
{ {
.pixelformat = V4L2_PIX_FMT_H264_SLICE_RAW, .pixelformat = V4L2_PIX_FMT_H264_SLICE,
.directions = CEDRUS_DECODE_SRC, .directions = CEDRUS_DECODE_SRC,
}, },
{ {
@ -104,7 +104,7 @@ static void cedrus_prepare_format(struct v4l2_pix_format *pix_fmt)
switch (pix_fmt->pixelformat) { switch (pix_fmt->pixelformat) {
case V4L2_PIX_FMT_MPEG2_SLICE: case V4L2_PIX_FMT_MPEG2_SLICE:
case V4L2_PIX_FMT_H264_SLICE_RAW: case V4L2_PIX_FMT_H264_SLICE:
/* Zero bytes per line for encoded source. */ /* Zero bytes per line for encoded source. */
bytesperline = 0; bytesperline = 0;
@ -449,7 +449,7 @@ static int cedrus_start_streaming(struct vb2_queue *vq, unsigned int count)
ctx->current_codec = CEDRUS_CODEC_MPEG2; ctx->current_codec = CEDRUS_CODEC_MPEG2;
break; break;
case V4L2_PIX_FMT_H264_SLICE_RAW: case V4L2_PIX_FMT_H264_SLICE:
ctx->current_codec = CEDRUS_CODEC_H264; ctx->current_codec = CEDRUS_CODEC_H264;
break; break;

View file

@ -14,7 +14,7 @@
#include <linux/videodev2.h> #include <linux/videodev2.h>
/* Our pixel format isn't stable at the moment */ /* 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 * This is put insanely high to avoid conflicting with controls that