linux-kernel - Re: [PATCH v1 03/12] soc: qcom: geni-se: Introduce helper API for resource initialization

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]

Message-ID: <202511231819.jiLYo6Fl-lkp@intel.com>
Date: Sun, 23 Nov 2025 18:23:47 +0800
From: kernel test robot <lkp@...el.com>
To: Praveen Talari <praveen.talari@....qualcomm.com>,
	Andi Shyti <andi.shyti@...nel.org>, Rob Herring <robh@...nel.org>,
	Krzysztof Kozlowski <krzk@...nel.org>,
	Conor Dooley <conor+dt@...nel.org>,
	Mukesh Kumar Savaliya <mukesh.savaliya@....qualcomm.com>,
	Viken Dadhaniya <viken.dadhaniya@....qualcomm.com>,
	Bjorn Andersson <andersson@...nel.org>,
	Konrad Dybcio <konradybcio@...nel.org>,
	linux-arm-msm@...r.kernel.org, linux-i2c@...r.kernel.org,
	devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Cc: llvm@...ts.linux.dev, oe-kbuild-all@...ts.linux.dev,
	psodagud@...cinc.com, djaggi@...cinc.com, quic_msavaliy@...cinc.com,
	quic_vtanuku@...cinc.com, quic_arandive@...cinc.com,
	quic_shazhuss@...cinc.com
Subject: Re: [PATCH v1 03/12] soc: qcom: geni-se: Introduce helper API for
 resource initialization

Hi Praveen,

kernel test robot noticed the following build errors:

[auto build test ERROR on d724c6f85e80a23ed46b7ebc6e38b527c09d64f5]

url:    https://github.com/intel-lab-lkp/linux/commits/Praveen-Talari/soc-qcom-geni-se-Refactor-geni_icc_get-and-make-qup-memory-ICC-path-optional/20251122-130449
base:   d724c6f85e80a23ed46b7ebc6e38b527c09d64f5
patch link:    https://lore.kernel.org/r/20251122050018.283669-4-praveen.talari%40oss.qualcomm.com
patch subject: [PATCH v1 03/12] soc: qcom: geni-se: Introduce helper API for resource initialization
config: loongarch-randconfig-002-20251123 (https://download.01.org/0day-ci/archive/20251123/202511231819.jiLYo6Fl-lkp@intel.com/config)
compiler: clang version 18.1.8 (https://github.com/llvm/llvm-project 3b5b5c1ec4a3095ab096dd780e84d7ab81f3d7ff)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20251123/202511231819.jiLYo6Fl-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/202511231819.jiLYo6Fl-lkp@intel.com/

All errors (new ones prefixed by >>):

>> drivers/soc/qcom/qcom-geni-se.c:1046:10: error: incompatible pointer to integer conversion returning 'void *' from a function with result type 'int' [-Wint-conversion]
    1046 |                 return ERR_PTR(ret);
         |                        ^~~~~~~~~~~~
   1 error generated.


vim +1046 drivers/soc/qcom/qcom-geni-se.c

  1015	
  1016	/**
  1017	 * geni_se_resources_init() - Initialize resources for a GENI SE device.
  1018	 * @se: Pointer to the geni_se structure representing the GENI SE device.
  1019	 *
  1020	 * This function initializes various resources required by the GENI Serial Engine
  1021	 * (SE) device, including clock resources (core and SE clocks), interconnect
  1022	 * paths for communication.
  1023	 * It retrieves optional and mandatory clock resources, adds an OF-based
  1024	 * operating performance point (OPP) table, and sets up interconnect paths
  1025	 * with default bandwidths. The function also sets a flag (`has_opp`) to
  1026	 * indicate whether OPP support is available for the device.
  1027	 *
  1028	 * Return: 0 on success, or a negative errno on failure.
  1029	 */
  1030	int geni_se_resources_init(struct geni_se *se)
  1031	{
  1032		int ret;
  1033	
  1034		se->core_clk = devm_clk_get_optional(se->dev, "core");
  1035		if (IS_ERR(se->core_clk))
  1036			return dev_err_probe(se->dev, PTR_ERR(se->core_clk),
  1037					     "Failed to get optional core clk\n");
  1038	
  1039		se->clk = devm_clk_get(se->dev, "se");
  1040		if (IS_ERR(se->clk) && !has_acpi_companion(se->dev))
  1041			return dev_err_probe(se->dev, PTR_ERR(se->clk),
  1042					     "Failed to get SE clk\n");
  1043	
  1044		ret = devm_pm_opp_set_clkname(se->dev, "se");
  1045		if (ret)
> 1046			return ERR_PTR(ret);
  1047	
  1048		ret = devm_pm_opp_of_add_table(se->dev);
  1049		if (ret && ret != -ENODEV)
  1050			return dev_err_probe(se->dev, ret, "Failed to add OPP table\n");
  1051	
  1052		se->has_opp = (ret == 0);
  1053	
  1054		ret = geni_icc_get(se, "qup-memory");
  1055		if (ret)
  1056			return ret;
  1057	
  1058		return geni_icc_set_bw_ab(se, GENI_DEFAULT_BW, GENI_DEFAULT_BW, GENI_DEFAULT_BW);
  1059	}
  1060	EXPORT_SYMBOL_GPL(geni_se_resources_init);
  1061	

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