* Re: [PATCH net-next 3/3] net: phylink: simplify how SFP PHYs are attached
[not found] <E1t3DSr-000Vxg-Qx@rmk-PC.armlinux.org.uk>
@ 2024-10-23 13:11 ` kernel test robot
0 siblings, 0 replies; only message in thread
From: kernel test robot @ 2024-10-23 13:11 UTC (permalink / raw)
To: Russell King (Oracle); +Cc: llvm, oe-kbuild-all
Hi Russell,
kernel test robot noticed the following build warnings:
[auto build test WARNING on net-next/main]
url: https://github.com/intel-lab-lkp/linux/commits/Russell-King-Oracle/net-phylink-add-common-validation-for-sfp_select_interface/20241022-234251
base: net-next/main
patch link: https://lore.kernel.org/r/E1t3DSr-000Vxg-Qx%40rmk-PC.armlinux.org.uk
patch subject: [PATCH net-next 3/3] net: phylink: simplify how SFP PHYs are attached
config: x86_64-kexec (https://download.01.org/0day-ci/archive/20241023/202410232116.IgRFPrAS-lkp@intel.com/config)
compiler: clang version 18.1.8 (https://github.com/llvm/llvm-project 3b5b5c1ec4a3095ab096dd780e84d7ab81f3d7ff)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20241023/202410232116.IgRFPrAS-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202410232116.IgRFPrAS-lkp@intel.com/
All warnings (new ones prefixed by >>):
>> drivers/net/phy/phylink.c:3429:6: warning: unused variable 'ret' [-Wunused-variable]
3429 | int ret;
| ^~~
1 warning generated.
vim +/ret +3429 drivers/net/phy/phylink.c
7adb5b2126bc013 Russell King 2019-12-11 3424
ce0aa27ff3f68ed Russell King 2017-07-25 3425 static int phylink_sfp_connect_phy(void *upstream, struct phy_device *phy)
ce0aa27ff3f68ed Russell King 2017-07-25 3426 {
7e4183752735deb Baruch Siach 2018-10-03 3427 struct phylink *pl = upstream;
7adb5b2126bc013 Russell King 2019-12-11 3428 u8 mode;
938d44c2b502ba4 Russell King 2019-12-11 @3429 int ret;
7e4183752735deb Baruch Siach 2018-10-03 3430
52c956003a9d5bc Russell King 2019-12-11 3431 /*
52c956003a9d5bc Russell King 2019-12-11 3432 * This is the new way of dealing with flow control for PHYs,
52c956003a9d5bc Russell King 2019-12-11 3433 * as described by Timur Tabi in commit 529ed1275263 ("net: phy:
52c956003a9d5bc Russell King 2019-12-11 3434 * phy drivers should not set SUPPORTED_[Asym_]Pause") except
52c956003a9d5bc Russell King 2019-12-11 3435 * using our validate call to the MAC, we rely upon the MAC
52c956003a9d5bc Russell King 2019-12-11 3436 * clearing the bits from both supported and advertising fields.
52c956003a9d5bc Russell King 2019-12-11 3437 */
52c956003a9d5bc Russell King 2019-12-11 3438 phy_support_asym_pause(phy);
52c956003a9d5bc Russell King 2019-12-11 3439
7adb5b2126bc013 Russell King 2019-12-11 3440 if (phylink_phy_no_inband(phy))
7adb5b2126bc013 Russell King 2019-12-11 3441 mode = MLO_AN_PHY;
7adb5b2126bc013 Russell King 2019-12-11 3442 else
7adb5b2126bc013 Russell King 2019-12-11 3443 mode = MLO_AN_INBAND;
7adb5b2126bc013 Russell King 2019-12-11 3444
eca68a3c7d05b38 Marek Behún 2022-09-30 3445 /* Set the PHY's host supported interfaces */
eca68a3c7d05b38 Marek Behún 2022-09-30 3446 phy_interface_and(phy->host_interfaces, phylink_sfp_interfaces,
eca68a3c7d05b38 Marek Behún 2022-09-30 3447 pl->config->supported_interfaces);
eca68a3c7d05b38 Marek Behún 2022-09-30 3448
52c956003a9d5bc Russell King 2019-12-11 3449 /* Do the initial configuration */
e48bc58d0fa1e08 Russell King (Oracle 2024-10-22 3450) return phylink_sfp_config_phy(pl, mode, phy);
ce0aa27ff3f68ed Russell King 2017-07-25 3451 }
ce0aa27ff3f68ed Russell King 2017-07-25 3452
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] only message in thread