From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 585BD18A6D3; Sun, 6 Oct 2024 20:04:41 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=198.175.65.17 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728245083; cv=none; b=AKAJQMr9q8G/JnL/vCtny7VMb42mjtmSwONm+ztuT6AXYDnPWmy6CdpQRgbFrMNY8j7iyON681YT3msu+qH4gt2+PEtfe5AGBdhiQubjgsQWhh+N1tExTUmtJzJDAuI4dbFBaHULSPOK2ZQ+gk0yBTeDAvJ4iXLUljszOhWKKEc= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728245083; c=relaxed/simple; bh=jFJ2pP4x6dDzHcclHbsHUHuXjjQpry7txxpsVKHJTkg=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=e3B4bPpbWnfaTD0pOLjvvJ7VKLh/smVb6fs2VfJO888uSKzIpotz3GfQjYaMC/yuNnQIGcZIKDF73TQRhEN90d5ZsdLEx04IMfJ9/sFWJ6yB7JyDwapPpM4zpxaBPgWf3V0kL3MorH54ik4H3XvW6E4RvYco0RLfGE4xcNeWoWE= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com; spf=pass smtp.mailfrom=intel.com; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b=fCGIZgTu; arc=none smtp.client-ip=198.175.65.17 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=intel.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b="fCGIZgTu" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1728245082; x=1759781082; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=jFJ2pP4x6dDzHcclHbsHUHuXjjQpry7txxpsVKHJTkg=; b=fCGIZgTuNW5+1fxz4U0rKta+COythdfa3tpvrdt9xvx1xjfVNdRF+Vak 96OOCNm5gaZl9E1iTSNKkrYhRahH8XTEMQJx0OxjIYwncRuOmN0NuhyDs likDEC3L5Pl0wN8YeArg23rlrZ0uRHcw7XPlet8wFG4cJQrnw7dcdwQzG eM1yGe4vdO/BYqLdZvciJ2rWnIBQAVsHrkoRPiJHqn9/AbTD8BWQ/UmUn Y+OT5ZCIp9ZqyEN0WDx8ID/5eZnC/4xX5OkQ1h2+aOmcbK2CkVBuQjsKZ j7ZzWNWb/4f8khIkHmMXxnotBbkSpLApo2DWg9gcSzY7S9je8xrpWj7bd A==; X-CSE-ConnectionGUID: KDaQOe49SKalNGeeFZdZCg== X-CSE-MsgGUID: vZ1zimAaTJa/CcyjP6zp6Q== X-IronPort-AV: E=McAfee;i="6700,10204,11217"; a="27521639" X-IronPort-AV: E=Sophos;i="6.11,182,1725346800"; d="scan'208";a="27521639" Received: from orviesa007.jf.intel.com ([10.64.159.147]) by orvoesa109.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 06 Oct 2024 13:04:41 -0700 X-CSE-ConnectionGUID: ITkbClHETpOB+p9t9n1VtQ== X-CSE-MsgGUID: Qxq3R5HVR1imPt1IwNd6Vg== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.11,182,1725346800"; d="scan'208";a="75705157" Received: from lkp-server01.sh.intel.com (HELO a48cf1aa22e8) ([10.239.97.150]) by orviesa007.jf.intel.com with ESMTP; 06 Oct 2024 13:04:38 -0700 Received: from kbuild by a48cf1aa22e8 with local (Exim 4.96) (envelope-from ) id 1sxXUa-0004J0-0c; Sun, 06 Oct 2024 20:04:36 +0000 Date: Mon, 7 Oct 2024 04:04:13 +0800 From: kernel test robot To: Drew Fustini , Drew Fustini , Guo Ren , Fu Wei , Linus Walleij Cc: oe-kbuild-all@lists.linux.dev, linux-riscv@lists.infradead.org, linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/2] pinctrl: th1520: Convert to thp->mutex to guarded mutex Message-ID: <202410070352.6BZrRWQU-lkp@intel.com> References: <20241005-th1520-pinctrl-fixes-v1-1-5c65dffa0d00@tenstorrent.com> Precedence: bulk X-Mailing-List: linux-gpio@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20241005-th1520-pinctrl-fixes-v1-1-5c65dffa0d00@tenstorrent.com> Hi Drew, kernel test robot noticed the following build warnings: [auto build test WARNING on 2694868880705e8f6bb61b24b1b25adc42a4a217] url: https://github.com/intel-lab-lkp/linux/commits/Drew-Fustini/pinctrl-th1520-Convert-to-thp-mutex-to-guarded-mutex/20241006-033647 base: 2694868880705e8f6bb61b24b1b25adc42a4a217 patch link: https://lore.kernel.org/r/20241005-th1520-pinctrl-fixes-v1-1-5c65dffa0d00%40tenstorrent.com patch subject: [PATCH 1/2] pinctrl: th1520: Convert to thp->mutex to guarded mutex config: powerpc64-randconfig-r073-20241007 (https://download.01.org/0day-ci/archive/20241007/202410070352.6BZrRWQU-lkp@intel.com/config) compiler: clang version 16.0.6 (https://github.com/llvm/llvm-project 7cbf1a2591520c2491aa35339f227775f4d3adf6) reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20241007/202410070352.6BZrRWQU-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 | Closes: https://lore.kernel.org/oe-kbuild-all/202410070352.6BZrRWQU-lkp@intel.com/ All warnings (new ones prefixed by >>): >> drivers/pinctrl/pinctrl-th1520.c:538:14: warning: variable 'child' is uninitialized when used here [-Wuninitialized] of_node_put(child); ^~~~~ drivers/pinctrl/pinctrl-th1520.c:420:27: note: initialize the variable 'child' to silence this warning struct device_node *child; ^ = NULL 1 warning generated. Kconfig warnings: (for reference only) WARNING: unmet direct dependencies detected for MODVERSIONS Depends on [n]: MODULES [=y] && !COMPILE_TEST [=y] Selected by [y]: - RANDSTRUCT_FULL [=y] && (CC_HAS_RANDSTRUCT [=y] || GCC_PLUGINS [=n]) && MODULES [=y] vim +/child +538 drivers/pinctrl/pinctrl-th1520.c bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 413 bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 414 static int th1520_pinctrl_dt_node_to_map(struct pinctrl_dev *pctldev, bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 415 struct device_node *np, bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 416 struct pinctrl_map **maps, bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 417 unsigned int *num_maps) bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 418 { bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 419 struct th1520_pinctrl *thp = pinctrl_dev_get_drvdata(pctldev); bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 420 struct device_node *child; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 421 struct pinctrl_map *map; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 422 unsigned long *configs; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 423 unsigned int nconfigs; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 424 unsigned int nmaps; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 425 int ret; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 426 bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 427 nmaps = 0; fb310b5cb13ad2 Drew Fustini 2024-10-05 428 for_each_available_child_of_node_scoped(np, child) { bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 429 int npins = of_property_count_strings(child, "pins"); bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 430 bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 431 if (npins <= 0) { bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 432 of_node_put(child); bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 433 dev_err(thp->pctl->dev, "no pins selected for %pOFn.%pOFn\n", bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 434 np, child); bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 435 return -EINVAL; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 436 } bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 437 nmaps += npins; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 438 if (of_property_present(child, "function")) bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 439 nmaps += npins; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 440 } bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 441 bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 442 map = kcalloc(nmaps, sizeof(*map), GFP_KERNEL); bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 443 if (!map) bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 444 return -ENOMEM; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 445 bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 446 nmaps = 0; fb310b5cb13ad2 Drew Fustini 2024-10-05 447 guard(mutex)(&thp->mutex); fb310b5cb13ad2 Drew Fustini 2024-10-05 448 for_each_available_child_of_node_scoped(np, child) { bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 449 unsigned int rollback = nmaps; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 450 enum th1520_muxtype muxtype; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 451 struct property *prop; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 452 const char *funcname; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 453 const char **pgnames; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 454 const char *pinname; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 455 int npins; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 456 bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 457 ret = pinconf_generic_parse_dt_config(child, pctldev, &configs, &nconfigs); bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 458 if (ret) { bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 459 dev_err(thp->pctl->dev, "%pOFn.%pOFn: error parsing pin config\n", bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 460 np, child); bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 461 goto put_child; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 462 } bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 463 bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 464 if (!of_property_read_string(child, "function", &funcname)) { bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 465 muxtype = th1520_muxtype_get(funcname); bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 466 if (!muxtype) { bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 467 dev_err(thp->pctl->dev, "%pOFn.%pOFn: unknown function '%s'\n", bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 468 np, child, funcname); bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 469 ret = -EINVAL; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 470 goto free_configs; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 471 } bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 472 bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 473 funcname = devm_kasprintf(thp->pctl->dev, GFP_KERNEL, "%pOFn.%pOFn", bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 474 np, child); bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 475 if (!funcname) { bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 476 ret = -ENOMEM; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 477 goto free_configs; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 478 } bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 479 bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 480 npins = of_property_count_strings(child, "pins"); bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 481 pgnames = devm_kcalloc(thp->pctl->dev, npins, sizeof(*pgnames), GFP_KERNEL); bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 482 if (!pgnames) { bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 483 ret = -ENOMEM; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 484 goto free_configs; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 485 } bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 486 } else { bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 487 funcname = NULL; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 488 } bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 489 bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 490 npins = 0; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 491 of_property_for_each_string(child, "pins", prop, pinname) { bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 492 unsigned int i; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 493 bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 494 for (i = 0; i < thp->desc.npins; i++) { bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 495 if (!strcmp(pinname, thp->desc.pins[i].name)) bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 496 break; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 497 } bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 498 if (i == thp->desc.npins) { bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 499 nmaps = rollback; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 500 dev_err(thp->pctl->dev, "%pOFn.%pOFn: unknown pin '%s'\n", bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 501 np, child, pinname); bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 502 goto free_configs; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 503 } bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 504 bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 505 if (nconfigs) { bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 506 map[nmaps].type = PIN_MAP_TYPE_CONFIGS_PIN; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 507 map[nmaps].data.configs.group_or_pin = thp->desc.pins[i].name; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 508 map[nmaps].data.configs.configs = configs; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 509 map[nmaps].data.configs.num_configs = nconfigs; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 510 nmaps += 1; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 511 } bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 512 if (funcname) { bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 513 pgnames[npins++] = thp->desc.pins[i].name; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 514 map[nmaps].type = PIN_MAP_TYPE_MUX_GROUP; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 515 map[nmaps].data.mux.function = funcname; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 516 map[nmaps].data.mux.group = thp->desc.pins[i].name; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 517 nmaps += 1; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 518 } bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 519 } bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 520 bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 521 if (funcname) { bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 522 ret = pinmux_generic_add_function(pctldev, funcname, pgnames, bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 523 npins, (void *)muxtype); bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 524 if (ret < 0) { bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 525 dev_err(thp->pctl->dev, "error adding function %s\n", funcname); bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 526 goto put_child; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 527 } bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 528 } bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 529 } bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 530 bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 531 *maps = map; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 532 *num_maps = nmaps; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 533 return 0; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 534 bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 535 free_configs: bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 536 kfree(configs); bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 537 put_child: bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 @538 of_node_put(child); bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 539 th1520_pinctrl_dt_free_map(pctldev, map, nmaps); bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 540 return ret; bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 541 } bed5cd6f8a9883 Emil Renner Berthing 2024-09-30 542 -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki