[PATCH 0/4] soc: imx: increase build coverage for imx8 soc driver

[PATCH 0/4] soc: imx: increase build coverage for imx8 soc driver

[Peng Fan]

From: Peng Fan <peng.fan@nxp.com>

Rename soc-imx8.c to soc-imx8m.c which is for i.MX8M family
Add SOC_IMX8M for build gate soc-imx8m.c
Increase build coverage for i.MX SoC driver

Peng Fan (4):
  soc: imx: Kconfig: add SOC_IMX8M entry
  arm64: defconfig: Enable CONFIG_SOC_IMX8M by default
  soc: Makefile: increase build coverage for i.MX
  soc: imx: Use CONFIG_SOC_IMX8M as build gate

 arch/arm64/configs/defconfig                | 1 +
 drivers/soc/Makefile                        | 2 +-
 drivers/soc/imx/Kconfig                     | 8 ++++++++
 drivers/soc/imx/Makefile                    | 2 +-
 drivers/soc/imx/{soc-imx8.c => soc-imx8m.c} | 0
 5 files changed, 11 insertions(+), 2 deletions(-)
 rename drivers/soc/imx/{soc-imx8.c => soc-imx8m.c} (100%)

-- 
2.16.4

[PATCH 1/4] soc: imx: Kconfig: add SOC_IMX8M entry

[Peng Fan]

From: Peng Fan <peng.fan@nxp.com>

Add SOC_IMX8M Kconfig entry that could let people to select
and deselect.

Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
 drivers/soc/imx/Kconfig | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/drivers/soc/imx/Kconfig b/drivers/soc/imx/Kconfig
index 0281ef9a1800..790b46d90f0f 100644
--- a/drivers/soc/imx/Kconfig
+++ b/drivers/soc/imx/Kconfig
@@ -17,4 +17,12 @@ config IMX_SCU_SOC
 	  Controller Unit SoC info module, it will provide the SoC info
 	  like SoC family, ID and revision etc.
 
+config SOC_IMX8M
+	bool "i.MX8M SoC family support"
+	depends on ARCH_MXC || COMPILE_TEST
+	help
+	  If you say yes here you get support for the NXP i.MX8M family
+	  support, it will provide the SoC info like SoC family,
+	  ID and revision etc.
+
 endmenu
-- 
2.16.4

[PATCH 2/4] arm64: defconfig: Enable CONFIG_SOC_IMX8M by default

[Peng Fan]

From: Peng Fan <peng.fan@nxp.com>

Enable CONFIG_SOC_IMX8M by default to build i.MX8M SoC drivers

Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
 arch/arm64/configs/defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index d0ea0d0d3b16..20087f1aba56 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig
@@ -729,6 +729,7 @@ CONFIG_RPMSG_QCOM_GLINK_SMEM=m
 CONFIG_RPMSG_QCOM_SMD=y
 CONFIG_RASPBERRYPI_POWER=y
 CONFIG_IMX_SCU_SOC=y
+CONFIG_SOC_IMX8M=y
 CONFIG_QCOM_GENI_SE=y
 CONFIG_QCOM_GLINK_SSR=m
 CONFIG_QCOM_RPMH=y
-- 
2.16.4

[PATCH 3/4] soc: Makefile: increase build coverage for i.MX

[Peng Fan]

From: Peng Fan <peng.fan@nxp.com>

Increase build coverage for i.MX SoC drivers.

Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
 drivers/soc/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/soc/Makefile b/drivers/soc/Makefile
index 2ec355003524..614986cd1713 100644
--- a/drivers/soc/Makefile
+++ b/drivers/soc/Makefile
@@ -11,7 +11,7 @@ obj-$(CONFIG_ARCH_DOVE)		+= dove/
 obj-$(CONFIG_MACH_DOVE)		+= dove/
 obj-y				+= fsl/
 obj-$(CONFIG_ARCH_GEMINI)	+= gemini/
-obj-$(CONFIG_ARCH_MXC)		+= imx/
+obj-y				+= imx/
 obj-$(CONFIG_ARCH_IXP4XX)	+= ixp4xx/
 obj-$(CONFIG_SOC_XWAY)		+= lantiq/
 obj-y				+= mediatek/
-- 
2.16.4

[PATCH 4/4] soc: imx: Use CONFIG_SOC_IMX8M as build gate

[Peng Fan]

From: Peng Fan <peng.fan@nxp.com>

The soc-imx8.c driver is actually for i.MX8M family, so rename it
to soc-imx8m.c.
Use CONFIG_SOC_IMX8M as build gate, not CONFIG_ARCH_MXC, to control
whether build this driver, also make it possible for compile test.

Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
 drivers/soc/imx/Makefile                    | 2 +-
 drivers/soc/imx/{soc-imx8.c => soc-imx8m.c} | 0
 2 files changed, 1 insertion(+), 1 deletion(-)
 rename drivers/soc/imx/{soc-imx8.c => soc-imx8m.c} (100%)

diff --git a/drivers/soc/imx/Makefile b/drivers/soc/imx/Makefile
index cf9ca42ff739..103e2c93c342 100644
--- a/drivers/soc/imx/Makefile
+++ b/drivers/soc/imx/Makefile
@@ -1,5 +1,5 @@
 # SPDX-License-Identifier: GPL-2.0-only
 obj-$(CONFIG_HAVE_IMX_GPC) += gpc.o
 obj-$(CONFIG_IMX_GPCV2_PM_DOMAINS) += gpcv2.o
-obj-$(CONFIG_ARCH_MXC) += soc-imx8.o
+obj-$(CONFIG_SOC_IMX8M) += soc-imx8m.o
 obj-$(CONFIG_IMX_SCU_SOC) += soc-imx-scu.o
diff --git a/drivers/soc/imx/soc-imx8.c b/drivers/soc/imx/soc-imx8m.c
similarity index 100%
rename from drivers/soc/imx/soc-imx8.c
rename to drivers/soc/imx/soc-imx8m.c
-- 
2.16.4

Re: [PATCH 3/4] soc: Makefile: increase build coverage for i.MX

[kbuild test robot]

[-- 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 --]

Re: [PATCH 0/4] soc: imx: increase build coverage for imx8 soc driver

[Leonard Crestez]

On 16.01.2020 05:48, Peng Fan wrote:
> From: Peng Fan <peng.fan@nxp.com>
> 
> Rename soc-imx8.c to soc-imx8m.c which is for i.MX8M family
> Add SOC_IMX8M for build gate soc-imx8m.c
> Increase build coverage for i.MX SoC driver
> 
> Peng Fan (4):
>    soc: imx: Kconfig: add SOC_IMX8M entry
>    arm64: defconfig: Enable CONFIG_SOC_IMX8M by default
>    soc: Makefile: increase build coverage for i.MX
>    soc: imx: Use CONFIG_SOC_IMX8M as build gate
> 
>   arch/arm64/configs/defconfig                | 1 +
>   drivers/soc/Makefile                        | 2 +-
>   drivers/soc/imx/Kconfig                     | 8 ++++++++
>   drivers/soc/imx/Makefile                    | 2 +-
>   drivers/soc/imx/{soc-imx8.c => soc-imx8m.c} | 0
>   5 files changed, 11 insertions(+), 2 deletions(-)
>   rename drivers/soc/imx/{soc-imx8.c => soc-imx8m.c} (100%)

I applied your patches and compiletest failed for me on x86:

../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

It seems that on x86 <linux/sizes.h> needs to be included explicitly 
while on arm it is already included indirectly through arch headers. Fix 
is very simple:

diff --git drivers/soc/imx/gpcv2.c drivers/soc/imx/gpcv2.c
index b0dffb06c05d..6cf8a7a412bd 100644
--- drivers/soc/imx/gpcv2.c
+++ drivers/soc/imx/gpcv2.c
@@ -14,6 +14,7 @@
  #include <linux/pm_domain.h>
  #include <linux/regmap.h>
  #include <linux/regulator/consumer.h>
+#include <linux/sizes.h>
  #include <dt-bindings/power/imx7-power.h>
  #include <dt-bindings/power/imx8mq-power.h>

---

My test looks like this:

make O=build_compiletest defconfig
make O=build_compiletest allmodconfig
echo CONFIG_COMPILE_TEST=y >> build_compiletest/.config
make O=build_compiletest "$@"

Other than this:

Reviewed-by: Leonard Crestez <leonard.crestez@nxp.com>