netdev - Re: [PATCH net-next v2] drivers/net: get rid of unnecessary initializations in .get

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:	Fri, 16 Oct 2015 02:36:55 +0800
From:	kbuild test robot <lkp@...el.com>
To:	Ivan Vecera <ivecera@...hat.com>
Cc:	kbuild-all@...org, netdev@...r.kernel.org, bwh@...nel.org
Subject: Re: [PATCH net-next v2] drivers/net: get rid of unnecessary
 initializations in .get_drvinfo()

Hi Ivan,

[auto build test WARNING on net-next/master -- if it's inappropriate base, please suggest rules for selecting the more suitable base]

url:    https://github.com/0day-ci/linux/commits/Ivan-Vecera/drivers-net-get-rid-of-unnecessary-initializations-in-get_drvinfo/20151016-010035
config: x86_64-randconfig-x011-10130227 (attached as .config)
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All warnings (new ones prefixed by >>):

   drivers/net/usb/sr9800.c: In function 'sr_get_drvinfo':
>> drivers/net/usb/sr9800.c:473:17: warning: unused variable 'dev' [-Wunused-variable]
     struct usbnet *dev = netdev_priv(net);
                    ^

vim +/dev +473 drivers/net/usb/sr9800.c

19a38d8e Liu Junliang 2014-02-10  457  
19a38d8e Liu Junliang 2014-02-10  458  	eeprom->magic = SR_EEPROM_MAGIC;
19a38d8e Liu Junliang 2014-02-10  459  
19a38d8e Liu Junliang 2014-02-10  460  	/* sr9800 returns 2 bytes from eeprom on read */
19a38d8e Liu Junliang 2014-02-10  461  	for (i = 0; i < eeprom->len / 2; i++) {
19a38d8e Liu Junliang 2014-02-10  462  		ret = sr_read_cmd(dev, SR_CMD_READ_EEPROM, eeprom->offset + i,
19a38d8e Liu Junliang 2014-02-10  463  				  0, 2, &ebuf[i]);
19a38d8e Liu Junliang 2014-02-10  464  		if (ret < 0)
19a38d8e Liu Junliang 2014-02-10  465  			return -EINVAL;
19a38d8e Liu Junliang 2014-02-10  466  	}
19a38d8e Liu Junliang 2014-02-10  467  	return 0;
19a38d8e Liu Junliang 2014-02-10  468  }
19a38d8e Liu Junliang 2014-02-10  469  
19a38d8e Liu Junliang 2014-02-10  470  static void sr_get_drvinfo(struct net_device *net,
19a38d8e Liu Junliang 2014-02-10  471  				 struct ethtool_drvinfo *info)
19a38d8e Liu Junliang 2014-02-10  472  {
19a38d8e Liu Junliang 2014-02-10 @473  	struct usbnet *dev = netdev_priv(net);
19a38d8e Liu Junliang 2014-02-10  474  
19a38d8e Liu Junliang 2014-02-10  475  	/* Inherit standard device info */
19a38d8e Liu Junliang 2014-02-10  476  	usbnet_get_drvinfo(net, info);
19a38d8e Liu Junliang 2014-02-10  477  	strncpy(info->driver, DRIVER_NAME, sizeof(info->driver));
19a38d8e Liu Junliang 2014-02-10  478  	strncpy(info->version, DRIVER_VERSION, sizeof(info->version));
19a38d8e Liu Junliang 2014-02-10  479  }
19a38d8e Liu Junliang 2014-02-10  480  
19a38d8e Liu Junliang 2014-02-10  481  static u32 sr_get_link(struct net_device *net)

:::::: The code at line 473 was first introduced by commit
:::::: 19a38d8e0aa33b4f4d11d3b4baa902ad169daa80 USB2NET : SR9800 : One chip USB2.0 USB2NET SR9800 Device Driver Support

:::::: TO: Liu Junliang <liujunliang_ljl@....com>
:::::: CC: David S. Miller <davem@...emloft.net>

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