linux-kernel - Re: [PATCH] reset: remove unused device pointer from struct reset

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] [day] [month] [year] [list]

Date:	Wed, 18 Nov 2015 21:54:40 +0800
From:	kbuild test robot <lkp@...el.com>
To:	Philipp Zabel <p.zabel@...gutronix.de>
Cc:	kbuild-all@...org, linux-kernel@...r.kernel.org,
	kernel@...gutronix.de, Philipp Zabel <p.zabel@...gutronix.de>
Subject: Re: [PATCH] reset: remove unused device pointer from struct
 reset_control

Hi Philipp,

[auto build test WARNING on: v4.4-rc1]
[also build test WARNING on: next-20151118]

url:    https://github.com/0day-ci/linux/commits/Philipp-Zabel/reset-remove-unused-device-pointer-from-struct-reset_control/20151118-214443
config: x86_64-randconfig-x012-11181928 (attached as .config)
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All warnings (new ones prefixed by >>):

   drivers/reset/core.c: In function 'reset_control_get':
>> drivers/reset/core.c:217:24: warning: unused variable 'rstc' [-Wunused-variable]
     struct reset_control *rstc;
                           ^

vim +/rstc +217 drivers/reset/core.c

61fc4131 Philipp Zabel 2012-11-19  201  
61fc4131 Philipp Zabel 2012-11-19  202  	return rstc;
61fc4131 Philipp Zabel 2012-11-19  203  }
fc0a5921 Maxime Ripard 2013-12-20  204  EXPORT_SYMBOL_GPL(of_reset_control_get);
fc0a5921 Maxime Ripard 2013-12-20  205  
fc0a5921 Maxime Ripard 2013-12-20  206  /**
fc0a5921 Maxime Ripard 2013-12-20  207   * reset_control_get - Lookup and obtain a reference to a reset controller.
fc0a5921 Maxime Ripard 2013-12-20  208   * @dev: device to be reset by the controller
fc0a5921 Maxime Ripard 2013-12-20  209   * @id: reset line name
fc0a5921 Maxime Ripard 2013-12-20  210   *
fc0a5921 Maxime Ripard 2013-12-20  211   * Returns a struct reset_control or IS_ERR() condition containing errno.
fc0a5921 Maxime Ripard 2013-12-20  212   *
fc0a5921 Maxime Ripard 2013-12-20  213   * Use of id names is optional.
fc0a5921 Maxime Ripard 2013-12-20  214   */
fc0a5921 Maxime Ripard 2013-12-20  215  struct reset_control *reset_control_get(struct device *dev, const char *id)
fc0a5921 Maxime Ripard 2013-12-20  216  {
fc0a5921 Maxime Ripard 2013-12-20 @217  	struct reset_control *rstc;
fc0a5921 Maxime Ripard 2013-12-20  218  
fc0a5921 Maxime Ripard 2013-12-20  219  	if (!dev)
fc0a5921 Maxime Ripard 2013-12-20  220  		return ERR_PTR(-EINVAL);
fc0a5921 Maxime Ripard 2013-12-20  221  
5d79cb8b Philipp Zabel 2015-11-18  222  	return of_reset_control_get(dev->of_node, id);
fc0a5921 Maxime Ripard 2013-12-20  223  }
61fc4131 Philipp Zabel 2012-11-19  224  EXPORT_SYMBOL_GPL(reset_control_get);
61fc4131 Philipp Zabel 2012-11-19  225  

:::::: The code at line 217 was first introduced by commit
:::::: fc0a5921561c71be2c334a335c1680f7930434d7 reset: Add of_reset_control_get

:::::: TO: Maxime Ripard <maxime.ripard@...e-electrons.com>
:::::: CC: Philipp Zabel <p.zabel@...gutronix.de>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Download attachment ".config.gz" of type "application/octet-stream" (28235 bytes)