lkml.org 
[lkml]   [2022]   [Jul]   [31]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
Subject[steev:linux-v5.19.0-rc8-tests 146/182] drivers/gpu/drm/msm/dsi/dsi_host.c:1997:14: warning: variable 'device_node' is uninitialized when used here
tree:   https://github.com/steev/linux linux-v5.19.0-rc8-tests
head: 171125613e081f0d0d9aeb710dc7469ce2c6a219
commit: d99732349a84f6046d58df883c6c68fd7351faa9 [146/182] drm/msm/dsi: switch to DRM_PANEL_BRIDGE
config: arm64-buildonly-randconfig-r005-20220731 (https://download.01.org/0day-ci/archive/20220731/202207311957.wVEYCh2c-lkp@intel.com/config)
compiler: clang version 16.0.0 (https://github.com/llvm/llvm-project 52cd00cabf479aa7eb6dbb063b7ba41ea57bce9e)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install arm64 cross compiling tool for clang build
# apt-get install binutils-aarch64-linux-gnu
# https://github.com/steev/linux/commit/d99732349a84f6046d58df883c6c68fd7351faa9
git remote add steev https://github.com/steev/linux
git fetch --no-tags steev linux-v5.19.0-rc8-tests
git checkout d99732349a84f6046d58df883c6c68fd7351faa9
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=arm64 SHELL=/bin/bash drivers/gpu/drm/msm/

If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@intel.com>

All warnings (new ones prefixed by >>):

>> drivers/gpu/drm/msm/dsi/dsi_host.c:1997:14: warning: variable 'device_node' is uninitialized when used here [-Wuninitialized]
of_node_put(device_node);
^~~~~~~~~~~
drivers/gpu/drm/msm/dsi/dsi_host.c:1964:44: note: initialize the variable 'device_node' to silence this warning
struct device_node *endpoint, *device_node;
^
= NULL
1 warning generated.
--
>> drivers/gpu/drm/msm/dsi/dsi_manager.c:533:7: warning: variable 'connector' is used uninitialized whenever 'if' condition is false [-Wsometimes-uninitialized]
if (ret < 0)
^~~~~~~
drivers/gpu/drm/msm/dsi/dsi_manager.c:551:9: note: uninitialized use occurs here
return connector;
^~~~~~~~~
drivers/gpu/drm/msm/dsi/dsi_manager.c:533:3: note: remove the 'if' if its condition is always true
if (ret < 0)
^~~~~~~~~~~~
drivers/gpu/drm/msm/dsi/dsi_manager.c:509:33: note: initialize the variable 'connector' to silence this warning
struct drm_connector *connector;
^
= NULL
1 warning generated.


vim +/device_node +1997 drivers/gpu/drm/msm/dsi/dsi_host.c

b9080324d6ca24 Vinod Koul 2022-04-06 1959
f7009d266d8b2f Archit Taneja 2015-06-25 1960 static int dsi_host_parse_dt(struct msm_dsi_host *msm_host)
f7009d266d8b2f Archit Taneja 2015-06-25 1961 {
f7009d266d8b2f Archit Taneja 2015-06-25 1962 struct device *dev = &msm_host->pdev->dev;
f7009d266d8b2f Archit Taneja 2015-06-25 1963 struct device_node *np = dev->of_node;
a9ddac9c576571 Archit Taneja 2015-08-03 1964 struct device_node *endpoint, *device_node;
a1b1a4f7e4c3cf Archit Taneja 2017-01-04 1965 int ret = 0;
f7009d266d8b2f Archit Taneja 2015-06-25 1966
f7009d266d8b2f Archit Taneja 2015-06-25 1967 /*
b9ac76f6ac9841 Archit Taneja 2016-04-27 1968 * Get the endpoint of the output port of the DSI host. In our case,
b9ac76f6ac9841 Archit Taneja 2016-04-27 1969 * this is mapped to port number with reg = 1. Don't return an error if
b9ac76f6ac9841 Archit Taneja 2016-04-27 1970 * the remote endpoint isn't defined. It's possible that there is
b9ac76f6ac9841 Archit Taneja 2016-04-27 1971 * nothing connected to the dsi output.
f7009d266d8b2f Archit Taneja 2015-06-25 1972 */
b9ac76f6ac9841 Archit Taneja 2016-04-27 1973 endpoint = of_graph_get_endpoint_by_regs(np, 1, -1);
f7009d266d8b2f Archit Taneja 2015-06-25 1974 if (!endpoint) {
6a41da17e87dee Mamta Shukla 2018-10-20 1975 DRM_DEV_DEBUG(dev, "%s: no endpoint\n", __func__);
f7009d266d8b2f Archit Taneja 2015-06-25 1976 return 0;
f7009d266d8b2f Archit Taneja 2015-06-25 1977 }
f7009d266d8b2f Archit Taneja 2015-06-25 1978
26f7d1f4d9ab10 Archit Taneja 2016-02-25 1979 ret = dsi_host_parse_lane_data(msm_host, endpoint);
26f7d1f4d9ab10 Archit Taneja 2016-02-25 1980 if (ret) {
6a41da17e87dee Mamta Shukla 2018-10-20 1981 DRM_DEV_ERROR(dev, "%s: invalid lane configuration %d\n",
26f7d1f4d9ab10 Archit Taneja 2016-02-25 1982 __func__, ret);
feb085ec8a3dc1 Sean Paul 2018-07-30 1983 ret = -EINVAL;
26f7d1f4d9ab10 Archit Taneja 2016-02-25 1984 goto err;
26f7d1f4d9ab10 Archit Taneja 2016-02-25 1985 }
26f7d1f4d9ab10 Archit Taneja 2016-02-25 1986
0c7df47f6a4c5d Archit Taneja 2015-10-14 1987 if (of_property_read_bool(np, "syscon-sfpb")) {
0c7df47f6a4c5d Archit Taneja 2015-10-14 1988 msm_host->sfpb = syscon_regmap_lookup_by_phandle(np,
0c7df47f6a4c5d Archit Taneja 2015-10-14 1989 "syscon-sfpb");
0c7df47f6a4c5d Archit Taneja 2015-10-14 1990 if (IS_ERR(msm_host->sfpb)) {
6a41da17e87dee Mamta Shukla 2018-10-20 1991 DRM_DEV_ERROR(dev, "%s: failed to get sfpb regmap\n",
0c7df47f6a4c5d Archit Taneja 2015-10-14 1992 __func__);
26f7d1f4d9ab10 Archit Taneja 2016-02-25 1993 ret = PTR_ERR(msm_host->sfpb);
0c7df47f6a4c5d Archit Taneja 2015-10-14 1994 }
0c7df47f6a4c5d Archit Taneja 2015-10-14 1995 }
0c7df47f6a4c5d Archit Taneja 2015-10-14 1996
26f7d1f4d9ab10 Archit Taneja 2016-02-25 @1997 of_node_put(device_node);
26f7d1f4d9ab10 Archit Taneja 2016-02-25 1998
26f7d1f4d9ab10 Archit Taneja 2016-02-25 1999 err:
26f7d1f4d9ab10 Archit Taneja 2016-02-25 2000 of_node_put(endpoint);
26f7d1f4d9ab10 Archit Taneja 2016-02-25 2001
26f7d1f4d9ab10 Archit Taneja 2016-02-25 2002 return ret;
f7009d266d8b2f Archit Taneja 2015-06-25 2003 }
f7009d266d8b2f Archit Taneja 2015-06-25 2004

:::::: The code at line 1997 was first introduced by commit
:::::: 26f7d1f4d9ab108b28b8d0f340b98941073c6f4d drm/msm/dsi: Parse DSI lanes via DT

:::::: TO: Archit Taneja <architt@codeaurora.org>
:::::: CC: Rob Clark <robdclark@gmail.com>

--
0-DAY CI Kernel Test Service
https://01.org/lkp

\
 
 \ /
  Last update: 2022-07-31 13:57    [W:0.503 / U:0.084 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site