lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Wed, 27 Dec 2023 19:10:42 +0800
From: kernel test robot <lkp@...el.com>
To: Konrad Dybcio <konrad.dybcio@...aro.org>,
	Bjorn Andersson <andersson@...nel.org>,
	Liam Girdwood <lgirdwood@...il.com>,
	Mark Brown <broonie@...nel.org>
Cc: oe-kbuild-all@...ts.linux.dev,
	Marijn Suijten <marijn.suijten@...ainline.org>,
	linux-arm-msm@...r.kernel.org, linux-kernel@...r.kernel.org,
	Konrad Dybcio <konrad.dybcio@...aro.org>
Subject: Re: [PATCH] regulator: qcom_smd: Keep one rpm handle for all vregs

Hi Konrad,

kernel test robot noticed the following build warnings:

[auto build test WARNING on 39676dfe52331dba909c617f213fdb21015c8d10]

url:    https://github.com/intel-lab-lkp/linux/commits/Konrad-Dybcio/regulator-qcom_smd-Keep-one-rpm-handle-for-all-vregs/20231227-093153
base:   39676dfe52331dba909c617f213fdb21015c8d10
patch link:    https://lore.kernel.org/r/20231227-topic-rpm_vreg_cleanup-v1-1-949da0864ac5%40linaro.org
patch subject: [PATCH] regulator: qcom_smd: Keep one rpm handle for all vregs
config: arc-randconfig-001-20231227 (https://download.01.org/0day-ci/archive/20231227/202312271812.C5jldM3L-lkp@intel.com/config)
compiler: arceb-elf-gcc (GCC) 13.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20231227/202312271812.C5jldM3L-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@...el.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202312271812.C5jldM3L-lkp@intel.com/

All warnings (new ones prefixed by >>):

>> drivers/regulator/qcom_smd-regulator.c:1395: warning: Excess function parameter 'rpm' description in 'rpm_regulator_init_vreg'


vim +1395 drivers/regulator/qcom_smd-regulator.c

da65e367b67e15 Bjorn Andersson 2015-08-17  1379  
14e2976fbabdac Konrad Dybcio   2021-12-30  1380  /**
14e2976fbabdac Konrad Dybcio   2021-12-30  1381   * rpm_regulator_init_vreg() - initialize all attributes of a qcom_smd-regulator
14e2976fbabdac Konrad Dybcio   2021-12-30  1382   * @vreg:		Pointer to the individual qcom_smd-regulator resource
14e2976fbabdac Konrad Dybcio   2021-12-30  1383   * @dev:		Pointer to the top level qcom_smd-regulator PMIC device
14e2976fbabdac Konrad Dybcio   2021-12-30  1384   * @node:		Pointer to the individual qcom_smd-regulator resource
14e2976fbabdac Konrad Dybcio   2021-12-30  1385   *			device node
14e2976fbabdac Konrad Dybcio   2021-12-30  1386   * @rpm:		Pointer to the rpm bus node
14e2976fbabdac Konrad Dybcio   2021-12-30  1387   * @pmic_rpm_data:	Pointer to a null-terminated array of qcom_smd-regulator
14e2976fbabdac Konrad Dybcio   2021-12-30  1388   *			resources defined for the top level PMIC device
14e2976fbabdac Konrad Dybcio   2021-12-30  1389   *
14e2976fbabdac Konrad Dybcio   2021-12-30  1390   * Return: 0 on success, errno on failure
14e2976fbabdac Konrad Dybcio   2021-12-30  1391   */
14e2976fbabdac Konrad Dybcio   2021-12-30  1392  static int rpm_regulator_init_vreg(struct qcom_rpm_reg *vreg, struct device *dev,
d7cdd450e31088 Konrad Dybcio   2023-12-27  1393  				   struct device_node *node,
14e2976fbabdac Konrad Dybcio   2021-12-30  1394  				   const struct rpm_regulator_data *pmic_rpm_data)
da65e367b67e15 Bjorn Andersson 2015-08-17 @1395  {
da65e367b67e15 Bjorn Andersson 2015-08-17  1396  	struct regulator_config config = {};
14e2976fbabdac Konrad Dybcio   2021-12-30  1397  	const struct rpm_regulator_data *rpm_data;
da65e367b67e15 Bjorn Andersson 2015-08-17  1398  	struct regulator_dev *rdev;
14e2976fbabdac Konrad Dybcio   2021-12-30  1399  	int ret;
14e2976fbabdac Konrad Dybcio   2021-12-30  1400  
14e2976fbabdac Konrad Dybcio   2021-12-30  1401  	for (rpm_data = pmic_rpm_data; rpm_data->name; rpm_data++)
14e2976fbabdac Konrad Dybcio   2021-12-30  1402  		if (of_node_name_eq(node, rpm_data->name))
14e2976fbabdac Konrad Dybcio   2021-12-30  1403  			break;
14e2976fbabdac Konrad Dybcio   2021-12-30  1404  
14e2976fbabdac Konrad Dybcio   2021-12-30  1405  	if (!rpm_data->name) {
14e2976fbabdac Konrad Dybcio   2021-12-30  1406  		dev_err(dev, "Unknown regulator %pOFn\n", node);
14e2976fbabdac Konrad Dybcio   2021-12-30  1407  		return -EINVAL;
14e2976fbabdac Konrad Dybcio   2021-12-30  1408  	}
14e2976fbabdac Konrad Dybcio   2021-12-30  1409  
14e2976fbabdac Konrad Dybcio   2021-12-30  1410  	vreg->dev	= dev;
14e2976fbabdac Konrad Dybcio   2021-12-30  1411  	vreg->type	= rpm_data->type;
14e2976fbabdac Konrad Dybcio   2021-12-30  1412  	vreg->id	= rpm_data->id;
14e2976fbabdac Konrad Dybcio   2021-12-30  1413  
14e2976fbabdac Konrad Dybcio   2021-12-30  1414  	memcpy(&vreg->desc, rpm_data->desc, sizeof(vreg->desc));
14e2976fbabdac Konrad Dybcio   2021-12-30  1415  	vreg->desc.name = rpm_data->name;
14e2976fbabdac Konrad Dybcio   2021-12-30  1416  	vreg->desc.supply_name = rpm_data->supply;
14e2976fbabdac Konrad Dybcio   2021-12-30  1417  	vreg->desc.owner = THIS_MODULE;
14e2976fbabdac Konrad Dybcio   2021-12-30  1418  	vreg->desc.type = REGULATOR_VOLTAGE;
14e2976fbabdac Konrad Dybcio   2021-12-30  1419  	vreg->desc.of_match = rpm_data->name;
14e2976fbabdac Konrad Dybcio   2021-12-30  1420  
14e2976fbabdac Konrad Dybcio   2021-12-30  1421  	config.dev		= dev;
14e2976fbabdac Konrad Dybcio   2021-12-30  1422  	config.of_node		= node;
14e2976fbabdac Konrad Dybcio   2021-12-30  1423  	config.driver_data	= vreg;
14e2976fbabdac Konrad Dybcio   2021-12-30  1424  
14e2976fbabdac Konrad Dybcio   2021-12-30  1425  	rdev = devm_regulator_register(dev, &vreg->desc, &config);
14e2976fbabdac Konrad Dybcio   2021-12-30  1426  	if (IS_ERR(rdev)) {
14e2976fbabdac Konrad Dybcio   2021-12-30  1427  		ret = PTR_ERR(rdev);
14e2976fbabdac Konrad Dybcio   2021-12-30  1428  		dev_err(dev, "%pOFn: devm_regulator_register() failed, ret=%d\n", node, ret);
14e2976fbabdac Konrad Dybcio   2021-12-30  1429  		return ret;
14e2976fbabdac Konrad Dybcio   2021-12-30  1430  	}
14e2976fbabdac Konrad Dybcio   2021-12-30  1431  
14e2976fbabdac Konrad Dybcio   2021-12-30  1432  	return 0;
14e2976fbabdac Konrad Dybcio   2021-12-30  1433  }
14e2976fbabdac Konrad Dybcio   2021-12-30  1434  

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

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ