Re: [PATCH v8 3/5] media: stm32-dcmipp: STM32 DCMIPP camera interface driver

From: kernel test robot
Date: Wed Nov 22 2023 - 22:17:32 EST

Next message: syzbot: "Re: [syzbot] [bluetooth?] KASAN: slab-use-after-free Read in __lock_sock"
Previous message: kernel test robot: "Re: [PATCH v3 1/2] dt-bindings: soc: xilinx: Move xilinx.yaml from arm to soc"
In reply to: kernel test robot: "Re: [PATCH v8 3/5] media: stm32-dcmipp: STM32 DCMIPP camera interface driver"
Next in thread: Alain Volmat: "[PATCH v8 5/5] ARM: multi_v7_defconfig: enable STM32 DCMIPP media support"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

Hi Alain,

kernel test robot noticed the following build errors:

[auto build test ERROR on atorgue-stm32/stm32-next]
[also build test ERROR on media-tree/master linuxtv-media-stage/master linus/master v6.7-rc2 next-20231122]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url: https://github.com/intel-lab-lkp/linux/commits/Alain-Volmat/dt-bindings-media-add-bindings-for-stm32-dcmipp/20231122-154327
base: https://git.kernel.org/pub/scm/linux/kernel/git/atorgue/stm32.git stm32-next
patch link: https://lore.kernel.org/r/20231122073927.788810-4-alain.volmat%40foss.st.com
patch subject: [PATCH v8 3/5] media: stm32-dcmipp: STM32 DCMIPP camera interface driver
config: alpha-allyesconfig (https://download.01.org/0day-ci/archive/20231123/202311231015.CDjoMVIV-lkp@xxxxxxxxx/config)
compiler: alpha-linux-gcc (GCC) 13.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20231123/202311231015.CDjoMVIV-lkp@xxxxxxxxx/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@xxxxxxxxx>
| Closes: https://lore.kernel.org/oe-kbuild-all/202311231015.CDjoMVIV-lkp@xxxxxxxxx/

All errors (new ones prefixed by >>):

drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-parallel.c: In function 'dcmipp_par_init_cfg':
>> drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-parallel.c:154:22: error: implicit declaration of function 'v4l2_subdev_state_get_format'; did you mean 'v4l2_subdev_state_get_stream_format'? [-Werror=implicit-function-declaration]
154 | mf = v4l2_subdev_state_get_format(sd_state, i);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
| v4l2_subdev_state_get_stream_format
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-parallel.c:154:20: warning: assignment to 'struct v4l2_mbus_framefmt *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
154 | mf = v4l2_subdev_state_get_format(sd_state, i);
| ^
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-parallel.c: In function 'dcmipp_par_set_fmt':
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-parallel.c:236:12: warning: assignment to 'struct v4l2_mbus_framefmt *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
236 | mf = v4l2_subdev_state_get_format(sd_state, fmt->pad);
| ^
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-parallel.c:256:20: warning: assignment to 'struct v4l2_mbus_framefmt *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
256 | mf = v4l2_subdev_state_get_format(sd_state, 1);
| ^
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-parallel.c: In function 'dcmipp_par_configure':
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-parallel.c:311:18: warning: assignment to 'struct v4l2_mbus_framefmt *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
311 | sink_fmt = v4l2_subdev_state_get_format(state, 0);
| ^
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-parallel.c:312:17: warning: assignment to 'struct v4l2_mbus_framefmt *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
312 | src_fmt = v4l2_subdev_state_get_format(state, 1);
| ^
cc1: some warnings being treated as errors
--
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c: In function 'dcmipp_byteproc_init_cfg':
>> drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c:171:22: error: implicit declaration of function 'v4l2_subdev_state_get_format'; did you mean 'v4l2_subdev_state_get_stream_format'? [-Werror=implicit-function-declaration]
171 | mf = v4l2_subdev_state_get_format(sd_state, i);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
| v4l2_subdev_state_get_stream_format
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c:171:20: warning: assignment to 'struct v4l2_mbus_framefmt *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
171 | mf = v4l2_subdev_state_get_format(sd_state, i);
| ^
>> drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c:175:29: error: implicit declaration of function 'v4l2_subdev_state_get_compose'; did you mean 'v4l2_subdev_state_get_stream_compose'? [-Werror=implicit-function-declaration]
175 | r = v4l2_subdev_state_get_compose(sd_state, i);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| v4l2_subdev_state_get_stream_compose
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c:175:27: warning: assignment to 'struct v4l2_rect *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
175 | r = v4l2_subdev_state_get_compose(sd_state, i);
| ^
>> drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c:177:29: error: implicit declaration of function 'v4l2_subdev_state_get_crop'; did you mean 'v4l2_subdev_state_get_stream_crop'? [-Werror=implicit-function-declaration]
177 | r = v4l2_subdev_state_get_crop(sd_state, i);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
| v4l2_subdev_state_get_stream_crop
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c:177:27: warning: assignment to 'struct v4l2_rect *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
177 | r = v4l2_subdev_state_get_crop(sd_state, i);
| ^
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c: In function 'dcmipp_byteproc_enum_mbus_code':
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c:206:26: warning: assignment to 'struct v4l2_mbus_framefmt *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
206 | sink_fmt = v4l2_subdev_state_get_format(sd_state, 0);
| ^
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c: In function 'dcmipp_byteproc_enum_frame_size':
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c:230:25: warning: assignment to 'struct v4l2_rect *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
230 | compose = v4l2_subdev_state_get_compose(sd_state, 0);
| ^
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c: In function 'dcmipp_byteproc_set_fmt':
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c:249:12: warning: assignment to 'struct v4l2_mbus_framefmt *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
249 | mf = v4l2_subdev_state_get_format(sd_state, fmt->pad);
| ^
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c:251:14: warning: assignment to 'struct v4l2_rect *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
251 | crop = v4l2_subdev_state_get_crop(sd_state, 1);
| ^
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c:252:17: warning: assignment to 'struct v4l2_rect *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
252 | compose = v4l2_subdev_state_get_compose(sd_state, 0);
| ^
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c:255:31: error: invalid type argument of unary '*' (have 'int')
255 | fmt->format = *v4l2_subdev_state_get_format(sd_state, 0);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c:266:17: error: invalid type argument of unary '*' (have 'int')
266 | *v4l2_subdev_state_get_format(sd_state, 1) = fmt->format;
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c: In function 'dcmipp_byteproc_get_selection':
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c:297:18: warning: assignment to 'struct v4l2_mbus_framefmt *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
297 | sink_fmt = v4l2_subdev_state_get_format(sd_state, 0);
| ^
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c:298:14: warning: assignment to 'struct v4l2_rect *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
298 | crop = v4l2_subdev_state_get_crop(sd_state, 1);
| ^
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c:299:17: warning: assignment to 'struct v4l2_rect *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
299 | compose = v4l2_subdev_state_get_compose(sd_state, 0);
| ^
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c: In function 'dcmipp_byteproc_set_selection':
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c:349:14: warning: assignment to 'struct v4l2_rect *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
349 | crop = v4l2_subdev_state_get_crop(sd_state, 1);
| ^
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c:350:17: warning: assignment to 'struct v4l2_rect *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
350 | compose = v4l2_subdev_state_get_compose(sd_state, 0);
| ^
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c:357:20: warning: assignment to 'struct v4l2_mbus_framefmt *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
357 | mf = v4l2_subdev_state_get_format(sd_state, 1);
| ^
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c:365:20: warning: assignment to 'struct v4l2_mbus_framefmt *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
365 | mf = v4l2_subdev_state_get_format(sd_state, 0);
| ^
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c:370:20: warning: assignment to 'struct v4l2_mbus_framefmt *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
370 | mf = v4l2_subdev_state_get_format(sd_state, 1);
| ^
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c: In function 'dcmipp_byteproc_configure_scale_crop':
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c:406:18: warning: assignment to 'struct v4l2_mbus_framefmt *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
406 | sink_fmt = v4l2_subdev_state_get_format(state, 0);
| ^
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c:407:17: warning: assignment to 'struct v4l2_rect *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
407 | compose = v4l2_subdev_state_get_compose(state, 0);
| ^
drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-byteproc.c:408:14: warning: assignment to 'struct v4l2_rect *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
408 | crop = v4l2_subdev_state_get_crop(state, 1);
| ^
cc1: some warnings being treated as errors

vim +154 drivers/media/platform/st/stm32/stm32-dcmipp/dcmipp-parallel.c

145
146 static int dcmipp_par_init_cfg(struct v4l2_subdev *sd,
147 struct v4l2_subdev_state *sd_state)
148 {
149 unsigned int i;
150
151 for (i = 0; i < sd->entity.num_pads; i++) {
152 struct v4l2_mbus_framefmt *mf;
153
> 154 mf = v4l2_subdev_state_get_format(sd_state, i);
155 *mf = fmt_default;
156 }
157
158 return 0;
159 }
160

--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

Next message: syzbot: "Re: [syzbot] [bluetooth?] KASAN: slab-use-after-free Read in __lock_sock"
Previous message: kernel test robot: "Re: [PATCH v3 1/2] dt-bindings: soc: xilinx: Move xilinx.yaml from arm to soc"
In reply to: kernel test robot: "Re: [PATCH v8 3/5] media: stm32-dcmipp: STM32 DCMIPP camera interface driver"
Next in thread: Alain Volmat: "[PATCH v8 5/5] ARM: multi_v7_defconfig: enable STM32 DCMIPP media support"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]