tree: git://linuxtv.org/sailus/media_tree.git metadata-fix head: 56ea581e40d9aa9ffe425f4faa4b00ce389b9053 commit: cb744839166872eaaa05370a305f7f77c9b29852 [37/39] media: v4l: subdev: Set sub-device active state earlier config: powerpc-allmodconfig (https://download.01.org/0day-ci/archive/20231010/202310100428.9YKB4IiP-lkp@xxxxxxxxx/config) compiler: clang version 17.0.0 (https://github.com/llvm/llvm-project.git 4a5ac14ee968ff0ad5d2cc1ffa0299048db4c88a) reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20231010/202310100428.9YKB4IiP-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/202310100428.9YKB4IiP-lkp@xxxxxxxxx/ All errors (new ones prefixed by >>): >> drivers/staging/media/tegra-video/vi.c:453:15: error: too few arguments to function call, expected 4, have 3 453 | sd_state = __v4l2_subdev_state_alloc(subdev, "tegra:state->lock", | ~~~~~~~~~~~~~~~~~~~~~~~~~ 454 | &key); | ^ include/media/v4l2-subdev.h:1347:27: note: '__v4l2_subdev_state_alloc' declared here 1347 | struct v4l2_subdev_state *__v4l2_subdev_state_alloc(struct v4l2_subdev *sd, | ^ 1 error generated. -- >> drivers/media/platform/renesas/rcar-vin/rcar-v4l2.c:269:67: error: too few arguments to function call, expected 4, have 3 269 | sd_state = __v4l2_subdev_state_alloc(sd, "rvin:state->lock", &key); | ~~~~~~~~~~~~~~~~~~~~~~~~~ ^ include/media/v4l2-subdev.h:1347:27: note: '__v4l2_subdev_state_alloc' declared here 1347 | struct v4l2_subdev_state *__v4l2_subdev_state_alloc(struct v4l2_subdev *sd, | ^ 1 error generated. -- >> drivers/media/platform/st/stm32/stm32-dcmi.c:998:8: error: too few arguments to function call, expected 4, have 3 998 | ret = v4l2_subdev_call_state_try(dcmi->source, pad, set_fmt, &format); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ include/media/v4l2-subdev.h:1887:46: note: expanded from macro 'v4l2_subdev_call_state_try' 1887 | __v4l2_subdev_state_alloc(sd, name, &__key); \ | ~~~~~~~~~~~~~~~~~~~~~~~~~ ^ include/media/v4l2-subdev.h:1347:27: note: '__v4l2_subdev_state_alloc' declared here 1347 | struct v4l2_subdev_state *__v4l2_subdev_state_alloc(struct v4l2_subdev *sd, | ^ drivers/media/platform/st/stm32/stm32-dcmi.c:1159:8: error: too few arguments to function call, expected 4, have 3 1159 | ret = v4l2_subdev_call_state_try(dcmi->source, pad, set_fmt, &format); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ include/media/v4l2-subdev.h:1887:46: note: expanded from macro 'v4l2_subdev_call_state_try' 1887 | __v4l2_subdev_state_alloc(sd, name, &__key); \ | ~~~~~~~~~~~~~~~~~~~~~~~~~ ^ include/media/v4l2-subdev.h:1347:27: note: '__v4l2_subdev_state_alloc' declared here 1347 | struct v4l2_subdev_state *__v4l2_subdev_state_alloc(struct v4l2_subdev *sd, | ^ 2 errors generated. -- >> drivers/media/platform/renesas/vsp1/vsp1_entity.c:683:35: error: too few arguments to function call, expected 4, have 3 683 | entity->config = __v4l2_subdev_state_alloc(&entity->subdev, | ~~~~~~~~~~~~~~~~~~~~~~~~~ 684 | "vsp1:config->lock", &key); | ^ include/media/v4l2-subdev.h:1347:27: note: '__v4l2_subdev_state_alloc' declared here 1347 | struct v4l2_subdev_state *__v4l2_subdev_state_alloc(struct v4l2_subdev *sd, | ^ 1 error generated. vim +453 drivers/staging/media/tegra-video/vi.c 3d8a97eabef088 Sowjanya Komatineni 2020-05-04 424 3d8a97eabef088 Sowjanya Komatineni 2020-05-04 425 static int __tegra_channel_try_format(struct tegra_vi_channel *chan, 3d8a97eabef088 Sowjanya Komatineni 2020-05-04 426 struct v4l2_pix_format *pix) 3d8a97eabef088 Sowjanya Komatineni 2020-05-04 427 { 3d8a97eabef088 Sowjanya Komatineni 2020-05-04 428 const struct tegra_video_format *fmtinfo; ed647ea668fb27 Tomi Valkeinen 2022-04-12 429 static struct lock_class_key key; 3d8a97eabef088 Sowjanya Komatineni 2020-05-04 430 struct v4l2_subdev *subdev; ecefa105cc44ff Laurent Pinchart 2023-02-15 431 struct v4l2_subdev_format fmt = { ecefa105cc44ff Laurent Pinchart 2023-02-15 432 .which = V4L2_SUBDEV_FORMAT_TRY, ecefa105cc44ff Laurent Pinchart 2023-02-15 433 }; 0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 434 struct v4l2_subdev_state *sd_state; bdcad5ce6dde6e Sowjanya Komatineni 2020-08-12 435 struct v4l2_subdev_frame_size_enum fse = { bdcad5ce6dde6e Sowjanya Komatineni 2020-08-12 436 .which = V4L2_SUBDEV_FORMAT_TRY, bdcad5ce6dde6e Sowjanya Komatineni 2020-08-12 437 }; bdcad5ce6dde6e Sowjanya Komatineni 2020-08-12 438 struct v4l2_subdev_selection sdsel = { bdcad5ce6dde6e Sowjanya Komatineni 2020-08-12 439 .which = V4L2_SUBDEV_FORMAT_ACTIVE, bdcad5ce6dde6e Sowjanya Komatineni 2020-08-12 440 .target = V4L2_SEL_TGT_CROP_BOUNDS, bdcad5ce6dde6e Sowjanya Komatineni 2020-08-12 441 }; 1ebaeb09830f36 Sowjanya Komatineni 2020-08-12 442 int ret; 1ebaeb09830f36 Sowjanya Komatineni 2020-08-12 443 1ebaeb09830f36 Sowjanya Komatineni 2020-08-12 444 subdev = tegra_channel_get_remote_source_subdev(chan); 1ebaeb09830f36 Sowjanya Komatineni 2020-08-12 445 if (!subdev) 1ebaeb09830f36 Sowjanya Komatineni 2020-08-12 446 return -ENODEV; 3d8a97eabef088 Sowjanya Komatineni 2020-05-04 447 40aaab9d773b07 Tomi Valkeinen 2022-04-12 448 /* 40aaab9d773b07 Tomi Valkeinen 2022-04-12 449 * FIXME: Drop this call, drivers are not supposed to use 40aaab9d773b07 Tomi Valkeinen 2022-04-12 450 * __v4l2_subdev_state_alloc(). 40aaab9d773b07 Tomi Valkeinen 2022-04-12 451 */ ed647ea668fb27 Tomi Valkeinen 2022-04-12 452 sd_state = __v4l2_subdev_state_alloc(subdev, "tegra:state->lock", ed647ea668fb27 Tomi Valkeinen 2022-04-12 @453 &key); ba7a93e507f883 Dan Carpenter 2021-06-22 454 if (IS_ERR(sd_state)) ba7a93e507f883 Dan Carpenter 2021-06-22 455 return PTR_ERR(sd_state); 3d8a97eabef088 Sowjanya Komatineni 2020-05-04 456 /* 3d8a97eabef088 Sowjanya Komatineni 2020-05-04 457 * Retrieve the format information and if requested format isn't 3d8a97eabef088 Sowjanya Komatineni 2020-05-04 458 * supported, keep the current format. 3d8a97eabef088 Sowjanya Komatineni 2020-05-04 459 */ 3d8a97eabef088 Sowjanya Komatineni 2020-05-04 460 fmtinfo = tegra_get_format_by_fourcc(chan->vi, pix->pixelformat); 3d8a97eabef088 Sowjanya Komatineni 2020-05-04 461 if (!fmtinfo) { 3d8a97eabef088 Sowjanya Komatineni 2020-05-04 462 pix->pixelformat = chan->format.pixelformat; 3d8a97eabef088 Sowjanya Komatineni 2020-05-04 463 pix->colorspace = chan->format.colorspace; 3d8a97eabef088 Sowjanya Komatineni 2020-05-04 464 fmtinfo = tegra_get_format_by_fourcc(chan->vi, 3d8a97eabef088 Sowjanya Komatineni 2020-05-04 465 pix->pixelformat); 3d8a97eabef088 Sowjanya Komatineni 2020-05-04 466 } 3d8a97eabef088 Sowjanya Komatineni 2020-05-04 467 3d8a97eabef088 Sowjanya Komatineni 2020-05-04 468 pix->field = V4L2_FIELD_NONE; 3d8a97eabef088 Sowjanya Komatineni 2020-05-04 469 fmt.pad = 0; 3d8a97eabef088 Sowjanya Komatineni 2020-05-04 470 v4l2_fill_mbus_format(&fmt.format, pix, fmtinfo->code); bdcad5ce6dde6e Sowjanya Komatineni 2020-08-12 471 bdcad5ce6dde6e Sowjanya Komatineni 2020-08-12 472 /* bdcad5ce6dde6e Sowjanya Komatineni 2020-08-12 473 * Attempt to obtain the format size from subdev. bdcad5ce6dde6e Sowjanya Komatineni 2020-08-12 474 * If not available, try to get crop boundary from subdev. bdcad5ce6dde6e Sowjanya Komatineni 2020-08-12 475 */ bdcad5ce6dde6e Sowjanya Komatineni 2020-08-12 476 fse.code = fmtinfo->code; 0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 477 ret = v4l2_subdev_call(subdev, pad, enum_frame_size, sd_state, &fse); bdcad5ce6dde6e Sowjanya Komatineni 2020-08-12 478 if (ret) { 56f64b82356b74 Sowjanya Komatineni 2020-12-11 479 if (!v4l2_subdev_has_op(subdev, pad, get_selection)) { 0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 480 sd_state->pads->try_crop.width = 0; 0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 481 sd_state->pads->try_crop.height = 0; 56f64b82356b74 Sowjanya Komatineni 2020-12-11 482 } else { 56f64b82356b74 Sowjanya Komatineni 2020-12-11 483 ret = v4l2_subdev_call(subdev, pad, get_selection, 56f64b82356b74 Sowjanya Komatineni 2020-12-11 484 NULL, &sdsel); bdcad5ce6dde6e Sowjanya Komatineni 2020-08-12 485 if (ret) bdcad5ce6dde6e Sowjanya Komatineni 2020-08-12 486 return -EINVAL; 56f64b82356b74 Sowjanya Komatineni 2020-12-11 487 0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 488 sd_state->pads->try_crop.width = sdsel.r.width; 0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 489 sd_state->pads->try_crop.height = sdsel.r.height; 56f64b82356b74 Sowjanya Komatineni 2020-12-11 490 } bdcad5ce6dde6e Sowjanya Komatineni 2020-08-12 491 } else { 0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 492 sd_state->pads->try_crop.width = fse.max_width; 0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 493 sd_state->pads->try_crop.height = fse.max_height; bdcad5ce6dde6e Sowjanya Komatineni 2020-08-12 494 } bdcad5ce6dde6e Sowjanya Komatineni 2020-08-12 495 0d346d2a6f54f0 Tomi Valkeinen 2021-06-10 496 ret = v4l2_subdev_call(subdev, pad, set_fmt, sd_state, &fmt); 1ebaeb09830f36 Sowjanya Komatineni 2020-08-12 497 if (ret < 0) 1ebaeb09830f36 Sowjanya Komatineni 2020-08-12 498 return ret; 1ebaeb09830f36 Sowjanya Komatineni 2020-08-12 499 3d8a97eabef088 Sowjanya Komatineni 2020-05-04 500 v4l2_fill_pix_format(pix, &fmt.format); 1a3ea975d79418 Luca Ceresoli 2023-04-18 501 chan->vi->ops->vi_fmt_align(pix, fmtinfo->bpp); 3d8a97eabef088 Sowjanya Komatineni 2020-05-04 502 40aaab9d773b07 Tomi Valkeinen 2022-04-12 503 __v4l2_subdev_state_free(sd_state); 3d8a97eabef088 Sowjanya Komatineni 2020-05-04 504 3d8a97eabef088 Sowjanya Komatineni 2020-05-04 505 return 0; 3d8a97eabef088 Sowjanya Komatineni 2020-05-04 506 } 3d8a97eabef088 Sowjanya Komatineni 2020-05-04 507 :::::: The code at line 453 was first introduced by commit :::::: ed647ea668fb27cd21408d5cb7cc7d4c30417332 media: subdev: add subdev state locking :::::: TO: Tomi Valkeinen <tomi.valkeinen@xxxxxxxxxxxxxxxx> :::::: CC: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx> -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki
