linux-kernel - Re: [PATCH 1/2] power: move power supply drivers to power/supply

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]

Message-ID: <201606182052.MDTWFvcT%fengguang.wu@intel.com>
Date:	Sat, 18 Jun 2016 20:18:59 +0800
From:	kbuild test robot <lkp@...el.com>
To:	Sebastian Reichel <sre@...nel.org>
Cc:	kbuild-all@...org, linux-kernel@...r.kernel.org,
	Sebastian Reichel <sre@...nel.org>,
	Dmitry Eremin-Solenikov <dbaryshkov@...il.com>,
	David Woodhouse <dwmw2@...radead.org>,
	Rob Herring <robh+dt@...nel.org>,
	Pawel Moll <pawel.moll@....com>,
	Mark Rutland <mark.rutland@....com>,
	Ian Campbell <ijc+devicetree@...lion.org.uk>,
	Kumar Gala <galak@...eaurora.org>, linux-pm@...r.kernel.org,
	devicetree@...r.kernel.org
Subject: Re: [PATCH 1/2] power: move power supply drivers to power/supply

Hi,

[auto build test WARNING on battery/master]
[also build test WARNING on v4.7-rc3 next-20160617]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Sebastian-Reichel/Move-power-supply-subsystem-to-drivers-power-supply/20160617-213931
base:   git://git.infradead.org/battery-2.6.git master
config: s390-allyesconfig (attached as .config)
compiler: s390x-linux-gnu-gcc (Debian 5.3.1-8) 5.3.1 20160205
reproduce:
        wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        make.cross ARCH=s390 

All warnings (new ones prefixed by >>):

   drivers/power/supply/da9150-fg.c: In function 'da9150_fg_read_attr.isra.1':
>> drivers/power/supply/da9150-fg.c:108:1: warning: 'da9150_fg_read_attr.isra.1' uses dynamic stack allocation
    }
    ^

vim +108 drivers/power/supply/da9150-fg.c

a419b4fd drivers/power/da9150-fg.c Adam Thomson 2015-10-07   92  static u32 da9150_fg_read_attr(struct da9150_fg *fg, u8 code, u8 size)
a419b4fd drivers/power/da9150-fg.c Adam Thomson 2015-10-07   93  
a419b4fd drivers/power/da9150-fg.c Adam Thomson 2015-10-07   94  {
a419b4fd drivers/power/da9150-fg.c Adam Thomson 2015-10-07   95  	u8 buf[size];
a419b4fd drivers/power/da9150-fg.c Adam Thomson 2015-10-07   96  	u8 read_addr;
a419b4fd drivers/power/da9150-fg.c Adam Thomson 2015-10-07   97  	u32 res = 0;
a419b4fd drivers/power/da9150-fg.c Adam Thomson 2015-10-07   98  	int i;
a419b4fd drivers/power/da9150-fg.c Adam Thomson 2015-10-07   99  
a419b4fd drivers/power/da9150-fg.c Adam Thomson 2015-10-07  100  	/* Set QIF code (READ mode) */
a419b4fd drivers/power/da9150-fg.c Adam Thomson 2015-10-07  101  	read_addr = (code & DA9150_QIF_CODE_MASK) | DA9150_QIF_READ;
a419b4fd drivers/power/da9150-fg.c Adam Thomson 2015-10-07  102  
a419b4fd drivers/power/da9150-fg.c Adam Thomson 2015-10-07  103  	da9150_read_qif(fg->da9150, read_addr, size, buf);
a419b4fd drivers/power/da9150-fg.c Adam Thomson 2015-10-07  104  	for (i = 0; i < size; ++i)
a419b4fd drivers/power/da9150-fg.c Adam Thomson 2015-10-07  105  		res |= (buf[i] << (i * DA9150_QIF_BYTE_SIZE));
a419b4fd drivers/power/da9150-fg.c Adam Thomson 2015-10-07  106  
a419b4fd drivers/power/da9150-fg.c Adam Thomson 2015-10-07  107  	return res;
a419b4fd drivers/power/da9150-fg.c Adam Thomson 2015-10-07 @108  }
a419b4fd drivers/power/da9150-fg.c Adam Thomson 2015-10-07  109  
a419b4fd drivers/power/da9150-fg.c Adam Thomson 2015-10-07  110  static void da9150_fg_write_attr(struct da9150_fg *fg, u8 code, u8 size,
a419b4fd drivers/power/da9150-fg.c Adam Thomson 2015-10-07  111  				 u32 val)
a419b4fd drivers/power/da9150-fg.c Adam Thomson 2015-10-07  112  
a419b4fd drivers/power/da9150-fg.c Adam Thomson 2015-10-07  113  {
a419b4fd drivers/power/da9150-fg.c Adam Thomson 2015-10-07  114  	u8 buf[size];
a419b4fd drivers/power/da9150-fg.c Adam Thomson 2015-10-07  115  	u8 write_addr;
a419b4fd drivers/power/da9150-fg.c Adam Thomson 2015-10-07  116  	int i;

:::::: The code at line 108 was first introduced by commit
:::::: a419b4fd9138c6cd5049d14813e7d94e596e4540 power: Add support for DA9150 Fuel-Gauge

:::::: TO: Adam Thomson <Adam.Thomson.Opensource@...semi.com>
:::::: CC: Lee Jones <lee.jones@...aro.org>

---
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" (41729 bytes)