netdev - Re: [PATCH v2 net-next 10/12] net: ethernet: Add helper for set

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

Message-ID: <201809112149.kpfUKi2C%fengguang.wu@intel.com>
Date:   Tue, 11 Sep 2018 21:04:17 +0800
From:   kbuild test robot <lkp@...el.com>
To:     Andrew Lunn <andrew@...n.ch>
Cc:     kbuild-all@...org, David Miller <davem@...emloft.net>,
        netdev <netdev@...r.kernel.org>,
        Florian Fainelli <f.fainelli@...il.com>,
        Andrew Lunn <andrew@...n.ch>
Subject: Re: [PATCH v2 net-next 10/12] net: ethernet: Add helper for
 set_pauseparam for Asym Pause

Hi Andrew,

I love your patch! Perhaps something to improve:

[auto build test WARNING on net-next/master]

url:    https://github.com/0day-ci/linux/commits/Andrew-Lunn/Preparing-for-phylib-limkmodes/20180911-204149
config: x86_64-randconfig-x011-201836 (attached as .config)
compiler: gcc-7 (Debian 7.3.0-1) 7.3.0
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All warnings (new ones prefixed by >>):

   drivers/net/phy/phy_device.c: In function 'phy_set_asym_pause':
>> drivers/net/phy/phy_device.c:1838:11: warning: 'return' with a value, in function returning void
       return phy_start_aneg(phydev);
              ^~~~~~~~~~~~~~~~~~~~~~
   drivers/net/phy/phy_device.c:1824:6: note: declared here
    void phy_set_asym_pause(struct phy_device *phydev, bool rx, bool tx)
         ^~~~~~~~~~~~~~~~~~

vim +/return +1838 drivers/net/phy/phy_device.c

  1812	
  1813	/**
  1814	 * phy_set_asym_pause - Configure Pause and Asym Pause
  1815	 * @phydev: target phy_device struct
  1816	 * @rx: Receiver Pause is supported
  1817	 * @tx: Transmit Pause is supported
  1818	 *
  1819	 * Description: Configure advertised Pause support depending on if
  1820	 * transmit and receiver pause is supported. If there has been a
  1821	 * change in adverting, trigger a new autoneg. Generally called from
  1822	 * the set_pauseparam .ndo.
  1823	 */
  1824	void phy_set_asym_pause(struct phy_device *phydev, bool rx, bool tx)
  1825	{
  1826		u16 oldadv = phydev->advertising;
  1827		u16 newadv = oldadv &= ~(SUPPORTED_Pause | SUPPORTED_Asym_Pause);
  1828	
  1829		if (rx)
  1830			newadv |= SUPPORTED_Pause | SUPPORTED_Asym_Pause;
  1831		if (tx)
  1832			newadv ^= SUPPORTED_Asym_Pause;
  1833	
  1834		if (oldadv != newadv) {
  1835			phydev->advertising = newadv;
  1836	
  1837			if (phydev->autoneg)
> 1838				return phy_start_aneg(phydev);
  1839		}
  1840	}
  1841	EXPORT_SYMBOL(phy_set_asym_pause);
  1842	

---
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/gzip" (33600 bytes)