From: kernel test robot <lkp@intel.com>
To: Dikshita Agarwal <dikshita.agarwal@oss.qualcomm.com>,
Vikash Garodia <vikash.garodia@oss.qualcomm.com>,
Abhinav Kumar <abhinav.kumar@linux.dev>,
Bryan O'Donoghue <bod@kernel.org>,
Mauro Carvalho Chehab <mchehab@kernel.org>,
Hans Verkuil <hverkuil@xs4all.nl>,
Stefan Schmidt <stefan.schmidt@linaro.org>
Cc: llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev,
linux-media@vger.kernel.org, linux-arm-msm@vger.kernel.org,
linux-kernel@vger.kernel.org, Val Packett <val@packett.cool>,
Dikshita Agarwal <dikshita.agarwal@oss.qualcomm.com>
Subject: Re: [PATCH v2] media: iris: Refine internal buffer reconfiguration logic for resolution change
Date: Wed, 5 Nov 2025 09:08:57 +0800 [thread overview]
Message-ID: <202511050842.FTUh6ym8-lkp@intel.com> (raw)
In-Reply-To: <20251104-iris-seek-fix-v2-1-c9dace39b43d@oss.qualcomm.com>
Hi Dikshita,
kernel test robot noticed the following build errors:
[auto build test ERROR on 163917839c0eea3bdfe3620f27f617a55fd76302]
url: https://github.com/intel-lab-lkp/linux/commits/Dikshita-Agarwal/media-iris-Refine-internal-buffer-reconfiguration-logic-for-resolution-change/20251104-131307
base: 163917839c0eea3bdfe3620f27f617a55fd76302
patch link: https://lore.kernel.org/r/20251104-iris-seek-fix-v2-1-c9dace39b43d%40oss.qualcomm.com
patch subject: [PATCH v2] media: iris: Refine internal buffer reconfiguration logic for resolution change
config: loongarch-allmodconfig (https://download.01.org/0day-ci/archive/20251105/202511050842.FTUh6ym8-lkp@intel.com/config)
compiler: clang version 19.1.7 (https://github.com/llvm/llvm-project cd708029e0b2869e80abe31ddb175f7c35361f90)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20251105/202511050842.FTUh6ym8-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202511050842.FTUh6ym8-lkp@intel.com/
All error/warnings (new ones prefixed by >>):
>> drivers/media/platform/qcom/iris/iris_common.c:99:2: error: use of undeclared identifier 'first_ipsc'
99 | first_ipsc = inst->sub_state & IRIS_INST_SUB_FIRST_IPSC;
| ^
>> drivers/media/platform/qcom/iris/iris_common.c:101:2: error: use of undeclared identifier 'drain_active'
101 | drain_active = inst->sub_state & IRIS_INST_SUB_DRAIN &&
| ^
>> drivers/media/platform/qcom/iris/iris_common.c:104:2: error: use of undeclared identifier 'drc_active'
104 | drc_active = inst->sub_state & IRIS_INST_SUB_DRC &&
| ^
drivers/media/platform/qcom/iris/iris_common.c:107:6: error: use of undeclared identifier 'drc_active'
107 | if (drc_active)
| ^
drivers/media/platform/qcom/iris/iris_common.c:109:11: error: use of undeclared identifier 'drain_active'; did you mean 'swait_active'?
109 | else if (drain_active)
| ^~~~~~~~~~~~
| swait_active
include/linux/swait.h:121:19: note: 'swait_active' declared here
121 | static inline int swait_active(struct swait_queue_head *wq)
| ^
drivers/media/platform/qcom/iris/iris_common.c:113:6: error: use of undeclared identifier 'first_ipsc'
113 | if (first_ipsc || drc_active) {
| ^
drivers/media/platform/qcom/iris/iris_common.c:113:20: error: use of undeclared identifier 'drc_active'
113 | if (first_ipsc || drc_active) {
| ^
drivers/media/platform/qcom/iris/iris_common.c:127:8: error: use of undeclared identifier 'drain_active'; did you mean 'swait_active'?
127 | if (!drain_active)
| ^~~~~~~~~~~~
| swait_active
include/linux/swait.h:121:19: note: 'swait_active' declared here
121 | static inline int swait_active(struct swait_queue_head *wq)
| ^
>> drivers/media/platform/qcom/iris/iris_common.c:127:8: warning: address of function 'swait_active' will always evaluate to 'true' [-Wpointer-bool-conversion]
127 | if (!drain_active)
| ~^~~~~~~~~~~~
drivers/media/platform/qcom/iris/iris_common.c:127:8: note: prefix with the address-of operator to silence this warning
127 | if (!drain_active)
| ^
| &
1 warning and 8 errors generated.
vim +/first_ipsc +99 drivers/media/platform/qcom/iris/iris_common.c
90
91 int iris_process_streamon_output(struct iris_inst *inst)
92 {
93 const struct iris_hfi_command_ops *hfi_ops = inst->core->hfi_ops;
94 enum iris_inst_sub_state clear_sub_state = 0;
95 int ret = 0;
96
97 iris_scale_power(inst);
98
> 99 first_ipsc = inst->sub_state & IRIS_INST_SUB_FIRST_IPSC;
100
> 101 drain_active = inst->sub_state & IRIS_INST_SUB_DRAIN &&
102 inst->sub_state & IRIS_INST_SUB_DRAIN_LAST;
103
> 104 drc_active = inst->sub_state & IRIS_INST_SUB_DRC &&
105 inst->sub_state & IRIS_INST_SUB_DRC_LAST;
106
107 if (drc_active)
108 clear_sub_state = IRIS_INST_SUB_DRC | IRIS_INST_SUB_DRC_LAST;
109 else if (drain_active)
110 clear_sub_state = IRIS_INST_SUB_DRAIN | IRIS_INST_SUB_DRAIN_LAST;
111
112 /* Input internal buffer reconfiguration required in case of resolution change */
113 if (first_ipsc || drc_active) {
114 ret = iris_alloc_and_queue_input_int_bufs(inst);
115 if (ret)
116 return ret;
117 ret = iris_set_stage(inst, STAGE);
118 if (ret)
119 return ret;
120 ret = iris_set_pipe(inst, PIPE);
121 if (ret)
122 return ret;
123 }
124
125 if (inst->state == IRIS_INST_INPUT_STREAMING &&
126 inst->sub_state & IRIS_INST_SUB_INPUT_PAUSE) {
> 127 if (!drain_active)
128 ret = hfi_ops->session_resume_drc(inst,
129 V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE);
130 else if (hfi_ops->session_resume_drain)
131 ret = hfi_ops->session_resume_drain(inst,
132 V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE);
133 if (ret)
134 return ret;
135 clear_sub_state |= IRIS_INST_SUB_INPUT_PAUSE;
136 }
137
138 if (inst->sub_state & IRIS_INST_SUB_FIRST_IPSC)
139 clear_sub_state |= IRIS_INST_SUB_FIRST_IPSC;
140
141 ret = hfi_ops->session_start(inst, V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE);
142 if (ret)
143 return ret;
144
145 if (inst->sub_state & IRIS_INST_SUB_OUTPUT_PAUSE)
146 clear_sub_state |= IRIS_INST_SUB_OUTPUT_PAUSE;
147
148 ret = iris_inst_state_change_streamon(inst, V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE);
149 if (ret)
150 return ret;
151
152 inst->last_buffer_dequeued = false;
153
154 return iris_inst_change_sub_state(inst, clear_sub_state, 0);
155 }
156
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
parent reply other threads:[~2025-11-05 1:09 UTC|newest]
Thread overview: expand[flat|nested] mbox.gz Atom feed
[parent not found: <20251104-iris-seek-fix-v2-1-c9dace39b43d@oss.qualcomm.com>]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202511050842.FTUh6ym8-lkp@intel.com \
--to=lkp@intel.com \
--cc=abhinav.kumar@linux.dev \
--cc=bod@kernel.org \
--cc=dikshita.agarwal@oss.qualcomm.com \
--cc=hverkuil@xs4all.nl \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=llvm@lists.linux.dev \
--cc=mchehab@kernel.org \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=stefan.schmidt@linaro.org \
--cc=val@packett.cool \
--cc=vikash.garodia@oss.qualcomm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox