Hi "Marek,
[FYI, it's a private test report for your RFC patch.]
[auto build test WARNING on net-next/master]
url:
https://github.com/0day-ci/linux/commits/Marek-Beh-n/Add-support-for-LEDs...
base:
https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git
59632b220f2d61df274ed3a14a204e941051fdad
config: alpha-allyesconfig (attached as .config)
compiler: alpha-linux-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
wget
https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O
~/bin/make.cross
chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=alpha
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
drivers/net/phy/marvell.c: In function 'marvell_led_set_regval':
> drivers/net/phy/marvell.c:775:6: warning: variable 'reg'
set but not used [-Wunused-but-set-variable]
775 | int reg;
| ^~~
drivers/net/phy/marvell.c: In function '_marvell_led_brightness_set':
> drivers/net/phy/marvell.c:806:6: warning: variable 'led'
set but not used [-Wunused-but-set-variable]
806 | int led;
| ^~~
vim +/reg +775 drivers/net/phy/marvell.c
771
772 static int marvell_led_set_regval(struct phy_device *phydev, int led, u16 val)
773 {
774 u16 mask;
775 int reg;
776
777 switch (led) {
778 case 0 ... 3:
779 reg = MII_PHY_LED_CTRL;
780 break;
781 case 4 ... 5:
782 reg = MII_PHY_LED45_CTRL;
783 break;
784 default:
785 return -EINVAL;
786 }
787
788 val <<= (led % 4) * 4;
789 mask = 0xf << ((led % 4) * 4);
790
791 return phy_modify_paged(phydev, MII_MARVELL_LED_PAGE, MII_PHY_LED_CTRL, mask,
val);
792 }
793
794 static inline int marvell_led_index(struct led_classdev *cdev)
795 {
796 struct marvell_priv *priv = to_phy_device(cdev->dev->parent)->priv;
797
798 return cdev - &priv->leds[0];
799 }
800
801 static int _marvell_led_brightness_set(struct led_classdev *cdev, enum
led_brightness brightness,
802 bool check_trigger)
803 {
804 struct phy_device *phydev = to_phy_device(cdev->dev->parent);
805 struct marvell_priv *priv = phydev->priv;
806 int led;
807 u8 val;
808
809 /* don't do anything if one of HW triggers is set */
810 if (check_trigger && &marvell_led_triggers[0] <= cdev->trigger
&&
811 cdev->trigger <
&marvell_led_triggers[ARRAY_SIZE(marvell_led_triggers)])
812 return 0;
813
814 led = cdev - &priv->leds[0];
815 val = brightness ? MII_PHY_LED_CTRL_FORCE_ON : MII_PHY_LED_CTRL_FORCE_OFF;
816
817 return marvell_led_set_regval(phydev, marvell_led_index(cdev), val);
818 }
819
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org