Skip to content

L
linux
Commit 9aa5f008 authored by Kamil Debski's avatar Kamil Debski Committed by Mauro Carvalho Chehab
Browse files
Options

[media] v4l: s5p-mfc: Limit enum_fmt to output formats of current version 

MFC versions support a different set of formats, this specially applies
to the raw YUV formats. This patch changes enum_fmt, so that it only
reports formats that are supported by the used MFC version.
Signed-off-by: default avatarKamil Debski <k.debski@samsung.com>
Acked-by: default avatarSylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: default avatarMauro Carvalho Chehab <m.chehab@samsung.com>
parent 264e5bac
Hide whitespace changes
Inline Side-by-side
Showing with 67 additions and 42 deletions
drivers/media/platform/s5p-mfc/s5p_mfc.c
View file @ 9aa5f008
...@@ -1339,6 +1339,7 @@ struct s5p_mfc_buf_align mfc_buf_align_v5 = { ...@@ -1339,6 +1339,7 @@ struct s5p_mfc_buf_align mfc_buf_align_v5 = {
static struct s5p_mfc_variant mfc_drvdata_v5 = { static struct s5p_mfc_variant mfc_drvdata_v5 = {
.version = MFC_VERSION, .version = MFC_VERSION,
.version_bit = MFC_V5_BIT,
.port_num = MFC_NUM_PORTS, .port_num = MFC_NUM_PORTS,
.buf_size = &buf_size_v5, .buf_size = &buf_size_v5,
.buf_align = &mfc_buf_align_v5, .buf_align = &mfc_buf_align_v5,
...@@ -1365,6 +1366,7 @@ struct s5p_mfc_buf_align mfc_buf_align_v6 = { ...@@ -1365,6 +1366,7 @@ struct s5p_mfc_buf_align mfc_buf_align_v6 = {
static struct s5p_mfc_variant mfc_drvdata_v6 = { static struct s5p_mfc_variant mfc_drvdata_v6 = {
.version = MFC_VERSION_V6, .version = MFC_VERSION_V6,
.version_bit = MFC_V6_BIT,
.port_num = MFC_NUM_PORTS_V6, .port_num = MFC_NUM_PORTS_V6,
.buf_size = &buf_size_v6, .buf_size = &buf_size_v6,
.buf_align = &mfc_buf_align_v6, .buf_align = &mfc_buf_align_v6,
...@@ -1391,6 +1393,7 @@ struct s5p_mfc_buf_align mfc_buf_align_v7 = { ...@@ -1391,6 +1393,7 @@ struct s5p_mfc_buf_align mfc_buf_align_v7 = {
static struct s5p_mfc_variant mfc_drvdata_v7 = { static struct s5p_mfc_variant mfc_drvdata_v7 = {
.version = MFC_VERSION_V7, .version = MFC_VERSION_V7,
.version_bit = MFC_V7_BIT,
.port_num = MFC_NUM_PORTS_V7, .port_num = MFC_NUM_PORTS_V7,
.buf_size = &buf_size_v7, .buf_size = &buf_size_v7,
.buf_align = &mfc_buf_align_v7, .buf_align = &mfc_buf_align_v7,
......
drivers/media/platform/s5p-mfc/s5p_mfc_common.h
View file @ 9aa5f008
...@@ -223,6 +223,7 @@ struct s5p_mfc_buf_align { ...@@ -223,6 +223,7 @@ struct s5p_mfc_buf_align {
struct s5p_mfc_variant { struct s5p_mfc_variant {
unsigned int version; unsigned int version;
unsigned int port_num; unsigned int port_num;
u32 version_bit;
struct s5p_mfc_buf_size *buf_size; struct s5p_mfc_buf_size *buf_size;
struct s5p_mfc_buf_align *buf_align; struct s5p_mfc_buf_align *buf_align;
char *fw_name; char *fw_name;
...@@ -664,6 +665,7 @@ struct s5p_mfc_fmt { ...@@ -664,6 +665,7 @@ struct s5p_mfc_fmt {
u32 codec_mode; u32 codec_mode;
enum s5p_mfc_fmt_type type; enum s5p_mfc_fmt_type type;
u32 num_planes; u32 num_planes;
u32 versions;
}; };
/** /**
...@@ -703,4 +705,9 @@ void set_work_bit_irqsave(struct s5p_mfc_ctx *ctx); ...@@ -703,4 +705,9 @@ void set_work_bit_irqsave(struct s5p_mfc_ctx *ctx);
#define IS_MFCV6_PLUS(dev) (dev->variant->version >= 0x60 ? 1 : 0) #define IS_MFCV6_PLUS(dev) (dev->variant->version >= 0x60 ? 1 : 0)
#define IS_MFCV7_PLUS(dev) (dev->variant->version >= 0x70 ? 1 : 0) #define IS_MFCV7_PLUS(dev) (dev->variant->version >= 0x70 ? 1 : 0)
#define MFC_V5_BIT BIT(0)
#define MFC_V6_BIT BIT(1)
#define MFC_V7_BIT BIT(2)
#endif /* S5P_MFC_COMMON_H_ */ #endif /* S5P_MFC_COMMON_H_ */
drivers/media/platform/s5p-mfc/s5p_mfc_dec.c
View file @ 9aa5f008
...@@ -39,6 +39,7 @@ static struct s5p_mfc_fmt formats[] = { ...@@ -39,6 +39,7 @@ static struct s5p_mfc_fmt formats[] = {
.codec_mode = S5P_MFC_CODEC_NONE, .codec_mode = S5P_MFC_CODEC_NONE,
.type = MFC_FMT_RAW, .type = MFC_FMT_RAW,
.num_planes = 2, .num_planes = 2,
.versions = MFC_V6_BIT | MFC_V7_BIT,
}, },
{ {
.name = "4:2:0 2 Planes 64x32 Tiles", .name = "4:2:0 2 Planes 64x32 Tiles",
...@@ -46,6 +47,7 @@ static struct s5p_mfc_fmt formats[] = { ...@@ -46,6 +47,7 @@ static struct s5p_mfc_fmt formats[] = {
.codec_mode = S5P_MFC_CODEC_NONE, .codec_mode = S5P_MFC_CODEC_NONE,
.type = MFC_FMT_RAW, .type = MFC_FMT_RAW,
.num_planes = 2, .num_planes = 2,
.versions = MFC_V5_BIT,
}, },
{ {
.name = "4:2:0 2 Planes Y/CbCr", .name = "4:2:0 2 Planes Y/CbCr",
...@@ -53,6 +55,7 @@ static struct s5p_mfc_fmt formats[] = { ...@@ -53,6 +55,7 @@ static struct s5p_mfc_fmt formats[] = {
.codec_mode = S5P_MFC_CODEC_NONE, .codec_mode = S5P_MFC_CODEC_NONE,
.type = MFC_FMT_RAW, .type = MFC_FMT_RAW,
.num_planes = 2, .num_planes = 2,
.versions = MFC_V6_BIT | MFC_V7_BIT,
}, },
{ {
.name = "4:2:0 2 Planes Y/CrCb", .name = "4:2:0 2 Planes Y/CrCb",
...@@ -60,6 +63,7 @@ static struct s5p_mfc_fmt formats[] = { ...@@ -60,6 +63,7 @@ static struct s5p_mfc_fmt formats[] = {
.codec_mode = S5P_MFC_CODEC_NONE, .codec_mode = S5P_MFC_CODEC_NONE,
.type = MFC_FMT_RAW, .type = MFC_FMT_RAW,
.num_planes = 2, .num_planes = 2,
.versions = MFC_V6_BIT | MFC_V7_BIT,
}, },
{ {
.name = "H264 Encoded Stream", .name = "H264 Encoded Stream",
...@@ -67,6 +71,7 @@ static struct s5p_mfc_fmt formats[] = { ...@@ -67,6 +71,7 @@ static struct s5p_mfc_fmt formats[] = {
.codec_mode = S5P_MFC_CODEC_H264_DEC, .codec_mode = S5P_MFC_CODEC_H264_DEC,
.type = MFC_FMT_DEC, .type = MFC_FMT_DEC,
.num_planes = 1, .num_planes = 1,
.versions = MFC_V5_BIT | MFC_V6_BIT | MFC_V7_BIT,
}, },
{ {
.name = "H264/MVC Encoded Stream", .name = "H264/MVC Encoded Stream",
...@@ -74,6 +79,7 @@ static struct s5p_mfc_fmt formats[] = { ...@@ -74,6 +79,7 @@ static struct s5p_mfc_fmt formats[] = {
.codec_mode = S5P_MFC_CODEC_H264_MVC_DEC, .codec_mode = S5P_MFC_CODEC_H264_MVC_DEC,
.type = MFC_FMT_DEC, .type = MFC_FMT_DEC,
.num_planes = 1, .num_planes = 1,
.versions = MFC_V6_BIT | MFC_V7_BIT,
}, },
{ {
.name = "H263 Encoded Stream", .name = "H263 Encoded Stream",
...@@ -81,6 +87,7 @@ static struct s5p_mfc_fmt formats[] = { ...@@ -81,6 +87,7 @@ static struct s5p_mfc_fmt formats[] = {
.codec_mode = S5P_MFC_CODEC_H263_DEC, .codec_mode = S5P_MFC_CODEC_H263_DEC,
.type = MFC_FMT_DEC, .type = MFC_FMT_DEC,
.num_planes = 1, .num_planes = 1,
.versions = MFC_V5_BIT | MFC_V6_BIT | MFC_V7_BIT,
}, },
{ {
.name = "MPEG1 Encoded Stream", .name = "MPEG1 Encoded Stream",
...@@ -88,6 +95,7 @@ static struct s5p_mfc_fmt formats[] = { ...@@ -88,6 +95,7 @@ static struct s5p_mfc_fmt formats[] = {
.codec_mode = S5P_MFC_CODEC_MPEG2_DEC, .codec_mode = S5P_MFC_CODEC_MPEG2_DEC,
.type = MFC_FMT_DEC, .type = MFC_FMT_DEC,
.num_planes = 1, .num_planes = 1,
.versions = MFC_V5_BIT | MFC_V6_BIT | MFC_V7_BIT,
}, },
{ {
.name = "MPEG2 Encoded Stream", .name = "MPEG2 Encoded Stream",
...@@ -95,6 +103,7 @@ static struct s5p_mfc_fmt formats[] = { ...@@ -95,6 +103,7 @@ static struct s5p_mfc_fmt formats[] = {
.codec_mode = S5P_MFC_CODEC_MPEG2_DEC, .codec_mode = S5P_MFC_CODEC_MPEG2_DEC,
.type = MFC_FMT_DEC, .type = MFC_FMT_DEC,
.num_planes = 1, .num_planes = 1,
.versions = MFC_V5_BIT | MFC_V6_BIT | MFC_V7_BIT,
}, },
{ {
.name = "MPEG4 Encoded Stream", .name = "MPEG4 Encoded Stream",
...@@ -102,6 +111,7 @@ static struct s5p_mfc_fmt formats[] = { ...@@ -102,6 +111,7 @@ static struct s5p_mfc_fmt formats[] = {
.codec_mode = S5P_MFC_CODEC_MPEG4_DEC, .codec_mode = S5P_MFC_CODEC_MPEG4_DEC,
.type = MFC_FMT_DEC, .type = MFC_FMT_DEC,
.num_planes = 1, .num_planes = 1,
.versions = MFC_V5_BIT | MFC_V6_BIT | MFC_V7_BIT,
}, },
{ {
.name = "XviD Encoded Stream", .name = "XviD Encoded Stream",
...@@ -109,6 +119,7 @@ static struct s5p_mfc_fmt formats[] = { ...@@ -109,6 +119,7 @@ static struct s5p_mfc_fmt formats[] = {
.codec_mode = S5P_MFC_CODEC_MPEG4_DEC, .codec_mode = S5P_MFC_CODEC_MPEG4_DEC,
.type = MFC_FMT_DEC, .type = MFC_FMT_DEC,
.num_planes = 1, .num_planes = 1,
.versions = MFC_V5_BIT | MFC_V6_BIT | MFC_V7_BIT,
}, },
{ {
.name = "VC1 Encoded Stream", .name = "VC1 Encoded Stream",
...@@ -116,6 +127,7 @@ static struct s5p_mfc_fmt formats[] = { ...@@ -116,6 +127,7 @@ static struct s5p_mfc_fmt formats[] = {
.codec_mode = S5P_MFC_CODEC_VC1_DEC, .codec_mode = S5P_MFC_CODEC_VC1_DEC,
.type = MFC_FMT_DEC, .type = MFC_FMT_DEC,
.num_planes = 1, .num_planes = 1,
.versions = MFC_V5_BIT | MFC_V6_BIT | MFC_V7_BIT,
}, },
{ {
.name = "VC1 RCV Encoded Stream", .name = "VC1 RCV Encoded Stream",
...@@ -123,6 +135,7 @@ static struct s5p_mfc_fmt formats[] = { ...@@ -123,6 +135,7 @@ static struct s5p_mfc_fmt formats[] = {
.codec_mode = S5P_MFC_CODEC_VC1RCV_DEC, .codec_mode = S5P_MFC_CODEC_VC1RCV_DEC,
.type = MFC_FMT_DEC, .type = MFC_FMT_DEC,
.num_planes = 1, .num_planes = 1,
.versions = MFC_V5_BIT | MFC_V6_BIT | MFC_V7_BIT,
}, },
{ {
.name = "VP8 Encoded Stream", .name = "VP8 Encoded Stream",
...@@ -130,6 +143,7 @@ static struct s5p_mfc_fmt formats[] = { ...@@ -130,6 +143,7 @@ static struct s5p_mfc_fmt formats[] = {
.codec_mode = S5P_MFC_CODEC_VP8_DEC, .codec_mode = S5P_MFC_CODEC_VP8_DEC,
.type = MFC_FMT_DEC, .type = MFC_FMT_DEC,
.num_planes = 1, .num_planes = 1,
.versions = MFC_V6_BIT | MFC_V7_BIT,
}, },
}; };
...@@ -260,8 +274,10 @@ static int vidioc_querycap(struct file *file, void *priv, ...@@ -260,8 +274,10 @@ static int vidioc_querycap(struct file *file, void *priv,
} }
/* Enumerate format */ /* Enumerate format */
static int vidioc_enum_fmt(struct v4l2_fmtdesc *f, bool mplane, bool out) static int vidioc_enum_fmt(struct file *file, struct v4l2_fmtdesc *f,
bool mplane, bool out)
{ {
struct s5p_mfc_dev *dev = video_drvdata(file);
struct s5p_mfc_fmt *fmt; struct s5p_mfc_fmt *fmt;
int i, j = 0; int i, j = 0;
...@@ -274,6 +290,8 @@ static int vidioc_enum_fmt(struct v4l2_fmtdesc *f, bool mplane, bool out) ...@@ -274,6 +290,8 @@ static int vidioc_enum_fmt(struct v4l2_fmtdesc *f, bool mplane, bool out)
continue; continue;
else if (!out && formats[i].type != MFC_FMT_RAW) else if (!out && formats[i].type != MFC_FMT_RAW)
continue; continue;
else if ((dev->variant->version_bit & formats[i].versions) == 0)
continue;
if (j == f->index) if (j == f->index)
break; break;
...@@ -290,25 +308,25 @@ static int vidioc_enum_fmt(struct v4l2_fmtdesc *f, bool mplane, bool out) ...@@ -290,25 +308,25 @@ static int vidioc_enum_fmt(struct v4l2_fmtdesc *f, bool mplane, bool out)
static int vidioc_enum_fmt_vid_cap(struct file *file, void *pirv, static int vidioc_enum_fmt_vid_cap(struct file *file, void *pirv,
struct v4l2_fmtdesc *f) struct v4l2_fmtdesc *f)
{ {
return vidioc_enum_fmt(f, false, false); return vidioc_enum_fmt(file, f, false, false);
} }
static int vidioc_enum_fmt_vid_cap_mplane(struct file *file, void *pirv, static int vidioc_enum_fmt_vid_cap_mplane(struct file *file, void *pirv,
struct v4l2_fmtdesc *f) struct v4l2_fmtdesc *f)
{ {
return vidioc_enum_fmt(f, true, false); return vidioc_enum_fmt(file, f, true, false);
} }
static int vidioc_enum_fmt_vid_out(struct file *file, void *prov, static int vidioc_enum_fmt_vid_out(struct file *file, void *priv,
struct v4l2_fmtdesc *f) struct v4l2_fmtdesc *f)
{ {
return vidioc_enum_fmt(f, false, true); return vidioc_enum_fmt(file, f, false, true);
} }
static int vidioc_enum_fmt_vid_out_mplane(struct file *file, void *prov, static int vidioc_enum_fmt_vid_out_mplane(struct file *file, void *priv,
struct v4l2_fmtdesc *f) struct v4l2_fmtdesc *f)
{ {
return vidioc_enum_fmt(f, true, true); return vidioc_enum_fmt(file, f, true, true);
} }
/* Get format */ /* Get format */
...@@ -384,11 +402,9 @@ static int vidioc_try_fmt(struct file *file, void *priv, struct v4l2_format *f) ...@@ -384,11 +402,9 @@ static int vidioc_try_fmt(struct file *file, void *priv, struct v4l2_format *f)
mfc_err("Unknown codec\n"); mfc_err("Unknown codec\n");
return -EINVAL; return -EINVAL;
} }
if (!IS_MFCV6_PLUS(dev)) { if ((dev->variant->version_bit & fmt->versions) == 0) {
if (fmt->fourcc == V4L2_PIX_FMT_VP8) { mfc_err("Unsupported format by this MFC version.\n");
mfc_err("Not supported format.\n"); return -EINVAL;
return -EINVAL;
}
} }
} else if (f->type == V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE) { } else if (f->type == V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE) {
fmt = find_format(f, MFC_FMT_RAW); fmt = find_format(f, MFC_FMT_RAW);
...@@ -396,13 +412,8 @@ static int vidioc_try_fmt(struct file *file, void *priv, struct v4l2_format *f) ...@@ -396,13 +412,8 @@ static int vidioc_try_fmt(struct file *file, void *priv, struct v4l2_format *f)
mfc_err("Unsupported format for destination.\n"); mfc_err("Unsupported format for destination.\n");
return -EINVAL; return -EINVAL;
} }
if (IS_MFCV6_PLUS(dev) && if ((dev->variant->version_bit & fmt->versions) == 0) {
(fmt->fourcc == V4L2_PIX_FMT_NV12MT)) { mfc_err("Unsupported format by this MFC version.\n");
mfc_err("Not supported format.\n");
return -EINVAL;
} else if (!IS_MFCV6_PLUS(dev) &&
(fmt->fourcc != V4L2_PIX_FMT_NV12MT)) {
mfc_err("Not supported format.\n");
return -EINVAL; return -EINVAL;
} }
} }
......
drivers/media/platform/s5p-mfc/s5p_mfc_enc.c
View file @ 9aa5f008
...@@ -42,6 +42,7 @@ static struct s5p_mfc_fmt formats[] = { ...@@ -42,6 +42,7 @@ static struct s5p_mfc_fmt formats[] = {
.codec_mode = S5P_MFC_CODEC_NONE, .codec_mode = S5P_MFC_CODEC_NONE,
.type = MFC_FMT_RAW, .type = MFC_FMT_RAW,
.num_planes = 2, .num_planes = 2,
.versions = MFC_V6_BIT | MFC_V7_BIT,
}, },
{ {
.name = "4:2:0 2 Planes 64x32 Tiles", .name = "4:2:0 2 Planes 64x32 Tiles",
...@@ -49,6 +50,7 @@ static struct s5p_mfc_fmt formats[] = { ...@@ -49,6 +50,7 @@ static struct s5p_mfc_fmt formats[] = {
.codec_mode = S5P_MFC_CODEC_NONE, .codec_mode = S5P_MFC_CODEC_NONE,
.type = MFC_FMT_RAW, .type = MFC_FMT_RAW,
.num_planes = 2, .num_planes = 2,
.versions = MFC_V5_BIT,
}, },
{ {
.name = "4:2:0 2 Planes Y/CbCr", .name = "4:2:0 2 Planes Y/CbCr",
...@@ -56,6 +58,7 @@ static struct s5p_mfc_fmt formats[] = { ...@@ -56,6 +58,7 @@ static struct s5p_mfc_fmt formats[] = {
.codec_mode = S5P_MFC_CODEC_NONE, .codec_mode = S5P_MFC_CODEC_NONE,
.type = MFC_FMT_RAW, .type = MFC_FMT_RAW,
.num_planes = 2, .num_planes = 2,
.versions = MFC_V5_BIT | MFC_V6_BIT | MFC_V7_BIT,
}, },
{ {
.name = "4:2:0 2 Planes Y/CrCb", .name = "4:2:0 2 Planes Y/CrCb",
...@@ -63,6 +66,7 @@ static struct s5p_mfc_fmt formats[] = { ...@@ -63,6 +66,7 @@ static struct s5p_mfc_fmt formats[] = {
.codec_mode = S5P_MFC_CODEC_NONE, .codec_mode = S5P_MFC_CODEC_NONE,
.type = MFC_FMT_RAW, .type = MFC_FMT_RAW,
.num_planes = 2, .num_planes = 2,
.versions = MFC_V5_BIT | MFC_V6_BIT | MFC_V7_BIT,
}, },
{ {
.name = "H264 Encoded Stream", .name = "H264 Encoded Stream",
...@@ -70,6 +74,7 @@ static struct s5p_mfc_fmt formats[] = { ...@@ -70,6 +74,7 @@ static struct s5p_mfc_fmt formats[] = {
.codec_mode = S5P_MFC_CODEC_H264_ENC, .codec_mode = S5P_MFC_CODEC_H264_ENC,
.type = MFC_FMT_ENC, .type = MFC_FMT_ENC,
.num_planes = 1, .num_planes = 1,
.versions = MFC_V5_BIT | MFC_V6_BIT | MFC_V7_BIT,
}, },
{ {
.name = "MPEG4 Encoded Stream", .name = "MPEG4 Encoded Stream",
...@@ -77,6 +82,7 @@ static struct s5p_mfc_fmt formats[] = { ...@@ -77,6 +82,7 @@ static struct s5p_mfc_fmt formats[] = {
.codec_mode = S5P_MFC_CODEC_MPEG4_ENC, .codec_mode = S5P_MFC_CODEC_MPEG4_ENC,
.type = MFC_FMT_ENC, .type = MFC_FMT_ENC,
.num_planes = 1, .num_planes = 1,
.versions = MFC_V5_BIT | MFC_V6_BIT | MFC_V7_BIT,
}, },
{ {
.name = "H263 Encoded Stream", .name = "H263 Encoded Stream",
...@@ -84,6 +90,7 @@ static struct s5p_mfc_fmt formats[] = { ...@@ -84,6 +90,7 @@ static struct s5p_mfc_fmt formats[] = {
.codec_mode = S5P_MFC_CODEC_H263_ENC, .codec_mode = S5P_MFC_CODEC_H263_ENC,
.type = MFC_FMT_ENC, .type = MFC_FMT_ENC,
.num_planes = 1, .num_planes = 1,
.versions = MFC_V5_BIT | MFC_V6_BIT | MFC_V7_BIT,
}, },
{ {
.name = "VP8 Encoded Stream", .name = "VP8 Encoded Stream",
...@@ -91,6 +98,7 @@ static struct s5p_mfc_fmt formats[] = { ...@@ -91,6 +98,7 @@ static struct s5p_mfc_fmt formats[] = {
.codec_mode = S5P_MFC_CODEC_VP8_ENC, .codec_mode = S5P_MFC_CODEC_VP8_ENC,
.type = MFC_FMT_ENC, .type = MFC_FMT_ENC,
.num_planes = 1, .num_planes = 1,
.versions = MFC_V7_BIT,
}, },
}; };
...@@ -940,8 +948,10 @@ static int vidioc_querycap(struct file *file, void *priv, ...@@ -940,8 +948,10 @@ static int vidioc_querycap(struct file *file, void *priv,
return 0; return 0;
} }
static int vidioc_enum_fmt(struct v4l2_fmtdesc *f, bool mplane, bool out) static int vidioc_enum_fmt(struct file *file, struct v4l2_fmtdesc *f,
bool mplane, bool out)
{ {
struct s5p_mfc_dev *dev = video_drvdata(file);
struct s5p_mfc_fmt *fmt; struct s5p_mfc_fmt *fmt;
int i, j = 0; int i, j = 0;
...@@ -954,6 +964,9 @@ static int vidioc_enum_fmt(struct v4l2_fmtdesc *f, bool mplane, bool out) ...@@ -954,6 +964,9 @@ static int vidioc_enum_fmt(struct v4l2_fmtdesc *f, bool mplane, bool out)
continue; continue;
else if (!out && formats[i].type != MFC_FMT_ENC) else if (!out && formats[i].type != MFC_FMT_ENC)
continue; continue;
else if ((dev->variant->version_bit & formats[i].versions) == 0)
continue;
if (j == f->index) { if (j == f->index) {
fmt = &formats[i]; fmt = &formats[i];
strlcpy(f->description, fmt->name, strlcpy(f->description, fmt->name,
...@@ -969,25 +982,25 @@ static int vidioc_enum_fmt(struct v4l2_fmtdesc *f, bool mplane, bool out) ...@@ -969,25 +982,25 @@ static int vidioc_enum_fmt(struct v4l2_fmtdesc *f, bool mplane, bool out)
static int vidioc_enum_fmt_vid_cap(struct file *file, void *pirv, static int vidioc_enum_fmt_vid_cap(struct file *file, void *pirv,
struct v4l2_fmtdesc *f) struct v4l2_fmtdesc *f)
{ {
return vidioc_enum_fmt(f, false, false); return vidioc_enum_fmt(file, f, false, false);
} }
static int vidioc_enum_fmt_vid_cap_mplane(struct file *file, void *pirv, static int vidioc_enum_fmt_vid_cap_mplane(struct file *file, void *pirv,
struct v4l2_fmtdesc *f) struct v4l2_fmtdesc *f)
{ {
return vidioc_enum_fmt(f, true, false); return vidioc_enum_fmt(file, f, true, false);
} }
static int vidioc_enum_fmt_vid_out(struct file *file, void *prov, static int vidioc_enum_fmt_vid_out(struct file *file, void *prov,
struct v4l2_fmtdesc *f) struct v4l2_fmtdesc *f)
{ {
return vidioc_enum_fmt(f, false, true); return vidioc_enum_fmt(file, f, false, true);
} }
static int vidioc_enum_fmt_vid_out_mplane(struct file *file, void *prov, static int vidioc_enum_fmt_vid_out_mplane(struct file *file, void *prov,
struct v4l2_fmtdesc *f) struct v4l2_fmtdesc *f)
{ {
return vidioc_enum_fmt(f, true, true); return vidioc_enum_fmt(file, f, true, true);
} }
static int vidioc_g_fmt(struct file *file, void *priv, struct v4l2_format *f) static int vidioc_g_fmt(struct file *file, void *priv, struct v4l2_format *f)
...@@ -1038,16 +1051,14 @@ static int vidioc_try_fmt(struct file *file, void *priv, struct v4l2_format *f) ...@@ -1038,16 +1051,14 @@ static int vidioc_try_fmt(struct file *file, void *priv, struct v4l2_format *f)
mfc_err("failed to try output format\n"); mfc_err("failed to try output format\n");
return -EINVAL; return -EINVAL;
} }
if (!IS_MFCV7_PLUS(dev) && (fmt->fourcc == V4L2_PIX_FMT_VP8)) {
mfc_err("VP8 is supported only in MFC v7\n");
return -EINVAL;
}
if (pix_fmt_mp->plane_fmt[0].sizeimage == 0) { if (pix_fmt_mp->plane_fmt[0].sizeimage == 0) {
mfc_err("must be set encoding output size\n"); mfc_err("must be set encoding output size\n");
return -EINVAL; return -EINVAL;
} }
if ((dev->variant->version_bit & fmt->versions) == 0) {
mfc_err("Unsupported format by this MFC version.\n");
return -EINVAL;
}
pix_fmt_mp->plane_fmt[0].bytesperline = pix_fmt_mp->plane_fmt[0].bytesperline =
pix_fmt_mp->plane_fmt[0].sizeimage; pix_fmt_mp->plane_fmt[0].sizeimage;
...@@ -1058,22 +1069,15 @@ static int vidioc_try_fmt(struct file *file, void *priv, struct v4l2_format *f) ...@@ -1058,22 +1069,15 @@ static int vidioc_try_fmt(struct file *file, void *priv, struct v4l2_format *f)
return -EINVAL; return -EINVAL;
} }
if (!IS_MFCV6_PLUS(dev)) {
if (fmt->fourcc == V4L2_PIX_FMT_NV12MT_16X16) {
mfc_err("Not supported format.\n");
return -EINVAL;
}
} else if (IS_MFCV6_PLUS(dev)) {
if (fmt->fourcc == V4L2_PIX_FMT_NV12MT) {
mfc_err("Not supported format.\n");
return -EINVAL;
}
}
if (fmt->num_planes != pix_fmt_mp->num_planes) { if (fmt->num_planes != pix_fmt_mp->num_planes) {
mfc_err("failed to try output format\n"); mfc_err("failed to try output format\n");
return -EINVAL; return -EINVAL;
} }
if ((dev->variant->version_bit & fmt->versions) == 0) {
mfc_err("Unsupported format by this MFC version.\n");
return -EINVAL;
}
v4l_bound_align_image(&pix_fmt_mp->width, 8, 1920, 1, v4l_bound_align_image(&pix_fmt_mp->width, 8, 1920, 1,
&pix_fmt_mp->height, 4, 1080, 1, 0); &pix_fmt_mp->height, 4, 1080, 1, 0);
} else { } else {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7