Re: [PATCH] phy, leds: add support for led triggers on phy link state change

From: kbuild test robot
Date: Wed Aug 17 2016 - 11:25:03 EST


Hi Josh,

[auto build test ERROR on net-next/master]
[also build test ERROR on v4.8-rc2 next-20160817]
[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/Zach-Brown/phy-leds-add-support-for-led-triggers-on-phy-link-state-change/20160817-221908
config: i386-allmodconfig (attached as .config)
compiler: gcc-6 (Debian 6.1.1-9) 6.1.1 20160705
reproduce:
# save the attached .config to linux build tree
make ARCH=i386

All errors (new ones prefixed by >>):

In file included from drivers/net/ethernet/marvell/skge.c:50:0:
>> drivers/net/ethernet/marvell/skge.h:666:2: error: redeclaration of enumerator 'LED_OFF'
LED_OFF = 1<<0, /* switch LED off */
^~~~~~~
In file included from include/linux/phy_led_triggers.h:20:0,
from include/linux/phy.h:28,
from include/net/dsa.h:19,
from include/linux/netdevice.h:44,
from drivers/net/ethernet/marvell/skge.c:32:
include/linux/leds.h:29:2: note: previous definition of 'LED_OFF' was here
LED_OFF = 0,
^~~~~~~
--
>> drivers/staging/rtl8712/rtl8712_led.c:55:2: error: redeclaration of enumerator 'LED_OFF'
LED_OFF = 2,
^~~~~~~
In file included from include/linux/phy_led_triggers.h:20:0,
from include/linux/phy.h:28,
from include/net/dsa.h:19,
from include/linux/netdevice.h:44,
from drivers/staging/rtl8712/osdep_service.h:38,
from drivers/staging/rtl8712/drv_types.h:33,
from drivers/staging/rtl8712/rtl8712_led.c:29:
include/linux/leds.h:29:2: note: previous definition of 'LED_OFF' was here
LED_OFF = 0,
^~~~~~~

vim +/LED_OFF +666 drivers/net/ethernet/marvell/skge.h

baef58b1 drivers/net/skge.h Stephen Hemminger 2005-05-12 660 enum {
baef58b1 drivers/net/skge.h Stephen Hemminger 2005-05-12 661 LED_BLK_ON = 1<<5, /* Link LED Blinking On */
baef58b1 drivers/net/skge.h Stephen Hemminger 2005-05-12 662 LED_BLK_OFF = 1<<4, /* Link LED Blinking Off */
baef58b1 drivers/net/skge.h Stephen Hemminger 2005-05-12 663 LED_SYNC_ON = 1<<3, /* Use Sync Wire to switch LED */
baef58b1 drivers/net/skge.h Stephen Hemminger 2005-05-12 664 LED_SYNC_OFF = 1<<2, /* Disable Sync Wire Input */
baef58b1 drivers/net/skge.h Stephen Hemminger 2005-05-12 665 LED_ON = 1<<1, /* switch LED on */
baef58b1 drivers/net/skge.h Stephen Hemminger 2005-05-12 @666 LED_OFF = 1<<0, /* switch LED off */
baef58b1 drivers/net/skge.h Stephen Hemminger 2005-05-12 667 };
baef58b1 drivers/net/skge.h Stephen Hemminger 2005-05-12 668
2770b517 drivers/net/skge.h Stephen Hemminger 2005-12-14 669 /* Receive GMAC FIFO (YUKON) */

:::::: The code at line 666 was first introduced by commit
:::::: baef58b1b09ac0e9339e021144b921560482c8bd [netdrvr] new driver skge, for SysKonnect cards

:::::: TO: Stephen Hemminger <shemminger@xxxxxxxx>
:::::: CC: Jeff Garzik <jgarzik@xxxxxxxxx>

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

Attachment: .config.gz
Description: Binary data