Re: [PATCH net-next 3/6] net: remove dsa.h include from linux/netdevice.h

From: kbuild test robot
Date: Sat Oct 10 2015 - 19:36:41 EST

Next message: kbuild test robot: "Re: [PATCH v8 2/9] Input: goodix - reset device at init"
Previous message: Arnaldo Carvalho de Melo: "Re: [PATCH 1/5] perf: unwind: fix unw_word_t pointer casts"
In reply to: huangdaode: "Re: [PATCH net-next 3/6] net: remove dsa.h include from linux/netdevice.h"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

Hi Vivien,

[auto build test ERROR on net-next/master -- if it's inappropriate base, please ignore]

config: x86_64-randconfig-n0-10110700 (attached as .config)
reproduce:
# save the attached .config to linux build tree
make ARCH=x86_64

All errors (new ones prefixed by >>):

drivers/net/usb/lan78xx.c: In function 'lan78xx_link_reset':
>> drivers/net/usb/lan78xx.c:841:8: error: implicit declaration of function 'phy_read' [-Werror=implicit-function-declaration]
ret = phy_read(phydev, LAN88XX_INT_STS);
^
>> drivers/net/usb/lan78xx.c:850:2: error: implicit declaration of function 'phy_read_status' [-Werror=implicit-function-declaration]
phy_read_status(phydev);
^
>> drivers/net/usb/lan78xx.c:852:13: error: dereferencing pointer to incomplete type 'struct phy_device'
if (!phydev->link && dev->link_on) {
^
>> drivers/net/usb/lan78xx.c:867:3: error: implicit declaration of function 'phy_ethtool_gset' [-Werror=implicit-function-declaration]
phy_ethtool_gset(phydev, &ecmd);
^
drivers/net/usb/lan78xx.c: In function 'lan78xx_set_wol':
>> drivers/net/usb/lan78xx.c:1067:2: error: implicit declaration of function 'phy_ethtool_set_wol' [-Werror=implicit-function-declaration]
phy_ethtool_set_wol(netdev->phydev, wol);
^
drivers/net/usb/lan78xx.c: In function 'lan78xx_get_eee':
>> drivers/net/usb/lan78xx.c:1085:8: error: implicit declaration of function 'phy_ethtool_get_eee' [-Werror=implicit-function-declaration]
ret = phy_ethtool_get_eee(phydev, edata);
^
drivers/net/usb/lan78xx.c: In function 'lan78xx_set_eee':
>> drivers/net/usb/lan78xx.c:1127:3: error: implicit declaration of function 'phy_ethtool_set_eee' [-Werror=implicit-function-declaration]
phy_ethtool_set_eee(net->phydev, edata);
^
drivers/net/usb/lan78xx.c: In function 'lan78xx_nway_reset':
>> drivers/net/usb/lan78xx.c:1151:9: error: implicit declaration of function 'phy_start_aneg' [-Werror=implicit-function-declaration]
return phy_start_aneg(net->phydev);
^
drivers/net/usb/lan78xx.c: In function 'lan78xx_get_mdix_status':
>> drivers/net/usb/lan78xx.c:1183:2: error: implicit declaration of function 'phy_write' [-Werror=implicit-function-declaration]
phy_write(phydev, LAN88XX_EXT_PAGE_ACCESS, LAN88XX_EXT_PAGE_SPACE_1);
^
drivers/net/usb/lan78xx.c: In function 'lan78xx_set_settings':
>> drivers/net/usb/lan78xx.c:1275:8: error: implicit declaration of function 'phy_ethtool_sset' [-Werror=implicit-function-declaration]
ret = phy_ethtool_sset(phydev, cmd);
^
drivers/net/usb/lan78xx.c: In function 'lan78xx_ioctl':
>> drivers/net/usb/lan78xx.c:1315:9: error: implicit declaration of function 'phy_mii_ioctl' [-Werror=implicit-function-declaration]
return phy_mii_ioctl(netdev->phydev, rq, cmd);
^
drivers/net/usb/lan78xx.c: In function 'lan78xx_mdiobus_read':
>> drivers/net/usb/lan78xx.c:1374:31: error: dereferencing pointer to incomplete type 'struct mii_bus'
struct lan78xx_net *dev = bus->priv;
^
drivers/net/usb/lan78xx.c: In function 'lan78xx_mdio_init':
>> drivers/net/usb/lan78xx.c:1447:17: error: implicit declaration of function 'mdiobus_alloc' [-Werror=implicit-function-declaration]
dev->mdiobus = mdiobus_alloc();
^
drivers/net/usb/lan78xx.c:1447:15: warning: assignment makes pointer from integer without a cast [-Wint-conversion]
dev->mdiobus = mdiobus_alloc();
^
>> drivers/net/usb/lan78xx.c:1458:29: error: 'MII_BUS_ID_SIZE' undeclared (first use in this function)
snprintf(dev->mdiobus->id, MII_BUS_ID_SIZE, "usb-%03d:%03d",
^
drivers/net/usb/lan78xx.c:1458:29: note: each undeclared identifier is reported only once for each function it appears in
>> drivers/net/usb/lan78xx.c:1461:44: error: 'PHY_MAX_ADDR' undeclared (first use in this function)
dev->mdiobus->irq = kzalloc(sizeof(int) * PHY_MAX_ADDR, GFP_KERNEL);
^
>> drivers/net/usb/lan78xx.c:1469:26: error: 'PHY_IGNORE_INTERRUPT' undeclared (first use in this function)
dev->mdiobus->irq[i] = PHY_IGNORE_INTERRUPT;
^
>> drivers/net/usb/lan78xx.c:1479:8: error: implicit declaration of function 'mdiobus_register' [-Werror=implicit-function-declaration]
ret = mdiobus_register(dev->mdiobus);
^
>> drivers/net/usb/lan78xx.c:1490:2: error: implicit declaration of function 'mdiobus_free' [-Werror=implicit-function-declaration]
mdiobus_free(dev->mdiobus);
^
drivers/net/usb/lan78xx.c: In function 'lan78xx_remove_mdio':
>> drivers/net/usb/lan78xx.c:1496:2: error: implicit declaration of function 'mdiobus_unregister' [-Werror=implicit-function-declaration]
mdiobus_unregister(dev->mdiobus);
^
drivers/net/usb/lan78xx.c: In function 'lan78xx_phy_init':
>> drivers/net/usb/lan78xx.c:1511:11: error: implicit declaration of function 'phy_find_first' [-Werror=implicit-function-declaration]
phydev = phy_find_first(dev->mdiobus);
^
drivers/net/usb/lan78xx.c:1511:9: warning: assignment makes pointer from integer without a cast [-Wint-conversion]
phydev = phy_find_first(dev->mdiobus);
^

vim +/phy_read +841 drivers/net/usb/lan78xx.c

ce85e13ad Woojung.Huh@xxxxxxxxxxxxx 2015-09-16 835 struct phy_device *phydev = dev->net->phydev;
55d7de9de Woojung.Huh@xxxxxxxxxxxxx 2015-07-30 836 struct ethtool_cmd ecmd = { .cmd = ETHTOOL_GSET };
99c79eceb Geert Uytterhoeven 2015-09-04 837 int ladv, radv, ret;
55d7de9de Woojung.Huh@xxxxxxxxxxxxx 2015-07-30 838 u32 buf;
55d7de9de Woojung.Huh@xxxxxxxxxxxxx 2015-07-30 839
55d7de9de Woojung.Huh@xxxxxxxxxxxxx 2015-07-30 840 /* clear PHY interrupt status */
bdfba55e0 Woojung.Huh@xxxxxxxxxxxxx 2015-09-16 @841 ret = phy_read(phydev, LAN88XX_INT_STS);
55d7de9de Woojung.Huh@xxxxxxxxxxxxx 2015-07-30 842 if (unlikely(ret < 0))
55d7de9de Woojung.Huh@xxxxxxxxxxxxx 2015-07-30 843 return -EIO;
55d7de9de Woojung.Huh@xxxxxxxxxxxxx 2015-07-30 844
55d7de9de Woojung.Huh@xxxxxxxxxxxxx 2015-07-30 845 /* clear LAN78xx interrupt status */
55d7de9de Woojung.Huh@xxxxxxxxxxxxx 2015-07-30 846 ret = lan78xx_write_reg(dev, INT_STS, INT_STS_PHY_INT_);
55d7de9de Woojung.Huh@xxxxxxxxxxxxx 2015-07-30 847 if (unlikely(ret < 0))
55d7de9de Woojung.Huh@xxxxxxxxxxxxx 2015-07-30 848 return -EIO;
55d7de9de Woojung.Huh@xxxxxxxxxxxxx 2015-07-30 849
ce85e13ad Woojung.Huh@xxxxxxxxxxxxx 2015-09-16 @850 phy_read_status(phydev);
ce85e13ad Woojung.Huh@xxxxxxxxxxxxx 2015-09-16 851
ce85e13ad Woojung.Huh@xxxxxxxxxxxxx 2015-09-16 @852 if (!phydev->link && dev->link_on) {
55d7de9de Woojung.Huh@xxxxxxxxxxxxx 2015-07-30 853 dev->link_on = false;
55d7de9de Woojung.Huh@xxxxxxxxxxxxx 2015-07-30 854 netif_carrier_off(dev->net);
55d7de9de Woojung.Huh@xxxxxxxxxxxxx 2015-07-30 855
55d7de9de Woojung.Huh@xxxxxxxxxxxxx 2015-07-30 856 /* reset MAC */
55d7de9de Woojung.Huh@xxxxxxxxxxxxx 2015-07-30 857 ret = lan78xx_read_reg(dev, MAC_CR, &buf);
55d7de9de Woojung.Huh@xxxxxxxxxxxxx 2015-07-30 858 if (unlikely(ret < 0))
55d7de9de Woojung.Huh@xxxxxxxxxxxxx 2015-07-30 859 return -EIO;
55d7de9de Woojung.Huh@xxxxxxxxxxxxx 2015-07-30 860 buf |= MAC_CR_RST_;
55d7de9de Woojung.Huh@xxxxxxxxxxxxx 2015-07-30 861 ret = lan78xx_write_reg(dev, MAC_CR, buf);
55d7de9de Woojung.Huh@xxxxxxxxxxxxx 2015-07-30 862 if (unlikely(ret < 0))
55d7de9de Woojung.Huh@xxxxxxxxxxxxx 2015-07-30 863 return -EIO;
ce85e13ad Woojung.Huh@xxxxxxxxxxxxx 2015-09-16 864 } else if (phydev->link && !dev->link_on) {
55d7de9de Woojung.Huh@xxxxxxxxxxxxx 2015-07-30 865 dev->link_on = true;
55d7de9de Woojung.Huh@xxxxxxxxxxxxx 2015-07-30 866
ce85e13ad Woojung.Huh@xxxxxxxxxxxxx 2015-09-16 @867 phy_ethtool_gset(phydev, &ecmd);
55d7de9de Woojung.Huh@xxxxxxxxxxxxx 2015-07-30 868
bdfba55e0 Woojung.Huh@xxxxxxxxxxxxx 2015-09-16 869 ret = phy_read(phydev, LAN88XX_INT_STS);
55d7de9de Woojung.Huh@xxxxxxxxxxxxx 2015-07-30 870

:::::: The code at line 841 was first introduced by commit
:::::: bdfba55e0d541a9547d737573ae11db7ed72e2bb lan78xx: Remove phy defines in lan78xx.h and use defines in include/linux/microchipphy.h

:::::: TO: Woojung.Huh@xxxxxxxxxxxxx <Woojung.Huh@xxxxxxxxxxxxx>
:::::: CC: David S. Miller <davem@xxxxxxxxxxxxx>

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

Attachment: .config.gz
Description: Binary data

Next message: kbuild test robot: "Re: [PATCH v8 2/9] Input: goodix - reset device at init"
Previous message: Arnaldo Carvalho de Melo: "Re: [PATCH 1/5] perf: unwind: fix unw_word_t pointer casts"
In reply to: huangdaode: "Re: [PATCH net-next 3/6] net: remove dsa.h include from linux/netdevice.h"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]