linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: kbuild test robot <lkp@intel.com>
To: Peng Fan <peng.fan@nxp.com>
Cc: kbuild-all@lists.01.org,
	"catalin.marinas@arm.com" <catalin.marinas@arm.com>,
	"will@kernel.org" <will@kernel.org>,
	"shawnguo@kernel.org" <shawnguo@kernel.org>,
	"s.hauer@pengutronix.de" <s.hauer@pengutronix.de>,
	"arnd@arndb.de" <arnd@arndb.de>,
	"festevam@gmail.com" <festevam@gmail.com>,
	"kernel@pengutronix.de" <kernel@pengutronix.de>,
	dl-linux-imx <linux-imx@nxp.com>,
	"olof@lixom.net" <olof@lixom.net>,
	Aisheng Dong <aisheng.dong@nxp.com>,
	Leonard Crestez <leonard.crestez@nxp.com>,
	Abel Vesa <abel.vesa@nxp.com>,
	"krzk@kernel.org" <krzk@kernel.org>,
	"linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Peng Fan <peng.fan@nxp.com>
Subject: Re: [PATCH 3/4] soc: Makefile: increase build coverage for i.MX
Date: Sat, 18 Jan 2020 16:38:19 +0800	[thread overview]
Message-ID: <202001181618.geTkcz9D%lkp@intel.com> (raw)
In-Reply-To: <1579146280-1750-4-git-send-email-peng.fan@nxp.com>

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

Hi Peng,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on shawnguo/for-next]
[also build test ERROR on arm64/for-next/core linus/master v5.5-rc6]
[cannot apply to next-20200117]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]

url:    https://github.com/0day-ci/linux/commits/Peng-Fan/soc-imx-increase-build-coverage-for-imx8-soc-driver/20200117-121515
base:   https://git.kernel.org/pub/scm/linux/kernel/git/shawnguo/linux.git for-next
config: sh-allmodconfig (attached as .config)
compiler: sh4-linux-gcc (GCC) 7.5.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.5.0 make.cross ARCH=sh 

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

All errors (new ones prefixed by >>):

   drivers/soc/imx/gpcv2.c: In function 'imx_gpcv2_probe':
>> drivers/soc/imx/gpcv2.c:561:21: error: 'SZ_4K' undeclared (first use in this function)
      .max_register   = SZ_4K,
                        ^~~~~
   drivers/soc/imx/gpcv2.c:561:21: note: each undeclared identifier is reported only once for each function it appears in

vim +/SZ_4K +561 drivers/soc/imx/gpcv2.c

03aa12629fc4f7 Andrey Smirnov 2017-03-28  549  
03aa12629fc4f7 Andrey Smirnov 2017-03-28  550  static int imx_gpcv2_probe(struct platform_device *pdev)
03aa12629fc4f7 Andrey Smirnov 2017-03-28  551  {
e125dcba83f59b Lucas Stach    2018-11-16  552  	const struct imx_pgc_domain_data *domain_data =
e125dcba83f59b Lucas Stach    2018-11-16  553  			of_device_get_match_data(&pdev->dev);
e125dcba83f59b Lucas Stach    2018-11-16  554  
e125dcba83f59b Lucas Stach    2018-11-16  555  	struct regmap_config regmap_config = {
03aa12629fc4f7 Andrey Smirnov 2017-03-28  556  		.reg_bits	= 32,
03aa12629fc4f7 Andrey Smirnov 2017-03-28  557  		.val_bits	= 32,
03aa12629fc4f7 Andrey Smirnov 2017-03-28  558  		.reg_stride	= 4,
e125dcba83f59b Lucas Stach    2018-11-16  559  		.rd_table	= domain_data->reg_access_table,
e125dcba83f59b Lucas Stach    2018-11-16  560  		.wr_table	= domain_data->reg_access_table,
03aa12629fc4f7 Andrey Smirnov 2017-03-28 @561  		.max_register   = SZ_4K,
03aa12629fc4f7 Andrey Smirnov 2017-03-28  562  	};
03aa12629fc4f7 Andrey Smirnov 2017-03-28  563  	struct device *dev = &pdev->dev;
03aa12629fc4f7 Andrey Smirnov 2017-03-28  564  	struct device_node *pgc_np, *np;
03aa12629fc4f7 Andrey Smirnov 2017-03-28  565  	struct regmap *regmap;
03aa12629fc4f7 Andrey Smirnov 2017-03-28  566  	void __iomem *base;
03aa12629fc4f7 Andrey Smirnov 2017-03-28  567  	int ret;
03aa12629fc4f7 Andrey Smirnov 2017-03-28  568  
03aa12629fc4f7 Andrey Smirnov 2017-03-28  569  	pgc_np = of_get_child_by_name(dev->of_node, "pgc");
03aa12629fc4f7 Andrey Smirnov 2017-03-28  570  	if (!pgc_np) {
03aa12629fc4f7 Andrey Smirnov 2017-03-28  571  		dev_err(dev, "No power domains specified in DT\n");
03aa12629fc4f7 Andrey Smirnov 2017-03-28  572  		return -EINVAL;
03aa12629fc4f7 Andrey Smirnov 2017-03-28  573  	}
03aa12629fc4f7 Andrey Smirnov 2017-03-28  574  
9f735c4e94fcbe Anson Huang    2019-04-01  575  	base = devm_platform_ioremap_resource(pdev, 0);
03aa12629fc4f7 Andrey Smirnov 2017-03-28  576  	if (IS_ERR(base))
03aa12629fc4f7 Andrey Smirnov 2017-03-28  577  		return PTR_ERR(base);
03aa12629fc4f7 Andrey Smirnov 2017-03-28  578  
03aa12629fc4f7 Andrey Smirnov 2017-03-28  579  	regmap = devm_regmap_init_mmio(dev, base, &regmap_config);
03aa12629fc4f7 Andrey Smirnov 2017-03-28  580  	if (IS_ERR(regmap)) {
03aa12629fc4f7 Andrey Smirnov 2017-03-28  581  		ret = PTR_ERR(regmap);
03aa12629fc4f7 Andrey Smirnov 2017-03-28  582  		dev_err(dev, "failed to init regmap (%d)\n", ret);
03aa12629fc4f7 Andrey Smirnov 2017-03-28  583  		return ret;
03aa12629fc4f7 Andrey Smirnov 2017-03-28  584  	}
03aa12629fc4f7 Andrey Smirnov 2017-03-28  585  
03aa12629fc4f7 Andrey Smirnov 2017-03-28  586  	for_each_child_of_node(pgc_np, np) {
03aa12629fc4f7 Andrey Smirnov 2017-03-28  587  		struct platform_device *pd_pdev;
73f59712a1a3e5 Anson Huang    2018-08-28  588  		struct imx_pgc_domain *domain;
03aa12629fc4f7 Andrey Smirnov 2017-03-28  589  		u32 domain_index;
03aa12629fc4f7 Andrey Smirnov 2017-03-28  590  
03aa12629fc4f7 Andrey Smirnov 2017-03-28  591  		ret = of_property_read_u32(np, "reg", &domain_index);
03aa12629fc4f7 Andrey Smirnov 2017-03-28  592  		if (ret) {
03aa12629fc4f7 Andrey Smirnov 2017-03-28  593  			dev_err(dev, "Failed to read 'reg' property\n");
03aa12629fc4f7 Andrey Smirnov 2017-03-28  594  			of_node_put(np);
03aa12629fc4f7 Andrey Smirnov 2017-03-28  595  			return ret;
03aa12629fc4f7 Andrey Smirnov 2017-03-28  596  		}
03aa12629fc4f7 Andrey Smirnov 2017-03-28  597  
73f59712a1a3e5 Anson Huang    2018-08-28  598  		if (domain_index >= domain_data->domains_num) {
03aa12629fc4f7 Andrey Smirnov 2017-03-28  599  			dev_warn(dev,
03aa12629fc4f7 Andrey Smirnov 2017-03-28  600  				 "Domain index %d is out of bounds\n",
03aa12629fc4f7 Andrey Smirnov 2017-03-28  601  				 domain_index);
03aa12629fc4f7 Andrey Smirnov 2017-03-28  602  			continue;
03aa12629fc4f7 Andrey Smirnov 2017-03-28  603  		}
03aa12629fc4f7 Andrey Smirnov 2017-03-28  604  
73f59712a1a3e5 Anson Huang    2018-08-28  605  		pd_pdev = platform_device_alloc("imx-pgc-domain",
03aa12629fc4f7 Andrey Smirnov 2017-03-28  606  						domain_index);
03aa12629fc4f7 Andrey Smirnov 2017-03-28  607  		if (!pd_pdev) {
03aa12629fc4f7 Andrey Smirnov 2017-03-28  608  			dev_err(dev, "Failed to allocate platform device\n");
03aa12629fc4f7 Andrey Smirnov 2017-03-28  609  			of_node_put(np);
03aa12629fc4f7 Andrey Smirnov 2017-03-28  610  			return -ENOMEM;
03aa12629fc4f7 Andrey Smirnov 2017-03-28  611  		}
03aa12629fc4f7 Andrey Smirnov 2017-03-28  612  
050f810e238f26 Andrey Smirnov 2018-04-10  613  		ret = platform_device_add_data(pd_pdev,
73f59712a1a3e5 Anson Huang    2018-08-28  614  					       &domain_data->domains[domain_index],
73f59712a1a3e5 Anson Huang    2018-08-28  615  					       sizeof(domain_data->domains[domain_index]));
050f810e238f26 Andrey Smirnov 2018-04-10  616  		if (ret) {
050f810e238f26 Andrey Smirnov 2018-04-10  617  			platform_device_put(pd_pdev);
050f810e238f26 Andrey Smirnov 2018-04-10  618  			of_node_put(np);
050f810e238f26 Andrey Smirnov 2018-04-10  619  			return ret;
050f810e238f26 Andrey Smirnov 2018-04-10  620  		}
050f810e238f26 Andrey Smirnov 2018-04-10  621  
050f810e238f26 Andrey Smirnov 2018-04-10  622  		domain = pd_pdev->dev.platform_data;
050f810e238f26 Andrey Smirnov 2018-04-10  623  		domain->regmap = regmap;
73f59712a1a3e5 Anson Huang    2018-08-28  624  		domain->genpd.power_on  = imx_gpc_pu_pgc_sw_pup_req;
73f59712a1a3e5 Anson Huang    2018-08-28  625  		domain->genpd.power_off = imx_gpc_pu_pgc_sw_pdn_req;
050f810e238f26 Andrey Smirnov 2018-04-10  626  
03aa12629fc4f7 Andrey Smirnov 2017-03-28  627  		pd_pdev->dev.parent = dev;
03aa12629fc4f7 Andrey Smirnov 2017-03-28  628  		pd_pdev->dev.of_node = np;
03aa12629fc4f7 Andrey Smirnov 2017-03-28  629  
03aa12629fc4f7 Andrey Smirnov 2017-03-28  630  		ret = platform_device_add(pd_pdev);
03aa12629fc4f7 Andrey Smirnov 2017-03-28  631  		if (ret) {
03aa12629fc4f7 Andrey Smirnov 2017-03-28  632  			platform_device_put(pd_pdev);
03aa12629fc4f7 Andrey Smirnov 2017-03-28  633  			of_node_put(np);
03aa12629fc4f7 Andrey Smirnov 2017-03-28  634  			return ret;
03aa12629fc4f7 Andrey Smirnov 2017-03-28  635  		}
03aa12629fc4f7 Andrey Smirnov 2017-03-28  636  	}
03aa12629fc4f7 Andrey Smirnov 2017-03-28  637  
03aa12629fc4f7 Andrey Smirnov 2017-03-28  638  	return 0;
03aa12629fc4f7 Andrey Smirnov 2017-03-28  639  }
03aa12629fc4f7 Andrey Smirnov 2017-03-28  640  

:::::: The code at line 561 was first introduced by commit
:::::: 03aa12629fc4f73acf28e519c9ee9cb1f5dd3706 soc: imx: Add GPCv2 power gating driver

:::::: TO: Andrey Smirnov <andrew.smirnov@gmail.com>
:::::: CC: Shawn Guo <shawnguo@kernel.org>

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

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 52792 bytes --]

  reply	other threads:[~2020-01-18  8:39 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-01-16  3:48 [PATCH 0/4] soc: imx: increase build coverage for imx8 soc driver Peng Fan
2020-01-16  3:49 ` [PATCH 1/4] soc: imx: Kconfig: add SOC_IMX8M entry Peng Fan
2020-01-16  3:49 ` [PATCH 2/4] arm64: defconfig: Enable CONFIG_SOC_IMX8M by default Peng Fan
2020-01-16  3:49 ` [PATCH 3/4] soc: Makefile: increase build coverage for i.MX Peng Fan
2020-01-18  8:38   ` kbuild test robot [this message]
2020-01-16  3:49 ` [PATCH 4/4] soc: imx: Use CONFIG_SOC_IMX8M as build gate Peng Fan
2020-01-20 12:45 ` [PATCH 0/4] soc: imx: increase build coverage for imx8 soc driver Leonard Crestez

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=202001181618.geTkcz9D%lkp@intel.com \
    --to=lkp@intel.com \
    --cc=abel.vesa@nxp.com \
    --cc=aisheng.dong@nxp.com \
    --cc=arnd@arndb.de \
    --cc=catalin.marinas@arm.com \
    --cc=festevam@gmail.com \
    --cc=kbuild-all@lists.01.org \
    --cc=kernel@pengutronix.de \
    --cc=krzk@kernel.org \
    --cc=leonard.crestez@nxp.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-imx@nxp.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=olof@lixom.net \
    --cc=peng.fan@nxp.com \
    --cc=s.hauer@pengutronix.de \
    --cc=shawnguo@kernel.org \
    --cc=will@kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).