* [PATCH] media: iris: drop struct iris_fmt
@ 2026-03-11 0:10 Dmitry Baryshkov
2026-03-13 12:52 ` Dikshita Agarwal
0 siblings, 1 reply; 2+ messages in thread
From: Dmitry Baryshkov @ 2026-03-11 0:10 UTC (permalink / raw)
To: Vikash Garodia, Dikshita Agarwal, Abhinav Kumar,
Bryan O'Donoghue, Mauro Carvalho Chehab
Cc: linux-media, linux-arm-msm, linux-kernel
The struct iris_fmt unites pixfmt with the plane type, however the type
from the struct is not actually used. Drop the struct completely and use
u32 pixfmt in all the callsites.
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
---
drivers/media/platform/qcom/iris/iris_instance.h | 5 --
.../platform/qcom/iris/iris_platform_common.h | 2 +-
.../media/platform/qcom/iris/iris_platform_gen1.c | 17 +---
.../media/platform/qcom/iris/iris_platform_gen2.c | 22 ++---
drivers/media/platform/qcom/iris/iris_vdec.c | 78 ++++++++----------
drivers/media/platform/qcom/iris/iris_venc.c | 96 +++++++++-------------
6 files changed, 80 insertions(+), 140 deletions(-)
diff --git a/drivers/media/platform/qcom/iris/iris_instance.h b/drivers/media/platform/qcom/iris/iris_instance.h
index 16965150f427..0eb256b0e82e 100644
--- a/drivers/media/platform/qcom/iris/iris_instance.h
+++ b/drivers/media/platform/qcom/iris/iris_instance.h
@@ -27,11 +27,6 @@ enum iris_fmt_type_cap {
IRIS_FMT_QC08C,
};
-struct iris_fmt {
- u32 pixfmt;
- u32 type;
-};
-
/**
* struct iris_inst - holds per video instance parameters
*
diff --git a/drivers/media/platform/qcom/iris/iris_platform_common.h b/drivers/media/platform/qcom/iris/iris_platform_common.h
index 5a489917580e..737ca4b1fe5c 100644
--- a/drivers/media/platform/qcom/iris/iris_platform_common.h
+++ b/drivers/media/platform/qcom/iris/iris_platform_common.h
@@ -237,7 +237,7 @@ struct iris_platform_data {
u64 dma_mask;
const char *fwname;
u32 pas_id;
- struct iris_fmt *inst_iris_fmts;
+ const u32 *inst_iris_fmts;
u32 inst_iris_fmts_size;
struct platform_inst_caps *inst_caps;
const struct platform_inst_fw_cap *inst_fw_caps_dec;
diff --git a/drivers/media/platform/qcom/iris/iris_platform_gen1.c b/drivers/media/platform/qcom/iris/iris_platform_gen1.c
index aa71f7f53ee3..c1e91e18d07f 100644
--- a/drivers/media/platform/qcom/iris/iris_platform_gen1.c
+++ b/drivers/media/platform/qcom/iris/iris_platform_gen1.c
@@ -20,19 +20,10 @@
#define BITRATE_PEAK_DEFAULT (BITRATE_DEFAULT * 2)
#define BITRATE_STEP 100
-static struct iris_fmt platform_fmts_sm8250_dec[] = {
- [IRIS_FMT_H264] = {
- .pixfmt = V4L2_PIX_FMT_H264,
- .type = V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE,
- },
- [IRIS_FMT_HEVC] = {
- .pixfmt = V4L2_PIX_FMT_HEVC,
- .type = V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE,
- },
- [IRIS_FMT_VP9] = {
- .pixfmt = V4L2_PIX_FMT_VP9,
- .type = V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE,
- },
+static u32 platform_fmts_sm8250_dec[] = {
+ [IRIS_FMT_H264] = V4L2_PIX_FMT_H264,
+ [IRIS_FMT_HEVC] = V4L2_PIX_FMT_HEVC,
+ [IRIS_FMT_VP9] = V4L2_PIX_FMT_VP9,
};
static struct platform_inst_fw_cap inst_fw_cap_sm8250_dec[] = {
diff --git a/drivers/media/platform/qcom/iris/iris_platform_gen2.c b/drivers/media/platform/qcom/iris/iris_platform_gen2.c
index 5da90d47f9c6..cd2725d3ff16 100644
--- a/drivers/media/platform/qcom/iris/iris_platform_gen2.c
+++ b/drivers/media/platform/qcom/iris/iris_platform_gen2.c
@@ -19,23 +19,11 @@
#define VIDEO_ARCH_LX 1
#define BITRATE_MAX 245000000
-static struct iris_fmt platform_fmts_sm8550_dec[] = {
- [IRIS_FMT_H264] = {
- .pixfmt = V4L2_PIX_FMT_H264,
- .type = V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE,
- },
- [IRIS_FMT_HEVC] = {
- .pixfmt = V4L2_PIX_FMT_HEVC,
- .type = V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE,
- },
- [IRIS_FMT_VP9] = {
- .pixfmt = V4L2_PIX_FMT_VP9,
- .type = V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE,
- },
- [IRIS_FMT_AV1] = {
- .pixfmt = V4L2_PIX_FMT_AV1,
- .type = V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE,
- },
+static const u32 platform_fmts_sm8550_dec[] = {
+ [IRIS_FMT_H264] = V4L2_PIX_FMT_H264,
+ [IRIS_FMT_HEVC] = V4L2_PIX_FMT_HEVC,
+ [IRIS_FMT_VP9] = V4L2_PIX_FMT_VP9,
+ [IRIS_FMT_AV1] = V4L2_PIX_FMT_AV1,
};
static const struct platform_inst_fw_cap inst_fw_cap_sm8550_dec[] = {
diff --git a/drivers/media/platform/qcom/iris/iris_vdec.c b/drivers/media/platform/qcom/iris/iris_vdec.c
index 719217399a30..2d3e2d72f221 100644
--- a/drivers/media/platform/qcom/iris/iris_vdec.c
+++ b/drivers/media/platform/qcom/iris/iris_vdec.c
@@ -67,23 +67,16 @@ void iris_vdec_inst_deinit(struct iris_inst *inst)
kfree(inst->fmt_src);
}
-static const struct iris_fmt iris_vdec_formats_cap[] = {
- [IRIS_FMT_NV12] = {
- .pixfmt = V4L2_PIX_FMT_NV12,
- .type = V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE,
- },
- [IRIS_FMT_QC08C] = {
- .pixfmt = V4L2_PIX_FMT_QC08C,
- .type = V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE,
- },
+static const u32 iris_vdec_formats_cap[] = {
+ [IRIS_FMT_NV12] = V4L2_PIX_FMT_NV12,
+ [IRIS_FMT_QC08C] = V4L2_PIX_FMT_QC08C,
};
-static const struct iris_fmt *
-find_format(struct iris_inst *inst, u32 pixfmt, u32 type)
+static bool check_format(struct iris_inst *inst, u32 pixfmt, u32 type)
{
- const struct iris_fmt *fmt = NULL;
- unsigned int size = 0;
- unsigned int i;
+ unsigned int size, i;
+ const u32 *fmt;
+
switch (type) {
case V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE:
fmt = inst->core->iris_platform_data->inst_iris_fmts;
@@ -94,25 +87,21 @@ find_format(struct iris_inst *inst, u32 pixfmt, u32 type)
size = ARRAY_SIZE(iris_vdec_formats_cap);
break;
default:
- return NULL;
+ return false;
}
for (i = 0; i < size; i++) {
- if (fmt[i].pixfmt == pixfmt)
- break;
+ if (fmt[i] == pixfmt)
+ return true;
}
- if (i == size || fmt[i].type != type)
- return NULL;
-
- return &fmt[i];
+ return false;
}
-static const struct iris_fmt *
-find_format_by_index(struct iris_inst *inst, u32 index, u32 type)
+static u32 find_format_by_index(struct iris_inst *inst, u32 index, u32 type)
{
- const struct iris_fmt *fmt = NULL;
- unsigned int size = 0;
+ const u32 *fmt;
+ unsigned int size;
switch (type) {
case V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE:
@@ -124,18 +113,18 @@ find_format_by_index(struct iris_inst *inst, u32 index, u32 type)
size = ARRAY_SIZE(iris_vdec_formats_cap);
break;
default:
- return NULL;
+ return 0;
}
- if (index >= size || fmt[index].type != type)
- return NULL;
+ if (index >= size)
+ return 0;
- return &fmt[index];
+ return fmt[index];
}
int iris_vdec_enum_fmt(struct iris_inst *inst, struct v4l2_fmtdesc *f)
{
- const struct iris_fmt *fmt;
+ u32 fmt;
switch (f->type) {
case V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE:
@@ -143,14 +132,14 @@ int iris_vdec_enum_fmt(struct iris_inst *inst, struct v4l2_fmtdesc *f)
if (!fmt)
return -EINVAL;
- f->pixelformat = fmt->pixfmt;
+ f->pixelformat = fmt;
f->flags = V4L2_FMT_FLAG_COMPRESSED | V4L2_FMT_FLAG_DYN_RESOLUTION;
break;
case V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE:
fmt = find_format_by_index(inst, f->index, f->type);
if (!fmt)
return -EINVAL;
- f->pixelformat = fmt->pixfmt;
+ f->pixelformat = fmt;
break;
default:
return -EINVAL;
@@ -163,15 +152,15 @@ int iris_vdec_try_fmt(struct iris_inst *inst, struct v4l2_format *f)
{
struct v4l2_pix_format_mplane *pixmp = &f->fmt.pix_mp;
struct v4l2_m2m_ctx *m2m_ctx = inst->m2m_ctx;
- const struct iris_fmt *fmt;
struct v4l2_format *f_inst;
struct vb2_queue *src_q;
+ bool supported;
memset(pixmp->reserved, 0, sizeof(pixmp->reserved));
- fmt = find_format(inst, pixmp->pixelformat, f->type);
+ supported = check_format(inst, pixmp->pixelformat, f->type);
switch (f->type) {
case V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE:
- if (!fmt) {
+ if (!supported) {
f_inst = inst->fmt_src;
f->fmt.pix_mp.width = f_inst->fmt.pix_mp.width;
f->fmt.pix_mp.height = f_inst->fmt.pix_mp.height;
@@ -179,7 +168,7 @@ int iris_vdec_try_fmt(struct iris_inst *inst, struct v4l2_format *f)
}
break;
case V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE:
- if (!fmt) {
+ if (!supported) {
f_inst = inst->fmt_dst;
f->fmt.pix_mp.pixelformat = f_inst->fmt.pix_mp.pixelformat;
f->fmt.pix_mp.width = f_inst->fmt.pix_mp.width;
@@ -228,7 +217,7 @@ int iris_vdec_s_fmt(struct iris_inst *inst, struct v4l2_format *f)
switch (f->type) {
case V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE:
- if (!(find_format(inst, f->fmt.pix_mp.pixelformat, f->type)))
+ if (!check_format(inst, f->fmt.pix_mp.pixelformat, f->type))
return -EINVAL;
fmt = inst->fmt_src;
@@ -266,7 +255,7 @@ int iris_vdec_s_fmt(struct iris_inst *inst, struct v4l2_format *f)
inst->crop.height = f->fmt.pix_mp.height;
break;
case V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE:
- if (!(find_format(inst, f->fmt.pix_mp.pixelformat, f->type)))
+ if (!check_format(inst, f->fmt.pix_mp.pixelformat, f->type))
return -EINVAL;
fmt = inst->fmt_dst;
@@ -295,16 +284,13 @@ int iris_vdec_s_fmt(struct iris_inst *inst, struct v4l2_format *f)
int iris_vdec_validate_format(struct iris_inst *inst, u32 pixelformat)
{
- const struct iris_fmt *fmt = NULL;
+ bool supported;
- fmt = find_format(inst, pixelformat, V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE);
- if (!fmt) {
- fmt = find_format(inst, pixelformat, V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE);
- if (!fmt)
- return -EINVAL;
- }
+ supported = check_format(inst, pixelformat, V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE);
+ if (!supported)
+ supported = check_format(inst, pixelformat, V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE);
- return 0;
+ return supported ? 0 : -EINVAL;
}
int iris_vdec_subscribe_event(struct iris_inst *inst, const struct v4l2_event_subscription *sub)
diff --git a/drivers/media/platform/qcom/iris/iris_venc.c b/drivers/media/platform/qcom/iris/iris_venc.c
index aa27b22704eb..0cba49df512d 100644
--- a/drivers/media/platform/qcom/iris/iris_venc.c
+++ b/drivers/media/platform/qcom/iris/iris_venc.c
@@ -85,34 +85,21 @@ void iris_venc_inst_deinit(struct iris_inst *inst)
kfree(inst->fmt_src);
}
-static const struct iris_fmt iris_venc_formats_cap[] = {
- [IRIS_FMT_H264] = {
- .pixfmt = V4L2_PIX_FMT_H264,
- .type = V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE,
- },
- [IRIS_FMT_HEVC] = {
- .pixfmt = V4L2_PIX_FMT_HEVC,
- .type = V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE,
- },
+static const u32 iris_venc_formats_cap[] = {
+ [IRIS_FMT_H264] = V4L2_PIX_FMT_H264,
+ [IRIS_FMT_HEVC] = V4L2_PIX_FMT_HEVC,
};
-static const struct iris_fmt iris_venc_formats_out[] = {
- [IRIS_FMT_NV12] = {
- .pixfmt = V4L2_PIX_FMT_NV12,
- .type = V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE,
- },
- [IRIS_FMT_QC08C] = {
- .pixfmt = V4L2_PIX_FMT_QC08C,
- .type = V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE,
- },
+static const u32 iris_venc_formats_out[] = {
+ [IRIS_FMT_NV12] = V4L2_PIX_FMT_NV12,
+ [IRIS_FMT_QC08C] = V4L2_PIX_FMT_QC08C,
};
-static const struct iris_fmt *
-find_format(struct iris_inst *inst, u32 pixfmt, u32 type)
+static bool check_format(struct iris_inst *inst, u32 pixfmt, u32 type)
{
- const struct iris_fmt *fmt = NULL;
- unsigned int size = 0;
- unsigned int i;
+ unsigned int size, i;
+ const u32 *fmt;
+
switch (type) {
case V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE:
fmt = iris_venc_formats_out;
@@ -123,25 +110,21 @@ find_format(struct iris_inst *inst, u32 pixfmt, u32 type)
size = ARRAY_SIZE(iris_venc_formats_cap);
break;
default:
- return NULL;
+ return false;
}
for (i = 0; i < size; i++) {
- if (fmt[i].pixfmt == pixfmt)
- break;
+ if (fmt[i] == pixfmt)
+ return true;
}
- if (i == size || fmt[i].type != type)
- return NULL;
-
- return &fmt[i];
+ return false;
}
-static const struct iris_fmt *
-find_format_by_index(struct iris_inst *inst, u32 index, u32 type)
+static u32 find_format_by_index(struct iris_inst *inst, u32 index, u32 type)
{
- const struct iris_fmt *fmt = NULL;
- unsigned int size = 0;
+ unsigned int size;
+ const u32 *fmt;
switch (type) {
case V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE:
@@ -153,18 +136,18 @@ find_format_by_index(struct iris_inst *inst, u32 index, u32 type)
size = ARRAY_SIZE(iris_venc_formats_cap);
break;
default:
- return NULL;
+ return 0;
}
- if (index >= size || fmt[index].type != type)
- return NULL;
+ if (index >= size)
+ return 0;
- return &fmt[index];
+ return fmt[index];
}
int iris_venc_enum_fmt(struct iris_inst *inst, struct v4l2_fmtdesc *f)
{
- const struct iris_fmt *fmt;
+ u32 fmt;
switch (f->type) {
case V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE:
@@ -172,14 +155,14 @@ int iris_venc_enum_fmt(struct iris_inst *inst, struct v4l2_fmtdesc *f)
if (!fmt)
return -EINVAL;
- f->pixelformat = fmt->pixfmt;
+ f->pixelformat = fmt;
break;
case V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE:
fmt = find_format_by_index(inst, f->index, f->type);
if (!fmt)
return -EINVAL;
- f->pixelformat = fmt->pixfmt;
+ f->pixelformat = fmt;
f->flags = V4L2_FMT_FLAG_COMPRESSED | V4L2_FMT_FLAG_ENC_CAP_FRAME_INTERVAL;
break;
default:
@@ -192,14 +175,14 @@ int iris_venc_enum_fmt(struct iris_inst *inst, struct v4l2_fmtdesc *f)
int iris_venc_try_fmt(struct iris_inst *inst, struct v4l2_format *f)
{
struct v4l2_pix_format_mplane *pixmp = &f->fmt.pix_mp;
- const struct iris_fmt *fmt;
struct v4l2_format *f_inst;
+ bool supported;
memset(pixmp->reserved, 0, sizeof(pixmp->reserved));
- fmt = find_format(inst, pixmp->pixelformat, f->type);
+ supported = check_format(inst, pixmp->pixelformat, f->type);
switch (f->type) {
case V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE:
- if (!fmt) {
+ if (!supported) {
f_inst = inst->fmt_src;
f->fmt.pix_mp.width = f_inst->fmt.pix_mp.width;
f->fmt.pix_mp.height = f_inst->fmt.pix_mp.height;
@@ -207,7 +190,7 @@ int iris_venc_try_fmt(struct iris_inst *inst, struct v4l2_format *f)
}
break;
case V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE:
- if (!fmt) {
+ if (!supported) {
f_inst = inst->fmt_dst;
f->fmt.pix_mp.width = f_inst->fmt.pix_mp.width;
f->fmt.pix_mp.height = f_inst->fmt.pix_mp.height;
@@ -228,17 +211,17 @@ int iris_venc_try_fmt(struct iris_inst *inst, struct v4l2_format *f)
static int iris_venc_s_fmt_output(struct iris_inst *inst, struct v4l2_format *f)
{
- const struct iris_fmt *venc_fmt;
struct v4l2_format *fmt;
u32 codec_align;
+ bool supported;
iris_venc_try_fmt(inst, f);
- venc_fmt = find_format(inst, f->fmt.pix_mp.pixelformat, f->type);
- if (!venc_fmt)
+ supported = check_format(inst, f->fmt.pix_mp.pixelformat, f->type);
+ if (!supported)
return -EINVAL;
- codec_align = venc_fmt->pixfmt == V4L2_PIX_FMT_HEVC ? 32 : 16;
+ codec_align = (f->fmt.pix_mp.pixelformat == V4L2_PIX_FMT_HEVC) ? 32 : 16;
fmt = inst->fmt_dst;
fmt->type = V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE;
@@ -281,7 +264,7 @@ static int iris_venc_s_fmt_input(struct iris_inst *inst, struct v4l2_format *f)
iris_venc_try_fmt(inst, f);
- if (!(find_format(inst, f->fmt.pix_mp.pixelformat, f->type)))
+ if (!check_format(inst, f->fmt.pix_mp.pixelformat, f->type))
return -EINVAL;
fmt = inst->fmt_src;
@@ -350,16 +333,13 @@ int iris_venc_s_fmt(struct iris_inst *inst, struct v4l2_format *f)
int iris_venc_validate_format(struct iris_inst *inst, u32 pixelformat)
{
- const struct iris_fmt *fmt = NULL;
+ bool supported;
- fmt = find_format(inst, pixelformat, V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE);
- if (!fmt) {
- fmt = find_format(inst, pixelformat, V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE);
- if (!fmt)
- return -EINVAL;
- }
+ supported = check_format(inst, pixelformat, V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE);
+ if (!supported)
+ supported = check_format(inst, pixelformat, V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE);
- return 0;
+ return supported ? 0 : -EINVAL;
}
int iris_venc_subscribe_event(struct iris_inst *inst,
---
base-commit: 69f994912b677367e59934a2c34f365cf42d517f
change-id: 20260311-iris-remote-fmts-53336c2b89a0
Best regards,
--
With best wishes
Dmitry
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] media: iris: drop struct iris_fmt
2026-03-11 0:10 [PATCH] media: iris: drop struct iris_fmt Dmitry Baryshkov
@ 2026-03-13 12:52 ` Dikshita Agarwal
0 siblings, 0 replies; 2+ messages in thread
From: Dikshita Agarwal @ 2026-03-13 12:52 UTC (permalink / raw)
To: Dmitry Baryshkov, Vikash Garodia, Abhinav Kumar,
Bryan O'Donoghue, Mauro Carvalho Chehab
Cc: linux-media, linux-arm-msm, linux-kernel
On 3/11/2026 5:40 AM, Dmitry Baryshkov wrote:
> The struct iris_fmt unites pixfmt with the plane type, however the type
> from the struct is not actually used. Drop the struct completely and use
> u32 pixfmt in all the callsites.
>
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
> ---
> drivers/media/platform/qcom/iris/iris_instance.h | 5 --
> .../platform/qcom/iris/iris_platform_common.h | 2 +-
> .../media/platform/qcom/iris/iris_platform_gen1.c | 17 +---
> .../media/platform/qcom/iris/iris_platform_gen2.c | 22 ++---
> drivers/media/platform/qcom/iris/iris_vdec.c | 78 ++++++++----------
> drivers/media/platform/qcom/iris/iris_venc.c | 96 +++++++++-------------
> 6 files changed, 80 insertions(+), 140 deletions(-)
>
> diff --git a/drivers/media/platform/qcom/iris/iris_instance.h b/drivers/media/platform/qcom/iris/iris_instance.h
> index 16965150f427..0eb256b0e82e 100644
> --- a/drivers/media/platform/qcom/iris/iris_instance.h
> +++ b/drivers/media/platform/qcom/iris/iris_instance.h
> @@ -27,11 +27,6 @@ enum iris_fmt_type_cap {
> IRIS_FMT_QC08C,
> };
>
> -struct iris_fmt {
> - u32 pixfmt;
> - u32 type;
> -};
> -
> /**
> * struct iris_inst - holds per video instance parameters
> *
> diff --git a/drivers/media/platform/qcom/iris/iris_platform_common.h b/drivers/media/platform/qcom/iris/iris_platform_common.h
> index 5a489917580e..737ca4b1fe5c 100644
> --- a/drivers/media/platform/qcom/iris/iris_platform_common.h
> +++ b/drivers/media/platform/qcom/iris/iris_platform_common.h
> @@ -237,7 +237,7 @@ struct iris_platform_data {
> u64 dma_mask;
> const char *fwname;
> u32 pas_id;
> - struct iris_fmt *inst_iris_fmts;
> + const u32 *inst_iris_fmts;
> u32 inst_iris_fmts_size;
> struct platform_inst_caps *inst_caps;
> const struct platform_inst_fw_cap *inst_fw_caps_dec;
> diff --git a/drivers/media/platform/qcom/iris/iris_platform_gen1.c b/drivers/media/platform/qcom/iris/iris_platform_gen1.c
> index aa71f7f53ee3..c1e91e18d07f 100644
> --- a/drivers/media/platform/qcom/iris/iris_platform_gen1.c
> +++ b/drivers/media/platform/qcom/iris/iris_platform_gen1.c
> @@ -20,19 +20,10 @@
> #define BITRATE_PEAK_DEFAULT (BITRATE_DEFAULT * 2)
> #define BITRATE_STEP 100
>
> -static struct iris_fmt platform_fmts_sm8250_dec[] = {
> - [IRIS_FMT_H264] = {
> - .pixfmt = V4L2_PIX_FMT_H264,
> - .type = V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE,
> - },
> - [IRIS_FMT_HEVC] = {
> - .pixfmt = V4L2_PIX_FMT_HEVC,
> - .type = V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE,
> - },
> - [IRIS_FMT_VP9] = {
> - .pixfmt = V4L2_PIX_FMT_VP9,
> - .type = V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE,
> - },
> +static u32 platform_fmts_sm8250_dec[] = {
static const u32 platform_fmts_sm8250_dec ?
> + [IRIS_FMT_H264] = V4L2_PIX_FMT_H264,
> + [IRIS_FMT_HEVC] = V4L2_PIX_FMT_HEVC,
> + [IRIS_FMT_VP9] = V4L2_PIX_FMT_VP9,
> };
>
> static struct platform_inst_fw_cap inst_fw_cap_sm8250_dec[] = {
> diff --git a/drivers/media/platform/qcom/iris/iris_platform_gen2.c b/drivers/media/platform/qcom/iris/iris_platform_gen2.c
> index 5da90d47f9c6..cd2725d3ff16 100644
> --- a/drivers/media/platform/qcom/iris/iris_platform_gen2.c
> +++ b/drivers/media/platform/qcom/iris/iris_platform_gen2.c
> @@ -19,23 +19,11 @@
> #define VIDEO_ARCH_LX 1
> #define BITRATE_MAX 245000000
>
> -static struct iris_fmt platform_fmts_sm8550_dec[] = {
> - [IRIS_FMT_H264] = {
> - .pixfmt = V4L2_PIX_FMT_H264,
> - .type = V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE,
> - },
> - [IRIS_FMT_HEVC] = {
> - .pixfmt = V4L2_PIX_FMT_HEVC,
> - .type = V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE,
> - },
> - [IRIS_FMT_VP9] = {
> - .pixfmt = V4L2_PIX_FMT_VP9,
> - .type = V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE,
> - },
> - [IRIS_FMT_AV1] = {
> - .pixfmt = V4L2_PIX_FMT_AV1,
> - .type = V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE,
> - },
> +static const u32 platform_fmts_sm8550_dec[] = {
> + [IRIS_FMT_H264] = V4L2_PIX_FMT_H264,
> + [IRIS_FMT_HEVC] = V4L2_PIX_FMT_HEVC,
> + [IRIS_FMT_VP9] = V4L2_PIX_FMT_VP9,
> + [IRIS_FMT_AV1] = V4L2_PIX_FMT_AV1,
> };
>
> static const struct platform_inst_fw_cap inst_fw_cap_sm8550_dec[] = {
> diff --git a/drivers/media/platform/qcom/iris/iris_vdec.c b/drivers/media/platform/qcom/iris/iris_vdec.c
> index 719217399a30..2d3e2d72f221 100644
> --- a/drivers/media/platform/qcom/iris/iris_vdec.c
> +++ b/drivers/media/platform/qcom/iris/iris_vdec.c
> @@ -67,23 +67,16 @@ void iris_vdec_inst_deinit(struct iris_inst *inst)
> kfree(inst->fmt_src);
> }
>
> -static const struct iris_fmt iris_vdec_formats_cap[] = {
> - [IRIS_FMT_NV12] = {
> - .pixfmt = V4L2_PIX_FMT_NV12,
> - .type = V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE,
> - },
> - [IRIS_FMT_QC08C] = {
> - .pixfmt = V4L2_PIX_FMT_QC08C,
> - .type = V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE,
> - },
> +static const u32 iris_vdec_formats_cap[] = {
> + [IRIS_FMT_NV12] = V4L2_PIX_FMT_NV12,
> + [IRIS_FMT_QC08C] = V4L2_PIX_FMT_QC08C,
> };
>
> -static const struct iris_fmt *
> -find_format(struct iris_inst *inst, u32 pixfmt, u32 type)
> +static bool check_format(struct iris_inst *inst, u32 pixfmt, u32 type)
> {
> - const struct iris_fmt *fmt = NULL;
> - unsigned int size = 0;
> - unsigned int i;
> + unsigned int size, i;
> + const u32 *fmt;
> +
> switch (type) {
> case V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE:
> fmt = inst->core->iris_platform_data->inst_iris_fmts;
> @@ -94,25 +87,21 @@ find_format(struct iris_inst *inst, u32 pixfmt, u32 type)
> size = ARRAY_SIZE(iris_vdec_formats_cap);
> break;
> default:
> - return NULL;
> + return false;
> }
>
> for (i = 0; i < size; i++) {
> - if (fmt[i].pixfmt == pixfmt)
> - break;
> + if (fmt[i] == pixfmt)
> + return true;
> }
>
> - if (i == size || fmt[i].type != type)
> - return NULL;
> -
> - return &fmt[i];
> + return false;
> }
>
> -static const struct iris_fmt *
> -find_format_by_index(struct iris_inst *inst, u32 index, u32 type)
> +static u32 find_format_by_index(struct iris_inst *inst, u32 index, u32 type)
> {
> - const struct iris_fmt *fmt = NULL;
> - unsigned int size = 0;
> + const u32 *fmt;
> + unsigned int size;
pls follow reverse Christmas tree order
with that.
Reviewed-by: Dikshita Agarwal <dikshita.agarwal@oss.qualcomm.com>
Thanks,
Dikshita
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2026-03-13 12:53 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-03-11 0:10 [PATCH] media: iris: drop struct iris_fmt Dmitry Baryshkov
2026-03-13 12:52 ` Dikshita Agarwal
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox