Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
From:   Leonard Crestez <leonard.crestez@nxp.com>
To:     Peng Fan <peng.fan@nxp.com>,
        "shawnguo@kernel.org" <shawnguo@kernel.org>,
        "arnd@arndb.de" <arnd@arndb.de>
CC:     "s.hauer@pengutronix.de" <s.hauer@pengutronix.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>,
        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>
Subject: Re: [PATCH V3 2/2] soc: imx: increase build coverage for imx8m soc
 driver
Thread-Topic: [PATCH V3 2/2] soc: imx: increase build coverage for imx8m soc
 driver
Thread-Index: AQHV1aCjGc6WVJY+NES5jeQSOsfpxg==
Date:   Tue, 28 Jan 2020 14:35:41 +0000
Message-ID: <VI1PR04MB70231AC9140BD13FD0539678EE0A0@VI1PR04MB7023.eurprd04.prod.outlook.com>
References: <1580191098-5886-1-git-send-email-peng.fan@nxp.com>
 <1580191098-5886-3-git-send-email-peng.fan@nxp.com>
Accept-Language: en-US
Content-Language: en-US
received-spf: None (protection.outlook.com: nxp.com does not designate
 permitted sender hosts)
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: bc1f9f5e-cf20-4698-d8fe-08d7a3ff5a29
X-MS-Exchange-CrossTenant-originalarrivaltime: 28 Jan 2020 14:35:41.9142
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: zWtMO3SEnmqZ3QGKtZDmmVApdcInF4P6zzFHf+daNdH3nvhYJPIzmoD/EEG6ct54Iw5vAXBTNw0FvO/4WQyxWw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR04MB5599
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 28.01.2020 08:03, Peng Fan wrote:=0A=
> From: Peng Fan <peng.fan@nxp.com>=0A=
> =0A=
> The soc-imx8.c driver is actually for i.MX8M family, so rename it=0A=
> to soc-imx8m.c.=0A=
> =0A=
> Use CONFIG_SOC_IMX8M as build gate, not CONFIG_ARCH_MXC, to control=0A=
> whether build this driver, also make it possible for compile test.=0A=
> =0A=
> Default set it to y for ARCH_MXC && ARM64=0A=
> =0A=
> Signed-off-by: Peng Fan <peng.fan@nxp.com>=0A=
=0A=
Reviewed-by: Leonard Crestez <leonard.crestez@nxp.com>=0A=
=0A=
> ---=0A=
>   drivers/soc/Makefile                        | 2 +-=0A=
>   drivers/soc/imx/Kconfig                     | 9 +++++++++=0A=
>   drivers/soc/imx/Makefile                    | 2 +-=0A=
>   drivers/soc/imx/{soc-imx8.c =3D> soc-imx8m.c} | 0=0A=
>   4 files changed, 11 insertions(+), 2 deletions(-)=0A=
>   rename drivers/soc/imx/{soc-imx8.c =3D> soc-imx8m.c} (100%)=0A=
> =0A=
> diff --git a/drivers/soc/Makefile b/drivers/soc/Makefile=0A=
> index 2ec355003524..614986cd1713 100644=0A=
> --- a/drivers/soc/Makefile=0A=
> +++ b/drivers/soc/Makefile=0A=
> @@ -11,7 +11,7 @@ obj-$(CONFIG_ARCH_DOVE)		+=3D dove/=0A=
>   obj-$(CONFIG_MACH_DOVE)		+=3D dove/=0A=
>   obj-y				+=3D fsl/=0A=
>   obj-$(CONFIG_ARCH_GEMINI)	+=3D gemini/=0A=
> -obj-$(CONFIG_ARCH_MXC)		+=3D imx/=0A=
> +obj-y				+=3D imx/=0A=
>   obj-$(CONFIG_ARCH_IXP4XX)	+=3D ixp4xx/=0A=
>   obj-$(CONFIG_SOC_XWAY)		+=3D lantiq/=0A=
>   obj-y				+=3D mediatek/=0A=
> diff --git a/drivers/soc/imx/Kconfig b/drivers/soc/imx/Kconfig=0A=
> index 0281ef9a1800..70019cefa617 100644=0A=
> --- a/drivers/soc/imx/Kconfig=0A=
> +++ b/drivers/soc/imx/Kconfig=0A=
> @@ -17,4 +17,13 @@ config IMX_SCU_SOC=0A=
>   	  Controller Unit SoC info module, it will provide the SoC info=0A=
>   	  like SoC family, ID and revision etc.=0A=
>   =0A=
> +config SOC_IMX8M=0A=
> +	bool "i.MX8M SoC family support"=0A=
> +	depends on ARCH_MXC || COMPILE_TEST=0A=
> +	default ARCH_MXC && ARM64=0A=
> +	help=0A=
> +	  If you say yes here you get support for the NXP i.MX8M family=0A=
> +	  support, it will provide the SoC info like SoC family,=0A=
> +	  ID and revision etc.=0A=
> +=0A=
>   endmenu=0A=
> diff --git a/drivers/soc/imx/Makefile b/drivers/soc/imx/Makefile=0A=
> index cf9ca42ff739..103e2c93c342 100644=0A=
> --- a/drivers/soc/imx/Makefile=0A=
> +++ b/drivers/soc/imx/Makefile=0A=
> @@ -1,5 +1,5 @@=0A=
>   # SPDX-License-Identifier: GPL-2.0-only=0A=
>   obj-$(CONFIG_HAVE_IMX_GPC) +=3D gpc.o=0A=
>   obj-$(CONFIG_IMX_GPCV2_PM_DOMAINS) +=3D gpcv2.o=0A=
> -obj-$(CONFIG_ARCH_MXC) +=3D soc-imx8.o=0A=
> +obj-$(CONFIG_SOC_IMX8M) +=3D soc-imx8m.o=0A=
>   obj-$(CONFIG_IMX_SCU_SOC) +=3D soc-imx-scu.o=0A=
> diff --git a/drivers/soc/imx/soc-imx8.c b/drivers/soc/imx/soc-imx8m.c=0A=
> similarity index 100%=0A=
> rename from drivers/soc/imx/soc-imx8.c=0A=
> rename to drivers/soc/imx/soc-imx8m.c=0A=
=0A=