All of lore.kernel.org
 help / color / mirror / Atom feed
* [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
@ 2022-07-31 11:56 kernel test robot
  0 siblings, 0 replies; only message in thread
From: kernel test robot @ 2022-07-31 11:56 UTC (permalink / raw)
  To: Dmitry Baryshkov; +Cc: llvm, kbuild-all, linux-kernel, Steev Klimaszewski

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

^ permalink raw reply	[flat|nested] only message in thread
only message in thread, other threads:[~2022-07-31 11:57 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-07-31 11:56 [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 kernel test robot

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.