CC: [email protected] TO: Sakari Ailus <[email protected]> CC: Laurent Pinchart <[email protected]> CC: "Niklas Söderlund" <[email protected]> CC: Jacopo Mondi <[email protected]>
tree: git://linuxtv.org/pinchartl/media.git am65x/devel/v4 head: bd5d5f421c359cb5291d07410087503dd21bb0fe commit: 9e10bd721ab5707b8787fef07d50ba418056e9e1 [33/50] media: entity: Move the pipeline from entity to pads :::::: branch date: 7 hours ago :::::: commit date: 8 hours ago config: i386-randconfig-m021-20210128 (attached as .config) compiler: gcc-9 (Debian 9.3.0-15) 9.3.0 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <[email protected]> Reported-by: Dan Carpenter <[email protected]> smatch warnings: drivers/media/mc/mc-entity.c:443 __media_pipeline_start() error: uninitialized symbol 'ret'. vim +/ret +443 drivers/media/mc/mc-entity.c d295c6a460cd2a drivers/media/media-entity.c Niklas Söderlund 2017-06-15 401 e02188c90f6ef6 drivers/media/media-entity.c Laurent Pinchart 2010-08-25 402 /* ----------------------------------------------------------------------------- e02188c90f6ef6 drivers/media/media-entity.c Laurent Pinchart 2010-08-25 403 * Pipeline management e02188c90f6ef6 drivers/media/media-entity.c Laurent Pinchart 2010-08-25 404 */ e02188c90f6ef6 drivers/media/media-entity.c Laurent Pinchart 2010-08-25 405 20b852273642f4 drivers/media/media-entity.c Sakari Ailus 2016-11-21 406 __must_check int __media_pipeline_start(struct media_entity *entity, e02188c90f6ef6 drivers/media/media-entity.c Laurent Pinchart 2010-08-25 407 struct media_pipeline *pipe) e02188c90f6ef6 drivers/media/media-entity.c Laurent Pinchart 2010-08-25 408 { d10c98949d1a1f drivers/media/media-entity.c Javier Martinez Canillas 2015-08-19 409 struct media_device *mdev = entity->graph_obj.mdev; 20b852273642f4 drivers/media/media-entity.c Sakari Ailus 2016-11-21 410 struct media_graph *graph = &pipe->graph; 06e04736922bbe drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 411 struct media_pad *pad = entity->pads; 06e04736922bbe drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 412 struct media_pad *pad_err = pad; 57208e5e25f263 drivers/media/media-entity.c Mauro Carvalho Chehab 2015-08-07 413 struct media_link *link; af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 414 int ret; e02188c90f6ef6 drivers/media/media-entity.c Laurent Pinchart 2010-08-25 415 74a4133079f739 drivers/media/media-entity.c Sakari Ailus 2015-12-16 416 if (!pipe->streaming_count++) { 20b852273642f4 drivers/media/media-entity.c Sakari Ailus 2016-11-21 417 ret = media_graph_walk_init(&pipe->graph, mdev); 74a4133079f739 drivers/media/media-entity.c Sakari Ailus 2015-12-16 418 if (ret) 74a4133079f739 drivers/media/media-entity.c Sakari Ailus 2015-12-16 419 goto error_graph_walk_start; 106b9907c368e3 drivers/media/media-entity.c Sakari Ailus 2015-12-16 420 } 106b9907c368e3 drivers/media/media-entity.c Sakari Ailus 2015-12-16 421 06e04736922bbe drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 422 media_graph_walk_start(&pipe->graph, pad); 06e04736922bbe drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 423 06e04736922bbe drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 424 while ((pad = media_graph_walk_next(graph))) { 06e04736922bbe drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 425 struct media_entity *entity = pad->entity; 9e10bd721ab570 drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 426 bool skip_validation = pad->pipe; 9e10bd721ab570 drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 427 struct media_pad *iter; e02188c90f6ef6 drivers/media/media-entity.c Laurent Pinchart 2010-08-25 428 ef69ee1bc2c10f drivers/media/media-entity.c Mauro Carvalho Chehab 2015-10-01 429 DECLARE_BITMAP(active, MEDIA_ENTITY_MAX_PADS); ef69ee1bc2c10f drivers/media/media-entity.c Mauro Carvalho Chehab 2015-10-01 430 DECLARE_BITMAP(has_no_links, MEDIA_ENTITY_MAX_PADS); af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 431 9e10bd721ab570 drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 432 media_entity_for_each_pad(entity, iter) { 9e10bd721ab570 drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 433 if (iter->pipe && iter->pipe != pipe) { 3eb87773dee633 drivers/media/media-entity.c Shuah Khan 2019-02-22 434 pr_err("Pipe active for %s. Can't start for %s\n", 9e10bd721ab570 drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 435 entity->name, iter->entity->name); 8aaf62b5b9bef7 drivers/media/media-entity.c Sakari Ailus 2015-11-29 436 ret = -EBUSY; 9e10bd721ab570 drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 437 } else { 9e10bd721ab570 drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 438 iter->pipe = pipe; 9e10bd721ab570 drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 439 } 9e10bd721ab570 drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 440 iter->stream_count++; 8aaf62b5b9bef7 drivers/media/media-entity.c Sakari Ailus 2015-11-29 441 } 8aaf62b5b9bef7 drivers/media/media-entity.c Sakari Ailus 2015-11-29 442 9e10bd721ab570 drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 @443 if (ret) 9e10bd721ab570 drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 444 goto error; af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 445 9e10bd721ab570 drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 446 /* Already part of the pipeline, skip validation. */ 9e10bd721ab570 drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 447 if (skip_validation) af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 448 continue; af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 449 af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 450 if (!entity->ops || !entity->ops->link_validate) af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 451 continue; af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 452 de49c285a36049 drivers/media/media-entity.c Sakari Ailus 2013-10-13 453 bitmap_zero(active, entity->num_pads); de49c285a36049 drivers/media/media-entity.c Sakari Ailus 2013-10-13 454 bitmap_fill(has_no_links, entity->num_pads); de49c285a36049 drivers/media/media-entity.c Sakari Ailus 2013-10-13 455 57208e5e25f263 drivers/media/media-entity.c Mauro Carvalho Chehab 2015-08-07 456 list_for_each_entry(link, &entity->links, list) { 06e04736922bbe drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 457 struct media_pad *other_pad = 06e04736922bbe drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 458 link->sink->entity == entity ? 06e04736922bbe drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 459 link->sink : link->source; af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 460 de49c285a36049 drivers/media/media-entity.c Sakari Ailus 2013-10-13 461 /* Mark that a pad is connected by a link. */ 06e04736922bbe drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 462 bitmap_clear(has_no_links, other_pad->index, 1); de49c285a36049 drivers/media/media-entity.c Sakari Ailus 2013-10-13 463 de49c285a36049 drivers/media/media-entity.c Sakari Ailus 2013-10-13 464 /* de49c285a36049 drivers/media/media-entity.c Sakari Ailus 2013-10-13 465 * Pads that either do not need to connect or de49c285a36049 drivers/media/media-entity.c Sakari Ailus 2013-10-13 466 * are connected through an enabled link are de49c285a36049 drivers/media/media-entity.c Sakari Ailus 2013-10-13 467 * fine. de49c285a36049 drivers/media/media-entity.c Sakari Ailus 2013-10-13 468 */ de49c285a36049 drivers/media/media-entity.c Sakari Ailus 2013-10-13 469 if (!(pad->flags & MEDIA_PAD_FL_MUST_CONNECT) || de49c285a36049 drivers/media/media-entity.c Sakari Ailus 2013-10-13 470 link->flags & MEDIA_LNK_FL_ENABLED) 06e04736922bbe drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 471 bitmap_set(active, other_pad->index, 1); af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 472 de49c285a36049 drivers/media/media-entity.c Sakari Ailus 2013-10-13 473 /* de49c285a36049 drivers/media/media-entity.c Sakari Ailus 2013-10-13 474 * Link validation will only take place for de49c285a36049 drivers/media/media-entity.c Sakari Ailus 2013-10-13 475 * sink ends of the link that are enabled. de49c285a36049 drivers/media/media-entity.c Sakari Ailus 2013-10-13 476 */ 06e04736922bbe drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 477 if (link->sink != other_pad || de49c285a36049 drivers/media/media-entity.c Sakari Ailus 2013-10-13 478 !(link->flags & MEDIA_LNK_FL_ENABLED)) af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 479 continue; af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 480 af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 481 ret = entity->ops->link_validate(link); fab9d30b7b4216 drivers/media/media-entity.c Sakari Ailus 2014-10-28 482 if (ret < 0 && ret != -ENOIOCTLCMD) { d10c98949d1a1f drivers/media/media-entity.c Javier Martinez Canillas 2015-08-19 483 dev_dbg(entity->graph_obj.mdev->dev, 91b619adeb3753 drivers/media/media-entity.c Sakari Ailus 2016-12-08 484 "link validation failed for '%s':%u -> '%s':%u, error %d\n", 823ea2a639611f drivers/media/media-entity.c Sakari Ailus 2015-02-12 485 link->source->entity->name, 823ea2a639611f drivers/media/media-entity.c Sakari Ailus 2015-02-12 486 link->source->index, 823ea2a639611f drivers/media/media-entity.c Sakari Ailus 2015-02-12 487 entity->name, link->sink->index, ret); af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 488 goto error; af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 489 } fab9d30b7b4216 drivers/media/media-entity.c Sakari Ailus 2014-10-28 490 } de49c285a36049 drivers/media/media-entity.c Sakari Ailus 2013-10-13 491 de49c285a36049 drivers/media/media-entity.c Sakari Ailus 2013-10-13 492 /* Either no links or validated links are fine. */ de49c285a36049 drivers/media/media-entity.c Sakari Ailus 2013-10-13 493 bitmap_or(active, active, has_no_links, entity->num_pads); de49c285a36049 drivers/media/media-entity.c Sakari Ailus 2013-10-13 494 de49c285a36049 drivers/media/media-entity.c Sakari Ailus 2013-10-13 495 if (!bitmap_full(active, entity->num_pads)) { 47dfdb3a888733 drivers/media/media-entity.c Helen Mae Koike Fornazier 2015-06-28 496 ret = -ENOLINK; d10c98949d1a1f drivers/media/media-entity.c Javier Martinez Canillas 2015-08-19 497 dev_dbg(entity->graph_obj.mdev->dev, 91b619adeb3753 drivers/media/media-entity.c Sakari Ailus 2016-12-08 498 "'%s':%u must be connected by an enabled link\n", fab9d30b7b4216 drivers/media/media-entity.c Sakari Ailus 2014-10-28 499 entity->name, 094f1ca516cd32 drivers/media/media-entity.c Sakari Ailus 2014-11-03 500 (unsigned)find_first_zero_bit( 094f1ca516cd32 drivers/media/media-entity.c Sakari Ailus 2014-11-03 501 active, entity->num_pads)); de49c285a36049 drivers/media/media-entity.c Sakari Ailus 2013-10-13 502 goto error; de49c285a36049 drivers/media/media-entity.c Sakari Ailus 2013-10-13 503 } e02188c90f6ef6 drivers/media/media-entity.c Laurent Pinchart 2010-08-25 504 } e02188c90f6ef6 drivers/media/media-entity.c Laurent Pinchart 2010-08-25 505 af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 506 return 0; af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 507 af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 508 error: af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 509 /* af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 510 * Link validation on graph failed. We revert what we did and af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 511 * return the error. af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 512 */ 06e04736922bbe drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 513 media_graph_walk_start(graph, pad_err); 06e04736922bbe drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 514 06e04736922bbe drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 515 while ((pad_err = media_graph_walk_next(graph))) { 9e10bd721ab570 drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 516 struct media_entity *entity = pad->entity; 9e10bd721ab570 drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 517 struct media_pad *iter; af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 518 9e10bd721ab570 drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 519 media_entity_for_each_pad(entity, iter) { 12030f489ef87d drivers/media/media-entity.c Sakari Ailus 2016-07-20 520 /* Sanity check for negative stream_count */ 9e10bd721ab570 drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 521 if (!WARN_ON_ONCE(iter->stream_count <= 0)) { 9e10bd721ab570 drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 522 --iter->stream_count; 9e10bd721ab570 drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 523 if (iter->stream_count == 0) 9e10bd721ab570 drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 524 iter->pipe = NULL; 9e10bd721ab570 drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 525 } 3801bc7d1b8dd7 drivers/media/media-entity.c Shuah Khan 2016-01-30 526 } af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 527 af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 528 /* af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 529 * We haven't increased stream_count further than this af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 530 * so we quit here. af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 531 */ 9e10bd721ab570 drivers/media/mc/mc-entity.c Sakari Ailus 2019-03-28 532 if (pad_err->entity == pad->entity) af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 533 break; af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 534 } af88be3887c1a0 drivers/media/media-entity.c Sakari Ailus 2012-01-11 535 74a4133079f739 drivers/media/media-entity.c Sakari Ailus 2015-12-16 536 error_graph_walk_start: 74a4133079f739 drivers/media/media-entity.c Sakari Ailus 2015-12-16 537 if (!--pipe->streaming_count) 20b852273642f4 drivers/media/media-entity.c Sakari Ailus 2016-11-21 538 media_graph_walk_cleanup(graph); 106b9907c368e3 drivers/media/media-entity.c Sakari Ailus 2015-12-16 539 fb49f20438f361 drivers/media/media-entity.c Shuah Khan 2016-02-11 540 return ret; fb49f20438f361 drivers/media/media-entity.c Shuah Khan 2016-02-11 541 } 20b852273642f4 drivers/media/media-entity.c Sakari Ailus 2016-11-21 542 EXPORT_SYMBOL_GPL(__media_pipeline_start); fb49f20438f361 drivers/media/media-entity.c Shuah Khan 2016-02-11 543 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/[email protected]
.config.gz
Description: application/gzip
_______________________________________________ kbuild mailing list -- [email protected] To unsubscribe send an email to [email protected]
