Hi Robert, Thank you for the patch! Yet something to improve:
[auto build test ERROR on linus/master] [cannot apply to v5.3-rc8 next-20190904] [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/Robert-Beckett/net-dsa-mv88e6xxx-features-to-handle-network-storms/20190911-142233 config: mips-allmodconfig (attached as .config) compiler: mips-linux-gcc (GCC) 7.4.0 reproduce: 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 GCC_VERSION=7.4.0 make.cross ARCH=mips If you fix the issue, kindly add following tag Reported-by: kbuild test robot <l...@intel.com> All errors (new ones prefixed by >>): net//dsa/port.c: In function 'dsa_port_setup_phy_of': >> net//dsa/port.c:541:41: error: invalid operands to binary | (have 'long >> unsigned int *' and 'long unsigned int') phydev->supported = PHY_GBIT_FEATURES | SUPPORTED_MII | ^ >> net//dsa/port.c:545:23: error: assignment to expression with array type phydev->advertising = phydev->supported; ^ vim +541 net//dsa/port.c 525 526 static int dsa_port_setup_phy_of(struct dsa_port *dp, bool enable) 527 { 528 struct dsa_switch *ds = dp->ds; 529 struct phy_device *phydev; 530 int port = dp->index; 531 int err = 0; 532 533 phydev = dsa_port_get_phy_device(dp); 534 if (!phydev) 535 return 0; 536 537 if (IS_ERR(phydev)) 538 return PTR_ERR(phydev); 539 540 if (enable) { > 541 phydev->supported = PHY_GBIT_FEATURES | SUPPORTED_MII | 542 SUPPORTED_AUI | SUPPORTED_FIBRE | 543 SUPPORTED_BNC | SUPPORTED_Pause | 544 SUPPORTED_Asym_Pause; > 545 phydev->advertising = phydev->supported; 546 547 err = genphy_config_init(phydev); 548 if (err < 0) 549 goto err_put_dev; 550 551 err = genphy_config_aneg(phydev); 552 if (err < 0) 553 goto err_put_dev; 554 555 err = genphy_resume(phydev); 556 if (err < 0) 557 goto err_put_dev; 558 559 err = genphy_read_status(phydev); 560 if (err < 0) 561 goto err_put_dev; 562 } else { 563 err = genphy_suspend(phydev); 564 if (err < 0) 565 goto err_put_dev; 566 } 567 568 if (ds->ops->adjust_link) 569 ds->ops->adjust_link(ds, port, phydev); 570 571 dev_dbg(ds->dev, "enabled port's phy: %s", phydev_name(phydev)); 572 573 err_put_dev: 574 put_device(&phydev->mdio.dev); 575 return err; 576 } 577 --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation
.config.gz
Description: application/gzip
