All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [RFC v5 net-next 05/13] net: mdio: mscc-miim: add ability to externally register phy reset control
Date: Sun, 19 Dec 2021 08:33:37 +0800	[thread overview]
Message-ID: <202112190845.kcGDVech-lkp@intel.com> (raw)
In-Reply-To: <20211218214954.109755-6-colin.foster@in-advantage.com>

[-- Attachment #1: Type: text/plain, Size: 8891 bytes --]

Hi Colin,

[FYI, it's a private test report for your RFC patch.]
[auto build test WARNING on next-20211217]
[cannot apply to net-next/master lee-mfd/for-mfd-next linusw-pinctrl/devel net/master horms-ipvs/master linus/master v5.16-rc5 v5.16-rc4 v5.16-rc3 v5.16-rc5]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/0day-ci/linux/commits/Colin-Foster/add-support-for-VSC75XX-control-over-SPI/20211219-055252
base:    c122052c3cb861b3e61a01d2c2ab9069e470663e
config: nds32-allyesconfig (https://download.01.org/0day-ci/archive/20211219/202112190845.kcGDVech-lkp(a)intel.com/config)
compiler: nds32le-linux-gcc (GCC) 11.2.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
        # https://github.com/0day-ci/linux/commit/82c0240e048ab8cb99b1acfa42497eb5e4a3a067
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Colin-Foster/add-support-for-VSC75XX-control-over-SPI/20211219-055252
        git checkout 82c0240e048ab8cb99b1acfa42497eb5e4a3a067
        # save the config file to linux build tree
        mkdir build_dir
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=nds32 SHELL=/bin/bash drivers/net/mdio/

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All warnings (new ones prefixed by >>):

   drivers/net/mdio/mdio-mscc-miim.c: In function 'mscc_miim_probe':
>> drivers/net/mdio/mdio-mscc-miim.c:227:31: warning: variable 'miim' set but not used [-Wunused-but-set-variable]
     227 |         struct mscc_miim_dev *miim;
         |                               ^~~~


vim +/miim +227 drivers/net/mdio/mdio-mscc-miim.c

a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  222  
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  223  static int mscc_miim_probe(struct platform_device *pdev)
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  224  {
c448c898ae890d9 drivers/net/mdio/mdio-mscc-miim.c Horatiu Vultur    2021-11-30  225  	struct regmap *mii_regmap, *phy_regmap = NULL;
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  226  	void __iomem *regs, *phy_regs;
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28 @227  	struct mscc_miim_dev *miim;
c448c898ae890d9 drivers/net/mdio/mdio-mscc-miim.c Horatiu Vultur    2021-11-30  228  	struct resource *res;
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  229  	struct mii_bus *bus;
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  230  	int ret;
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  231  
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  232  	regs = devm_platform_get_and_ioremap_resource(pdev, 0, NULL);
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  233  	if (IS_ERR(regs)) {
542671fe4d86ad4 drivers/net/phy/mdio-mscc-miim.c  Alexandre Belloni 2018-05-14  234  		dev_err(&pdev->dev, "Unable to map MIIM registers\n");
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  235  		return PTR_ERR(regs);
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  236  	}
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  237  
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  238  	mii_regmap = devm_regmap_init_mmio(&pdev->dev, regs,
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  239  					   &mscc_miim_regmap_config);
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  240  
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  241  	if (IS_ERR(mii_regmap)) {
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  242  		dev_err(&pdev->dev, "Unable to create MIIM regmap\n");
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  243  		return PTR_ERR(mii_regmap);
542671fe4d86ad4 drivers/net/phy/mdio-mscc-miim.c  Alexandre Belloni 2018-05-14  244  	}
542671fe4d86ad4 drivers/net/phy/mdio-mscc-miim.c  Alexandre Belloni 2018-05-14  245  
c448c898ae890d9 drivers/net/mdio/mdio-mscc-miim.c Horatiu Vultur    2021-11-30  246  	/* This resource is optional */
c448c898ae890d9 drivers/net/mdio/mdio-mscc-miim.c Horatiu Vultur    2021-11-30  247  	res = platform_get_resource(pdev, IORESOURCE_MEM, 1);
c448c898ae890d9 drivers/net/mdio/mdio-mscc-miim.c Horatiu Vultur    2021-11-30  248  	if (res) {
c448c898ae890d9 drivers/net/mdio/mdio-mscc-miim.c Horatiu Vultur    2021-11-30  249  		phy_regs = devm_ioremap_resource(&pdev->dev, res);
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  250  		if (IS_ERR(phy_regs)) {
542671fe4d86ad4 drivers/net/phy/mdio-mscc-miim.c  Alexandre Belloni 2018-05-14  251  			dev_err(&pdev->dev, "Unable to map internal phy registers\n");
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  252  			return PTR_ERR(phy_regs);
542671fe4d86ad4 drivers/net/phy/mdio-mscc-miim.c  Alexandre Belloni 2018-05-14  253  		}
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  254  
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  255  		phy_regmap = devm_regmap_init_mmio(&pdev->dev, phy_regs,
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  256  						   &mscc_miim_regmap_config);
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  257  		if (IS_ERR(phy_regmap)) {
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  258  			dev_err(&pdev->dev, "Unable to create phy register regmap\n");
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  259  			return PTR_ERR(phy_regmap);
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  260  		}
c448c898ae890d9 drivers/net/mdio/mdio-mscc-miim.c Horatiu Vultur    2021-11-30  261  	}
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  262  
82c0240e048ab8c drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-12-18  263  	ret = mscc_miim_setup(&pdev->dev, &bus, "mscc_miim", mii_regmap, 0,
82c0240e048ab8c drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-12-18  264  			      phy_regmap, 0);
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  265  	if (ret < 0) {
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  266  		dev_err(&pdev->dev, "Unable to setup the MDIO bus\n");
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  267  		return ret;
c6995117b60ef3f drivers/net/mdio/mdio-mscc-miim.c Horatiu Vultur    2021-09-28  268  	}
542671fe4d86ad4 drivers/net/phy/mdio-mscc-miim.c  Alexandre Belloni 2018-05-14  269  
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  270  	miim = bus->priv;
a27a762828375a2 drivers/net/mdio/mdio-mscc-miim.c Colin Foster      2021-11-28  271  
542671fe4d86ad4 drivers/net/phy/mdio-mscc-miim.c  Alexandre Belloni 2018-05-14  272  	ret = of_mdiobus_register(bus, pdev->dev.of_node);
542671fe4d86ad4 drivers/net/phy/mdio-mscc-miim.c  Alexandre Belloni 2018-05-14  273  	if (ret < 0) {
542671fe4d86ad4 drivers/net/phy/mdio-mscc-miim.c  Alexandre Belloni 2018-05-14  274  		dev_err(&pdev->dev, "Cannot register MDIO bus (%d)\n", ret);
542671fe4d86ad4 drivers/net/phy/mdio-mscc-miim.c  Alexandre Belloni 2018-05-14  275  		return ret;
542671fe4d86ad4 drivers/net/phy/mdio-mscc-miim.c  Alexandre Belloni 2018-05-14  276  	}
542671fe4d86ad4 drivers/net/phy/mdio-mscc-miim.c  Alexandre Belloni 2018-05-14  277  
542671fe4d86ad4 drivers/net/phy/mdio-mscc-miim.c  Alexandre Belloni 2018-05-14  278  	platform_set_drvdata(pdev, bus);
542671fe4d86ad4 drivers/net/phy/mdio-mscc-miim.c  Alexandre Belloni 2018-05-14  279  
542671fe4d86ad4 drivers/net/phy/mdio-mscc-miim.c  Alexandre Belloni 2018-05-14  280  	return 0;
542671fe4d86ad4 drivers/net/phy/mdio-mscc-miim.c  Alexandre Belloni 2018-05-14  281  }
542671fe4d86ad4 drivers/net/phy/mdio-mscc-miim.c  Alexandre Belloni 2018-05-14  282  

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

  reply	other threads:[~2021-12-19  0:33 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-12-18 21:49 [RFC v5 net-next 00/13] add support for VSC75XX control over SPI Colin Foster
2021-12-18 21:49 ` [RFC v5 net-next 01/13] mfd: ocelot: add support for external mfd control over SPI for the VSC7512 Colin Foster
2021-12-19  1:55   ` kernel test robot
2021-12-29 15:22   ` Lee Jones
2021-12-30  1:43     ` Colin Foster
2022-01-10 12:16       ` Lee Jones
2022-01-11  0:33         ` Colin Foster
2022-01-11 10:13           ` Lee Jones
2022-01-11 13:44             ` Mark Brown
2022-01-11 16:53               ` Colin Foster
2022-01-11 17:00                 ` Lee Jones
2022-01-11 18:28                   ` Colin Foster
2022-01-11 18:41                     ` Alexandre Belloni
2022-01-15  2:07                     ` Colin Foster
2021-12-18 21:49 ` [RFC v5 net-next 02/13] mfd: ocelot: offer an interface for MFD children to get regmaps Colin Foster
2021-12-29 15:23   ` Lee Jones
2021-12-29 19:34     ` Alexandre Belloni
2021-12-29 20:12       ` Lee Jones
2021-12-18 21:49 ` [RFC v5 net-next 03/13] net: mscc: ocelot: expose ocelot wm functions Colin Foster
2021-12-18 21:49 ` [RFC v5 net-next 04/13] net: dsa: felix: add configurable device quirks Colin Foster
2021-12-18 21:49 ` [RFC v5 net-next 05/13] net: mdio: mscc-miim: add ability to externally register phy reset control Colin Foster
2021-12-19  0:33   ` kernel test robot [this message]
2021-12-18 21:49 ` [RFC v5 net-next 06/13] net: dsa: ocelot: add external ocelot switch control Colin Foster
2021-12-18 21:49 ` [RFC v5 net-next 07/13] mfd: ocelot: enable the external switch interface Colin Foster
2021-12-29 15:24   ` Lee Jones
2021-12-18 21:49 ` [RFC v5 net-next 08/13] mfd: add interface to check whether a device is mfd Colin Foster
2021-12-29 15:25   ` Lee Jones
2021-12-30  2:04     ` Colin Foster
2021-12-30 13:43       ` Lee Jones
2021-12-30 20:12         ` Colin Foster
2022-01-10 12:23           ` Lee Jones
2021-12-18 21:49 ` [RFC v5 net-next 09/13] net: mdio: mscc-miim: add local dev variable to cleanup probe function Colin Foster
2021-12-18 21:49 ` [RFC v5 net-next 10/13] net: mdio: mscc-miim: add MFD functionality through ocelot-core Colin Foster
2021-12-19  0:13   ` kernel test robot
2021-12-18 21:49 ` [RFC v5 net-next 11/13] mfd: ocelot-core: add control for the external mdio interface Colin Foster
2021-12-29 15:26   ` Lee Jones
2021-12-18 21:49 ` [RFC v5 net-next 12/13] pinctrl: ocelot: add MFD functionality through ocelot-core Colin Foster
2021-12-18 21:49 ` [RFC v5 net-next 13/13] mfd: ocelot: add ocelot-pinctrl as a supported child interface Colin Foster
2021-12-29 15:26   ` Lee Jones

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=202112190845.kcGDVech-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=kbuild-all@lists.01.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.