[sailus-media-tree:metadata 80/122] drivers/media/i2c/ccs/ccs-core.c:2486:24: error: too many arguments to function 'ccs_get_format'; expected 3, have 4

[sailus-media-tree:metadata 80/122] drivers/media/i2c/ccs/ccs-core.c:2486:24: error: too many arguments to function 'ccs_get_format'; expected 3, have 4

[kernel test robot]

tree:   git://linuxtv.org/sailus/media_tree.git metadata
head:   489ab6e79288be8d219b68525a13699fde0248ae
commit: f3d14633599ac2e30aeb1e1af17db73b46682931 [80/122] media: ccs: Add support for embedded data stream
config: openrisc-allmodconfig (https://download.01.org/0day-ci/archive/20260429/202604291554.jge9fFXX-lkp@intel.com/config)
compiler: or1k-linux-gcc (GCC) 15.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260429/202604291554.jge9fFXX-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/202604291554.jge9fFXX-lkp@intel.com/

All errors (new ones prefixed by >>):

   drivers/media/i2c/ccs/ccs-core.c: In function 'ccs_set_format':
   drivers/media/i2c/ccs/ccs-core.c:2486:47: error: passing argument 2 of 'ccs_get_format' from incompatible pointer type [-Wincompatible-pointer-types]
    2486 |                 return ccs_get_format(subdev, ci, sd_state, fmt);
         |                                               ^~
         |                                               |
         |                                               const struct v4l2_subdev_client_info *
   drivers/media/i2c/ccs/ccs-core.c:2299:53: note: expected 'struct v4l2_subdev_state *' but argument is of type 'const struct v4l2_subdev_client_info *'
    2299 |                           struct v4l2_subdev_state *sd_state,
         |                           ~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~
   drivers/media/i2c/ccs/ccs-core.c:2486:51: error: passing argument 3 of 'ccs_get_format' from incompatible pointer type [-Wincompatible-pointer-types]
    2486 |                 return ccs_get_format(subdev, ci, sd_state, fmt);
         |                                                   ^~~~~~~~
         |                                                   |
         |                                                   struct v4l2_subdev_state *
   drivers/media/i2c/ccs/ccs-core.c:2300:54: note: expected 'struct v4l2_subdev_format *' but argument is of type 'struct v4l2_subdev_state *'
    2300 |                           struct v4l2_subdev_format *fmt)
         |                           ~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~
>> drivers/media/i2c/ccs/ccs-core.c:2486:24: error: too many arguments to function 'ccs_get_format'; expected 3, have 4
    2486 |                 return ccs_get_format(subdev, ci, sd_state, fmt);
         |                        ^~~~~~~~~~~~~~                       ~~~
   drivers/media/i2c/ccs/ccs-core.c:2298:12: note: declared here
    2298 | static int ccs_get_format(struct v4l2_subdev *subdev,
         |            ^~~~~~~~~~~~~~


vim +/ccs_get_format +2486 drivers/media/i2c/ccs/ccs-core.c

  2475	
  2476	static int ccs_set_format(struct v4l2_subdev *subdev,
  2477				  const struct v4l2_subdev_client_info *ci,
  2478				  struct v4l2_subdev_state *sd_state,
  2479				  struct v4l2_subdev_format *fmt)
  2480	{
  2481		struct ccs_sensor *sensor = to_ccs_sensor(subdev);
  2482		struct ccs_subdev *ssd = to_ccs_subdev(subdev);
  2483		struct v4l2_rect *crop;
  2484	
  2485		if (ssd == sensor->src && fmt->pad == CCS_PAD_META)
> 2486			return ccs_get_format(subdev, ci, sd_state, fmt);
  2487	
  2488		if (ssd == sensor->src && fmt->stream == CCS_STREAM_META) {
  2489			ccs_set_format_meta(subdev, sd_state, &fmt->format);
  2490	
  2491			return 0;
  2492		}
  2493	
  2494		if (fmt->pad == ssd->source_pad) {
  2495			int rval;
  2496	
  2497			rval = ccs_set_format_source(subdev, NULL, sd_state, fmt);
  2498			if (ccs_embedded_data_lines(sensor) && ssd == sensor->src)
  2499				ccs_set_format_meta(subdev, sd_state, NULL);
  2500	
  2501			return rval;
  2502		}
  2503	
  2504		/* Sink pad. Width and height are changeable here. */
  2505		fmt->format.code = ccs_get_mbus_code(subdev, fmt->pad);
  2506	
  2507		fmt->format.width &= ~1;
  2508		fmt->format.height &= ~1;
  2509		fmt->format.field = V4L2_FIELD_NONE;
  2510	
  2511		fmt->format.width =
  2512			clamp(fmt->format.width,
  2513			      CCS_LIM(sensor, MIN_X_OUTPUT_SIZE),
  2514			      CCS_LIM(sensor, MAX_X_OUTPUT_SIZE));
  2515		fmt->format.height =
  2516			clamp(fmt->format.height,
  2517			      CCS_LIM(sensor, MIN_Y_OUTPUT_SIZE),
  2518			      CCS_LIM(sensor, MAX_Y_OUTPUT_SIZE));
  2519	
  2520		crop = v4l2_subdev_state_get_crop(sd_state, ssd->sink_pad,
  2521						  CCS_STREAM_PIXEL);
  2522	
  2523		crop->left = 0;
  2524		crop->top = 0;
  2525		crop->width = fmt->format.width;
  2526		crop->height = fmt->format.height;
  2527		ccs_propagate(subdev, sd_state, V4L2_SEL_TGT_CROP);
  2528	
  2529		return 0;
  2530	}
  2531	

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