* [PATCH 1/3] pinctrl: freescale: drop the dependency on ARM64 for i.MX8M
2020-03-13 3:27 [PATCH 0/3] imx: drop dependency on ARM64 peng.fan
@ 2020-03-13 3:27 ` peng.fan
2020-03-13 10:09 ` Arnd Bergmann
2020-03-25 23:23 ` Linus Walleij
2020-03-13 3:27 ` [PATCH 2/3] clk: imx: " peng.fan
2020-03-13 3:27 ` [PATCH 3/3] soc: imx: select ARM_GIC_V3 " peng.fan
2 siblings, 2 replies; 13+ messages in thread
From: peng.fan @ 2020-03-13 3:27 UTC (permalink / raw)
To: shawnguo, sboyd, s.hauer, linus.walleij, arnd
Cc: kernel, festevam, linux-imx, aisheng.dong, stefan, Anson.Huang,
abel.vesa, linux-clk, linux-arm-kernel, linux-kernel, linux-gpio,
Peng Fan
From: Peng Fan <peng.fan@nxp.com>
Moving to support aarch32 mode on aarch64 hardware, need to drop
the dependency on ARM64 to make the driver could be selected for
ARM32.
Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
drivers/pinctrl/freescale/Kconfig | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/pinctrl/freescale/Kconfig b/drivers/pinctrl/freescale/Kconfig
index de775a85a51e..c784663b00ad 100644
--- a/drivers/pinctrl/freescale/Kconfig
+++ b/drivers/pinctrl/freescale/Kconfig
@@ -125,28 +125,28 @@ config PINCTRL_IMX7ULP
config PINCTRL_IMX8MM
bool "IMX8MM pinctrl driver"
- depends on ARCH_MXC && ARM64
+ depends on ARCH_MXC
select PINCTRL_IMX
help
Say Y here to enable the imx8mm pinctrl driver
config PINCTRL_IMX8MN
bool "IMX8MN pinctrl driver"
- depends on ARCH_MXC && ARM64
+ depends on ARCH_MXC
select PINCTRL_IMX
help
Say Y here to enable the imx8mn pinctrl driver
config PINCTRL_IMX8MP
bool "IMX8MP pinctrl driver"
- depends on ARCH_MXC && ARM64
+ depends on ARCH_MXC
select PINCTRL_IMX
help
Say Y here to enable the imx8mp pinctrl driver
config PINCTRL_IMX8MQ
bool "IMX8MQ pinctrl driver"
- depends on ARCH_MXC && ARM64
+ depends on ARCH_MXC
select PINCTRL_IMX
help
Say Y here to enable the imx8mq pinctrl driver
--
2.16.4
^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH 1/3] pinctrl: freescale: drop the dependency on ARM64 for i.MX8M
2020-03-13 3:27 ` [PATCH 1/3] pinctrl: freescale: drop the dependency on ARM64 for i.MX8M peng.fan
@ 2020-03-13 10:09 ` Arnd Bergmann
2020-03-25 23:23 ` Linus Walleij
1 sibling, 0 replies; 13+ messages in thread
From: Arnd Bergmann @ 2020-03-13 10:09 UTC (permalink / raw)
To: Peng Fan
Cc: Shawn Guo, Stephen Boyd, Sascha Hauer, Linus Walleij,
Sascha Hauer, Fabio Estevam, NXP Linux Team, Dong Aisheng,
Stefan Agner, Anson Huang, Abel Vesa, linux-clk, Linux ARM,
linux-kernel, open list:GPIO SUBSYSTEM
On Fri, Mar 13, 2020 at 4:34 AM <peng.fan@nxp.com> wrote:
>
> From: Peng Fan <peng.fan@nxp.com>
>
> Moving to support aarch32 mode on aarch64 hardware, need to drop
> the dependency on ARM64 to make the driver could be selected for
> ARM32.
>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
Acked-by: Arnd Bergmann <arnd@arndb.de>
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 1/3] pinctrl: freescale: drop the dependency on ARM64 for i.MX8M
2020-03-13 3:27 ` [PATCH 1/3] pinctrl: freescale: drop the dependency on ARM64 for i.MX8M peng.fan
2020-03-13 10:09 ` Arnd Bergmann
@ 2020-03-25 23:23 ` Linus Walleij
1 sibling, 0 replies; 13+ messages in thread
From: Linus Walleij @ 2020-03-25 23:23 UTC (permalink / raw)
To: Peng Fan
Cc: Shawn Guo, Stephen Boyd, Sascha Hauer, Arnd Bergmann,
Sascha Hauer, Fabio Estevam, NXP Linux Team, Dong Aisheng,
Stefan Agner, Anson Huang, Abel Vesa, linux-clk, Linux ARM,
linux-kernel, open list:GPIO SUBSYSTEM
On Fri, Mar 13, 2020 at 4:34 AM <peng.fan@nxp.com> wrote:
> From: Peng Fan <peng.fan@nxp.com>
>
> Moving to support aarch32 mode on aarch64 hardware, need to drop
> the dependency on ARM64 to make the driver could be selected for
> ARM32.
>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
Patch applied with Arnd's ACK.
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH 2/3] clk: imx: drop the dependency on ARM64 for i.MX8M
2020-03-13 3:27 [PATCH 0/3] imx: drop dependency on ARM64 peng.fan
2020-03-13 3:27 ` [PATCH 1/3] pinctrl: freescale: drop the dependency on ARM64 for i.MX8M peng.fan
@ 2020-03-13 3:27 ` peng.fan
2020-03-13 10:09 ` Arnd Bergmann
` (2 more replies)
2020-03-13 3:27 ` [PATCH 3/3] soc: imx: select ARM_GIC_V3 " peng.fan
2 siblings, 3 replies; 13+ messages in thread
From: peng.fan @ 2020-03-13 3:27 UTC (permalink / raw)
To: shawnguo, sboyd, s.hauer, linus.walleij, arnd
Cc: kernel, festevam, linux-imx, aisheng.dong, stefan, Anson.Huang,
abel.vesa, linux-clk, linux-arm-kernel, linux-kernel, linux-gpio,
Peng Fan
From: Peng Fan <peng.fan@nxp.com>
Moving to support aarch32 mode on aarch64 hardware, need to drop
the dependency on ARM64 to make the driver could be selected for
ARM32.
Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
drivers/clk/imx/Kconfig | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/clk/imx/Kconfig b/drivers/clk/imx/Kconfig
index 01eadee88d66..db0253fa3d64 100644
--- a/drivers/clk/imx/Kconfig
+++ b/drivers/clk/imx/Kconfig
@@ -10,25 +10,25 @@ config MXC_CLK_SCU
config CLK_IMX8MM
bool "IMX8MM CCM Clock Driver"
- depends on ARCH_MXC && ARM64
+ depends on ARCH_MXC
help
Build the driver for i.MX8MM CCM Clock Driver
config CLK_IMX8MN
bool "IMX8MN CCM Clock Driver"
- depends on ARCH_MXC && ARM64
+ depends on ARCH_MXC
help
Build the driver for i.MX8MN CCM Clock Driver
config CLK_IMX8MP
bool "IMX8MP CCM Clock Driver"
- depends on ARCH_MXC && ARM64
+ depends on ARCH_MXC
help
Build the driver for i.MX8MP CCM Clock Driver
config CLK_IMX8MQ
bool "IMX8MQ CCM Clock Driver"
- depends on ARCH_MXC && ARM64
+ depends on ARCH_MXC
help
Build the driver for i.MX8MQ CCM Clock Driver
--
2.16.4
^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH 2/3] clk: imx: drop the dependency on ARM64 for i.MX8M
2020-03-13 3:27 ` [PATCH 2/3] clk: imx: " peng.fan
@ 2020-03-13 10:09 ` Arnd Bergmann
2020-03-16 17:01 ` Stephen Boyd
2020-04-14 13:37 ` Shawn Guo
2 siblings, 0 replies; 13+ messages in thread
From: Arnd Bergmann @ 2020-03-13 10:09 UTC (permalink / raw)
To: Peng Fan
Cc: Shawn Guo, Stephen Boyd, Sascha Hauer, Linus Walleij,
Sascha Hauer, Fabio Estevam, NXP Linux Team, Dong Aisheng,
Stefan Agner, Anson Huang, Abel Vesa, linux-clk, Linux ARM,
linux-kernel, open list:GPIO SUBSYSTEM
On Fri, Mar 13, 2020 at 4:34 AM <peng.fan@nxp.com> wrote:
>
> From: Peng Fan <peng.fan@nxp.com>
>
> Moving to support aarch32 mode on aarch64 hardware, need to drop
> the dependency on ARM64 to make the driver could be selected for
> ARM32.
>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
Acked-by: Arnd Bergmann <arnd@arndb.de>
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 2/3] clk: imx: drop the dependency on ARM64 for i.MX8M
2020-03-13 3:27 ` [PATCH 2/3] clk: imx: " peng.fan
2020-03-13 10:09 ` Arnd Bergmann
@ 2020-03-16 17:01 ` Stephen Boyd
2020-04-14 13:37 ` Shawn Guo
2 siblings, 0 replies; 13+ messages in thread
From: Stephen Boyd @ 2020-03-16 17:01 UTC (permalink / raw)
To: arnd, linus.walleij, peng.fan, s.hauer, shawnguo
Cc: kernel, festevam, linux-imx, aisheng.dong, stefan, Anson.Huang,
abel.vesa, linux-clk, linux-arm-kernel, linux-kernel, linux-gpio,
Peng Fan
Quoting peng.fan@nxp.com (2020-03-12 20:27:15)
> From: Peng Fan <peng.fan@nxp.com>
>
> Moving to support aarch32 mode on aarch64 hardware, need to drop
> the dependency on ARM64 to make the driver could be selected for
> ARM32.
>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> ---
Acked-by: Stephen Boyd <sboyd@kernel.org>
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 2/3] clk: imx: drop the dependency on ARM64 for i.MX8M
2020-03-13 3:27 ` [PATCH 2/3] clk: imx: " peng.fan
2020-03-13 10:09 ` Arnd Bergmann
2020-03-16 17:01 ` Stephen Boyd
@ 2020-04-14 13:37 ` Shawn Guo
2 siblings, 0 replies; 13+ messages in thread
From: Shawn Guo @ 2020-04-14 13:37 UTC (permalink / raw)
To: peng.fan
Cc: sboyd, s.hauer, linus.walleij, arnd, kernel, festevam, linux-imx,
aisheng.dong, stefan, Anson.Huang, abel.vesa, linux-clk,
linux-arm-kernel, linux-kernel, linux-gpio
On Fri, Mar 13, 2020 at 11:27:15AM +0800, peng.fan@nxp.com wrote:
> From: Peng Fan <peng.fan@nxp.com>
>
> Moving to support aarch32 mode on aarch64 hardware, need to drop
> the dependency on ARM64 to make the driver could be selected for
> ARM32.
>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
Applied, thanks.
^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH 3/3] soc: imx: select ARM_GIC_V3 for i.MX8M
2020-03-13 3:27 [PATCH 0/3] imx: drop dependency on ARM64 peng.fan
2020-03-13 3:27 ` [PATCH 1/3] pinctrl: freescale: drop the dependency on ARM64 for i.MX8M peng.fan
2020-03-13 3:27 ` [PATCH 2/3] clk: imx: " peng.fan
@ 2020-03-13 3:27 ` peng.fan
2020-03-13 10:11 ` Arnd Bergmann
2 siblings, 1 reply; 13+ messages in thread
From: peng.fan @ 2020-03-13 3:27 UTC (permalink / raw)
To: shawnguo, sboyd, s.hauer, linus.walleij, arnd
Cc: kernel, festevam, linux-imx, aisheng.dong, stefan, Anson.Huang,
abel.vesa, linux-clk, linux-arm-kernel, linux-kernel, linux-gpio,
Peng Fan
From: Peng Fan <peng.fan@nxp.com>
Select ARM_GIC_V3, then it is able to use gic v3 driver in aarch32
mode linux on aarch64 hardware. For aarch64 mode, it not hurts
to select ARM_GIC_V3.
Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
drivers/soc/imx/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/soc/imx/Kconfig b/drivers/soc/imx/Kconfig
index 70019cefa617..0b69024296d5 100644
--- a/drivers/soc/imx/Kconfig
+++ b/drivers/soc/imx/Kconfig
@@ -21,6 +21,7 @@ config SOC_IMX8M
bool "i.MX8M SoC family support"
depends on ARCH_MXC || COMPILE_TEST
default ARCH_MXC && ARM64
+ select ARM_GIC_V3
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,
--
2.16.4
^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH 3/3] soc: imx: select ARM_GIC_V3 for i.MX8M
2020-03-13 3:27 ` [PATCH 3/3] soc: imx: select ARM_GIC_V3 " peng.fan
@ 2020-03-13 10:11 ` Arnd Bergmann
2020-03-16 6:43 ` Peng Fan
0 siblings, 1 reply; 13+ messages in thread
From: Arnd Bergmann @ 2020-03-13 10:11 UTC (permalink / raw)
To: Peng Fan
Cc: Shawn Guo, Stephen Boyd, Sascha Hauer, Linus Walleij,
Sascha Hauer, Fabio Estevam, NXP Linux Team, Dong Aisheng,
Stefan Agner, Anson Huang, Abel Vesa, linux-clk, Linux ARM,
linux-kernel, open list:GPIO SUBSYSTEM
On Fri, Mar 13, 2020 at 4:34 AM <peng.fan@nxp.com> wrote:
>
> From: Peng Fan <peng.fan@nxp.com>
>
> Select ARM_GIC_V3, then it is able to use gic v3 driver in aarch32
> mode linux on aarch64 hardware. For aarch64 mode, it not hurts
> to select ARM_GIC_V3.
>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> ---
Acked-by: Arnd Bergmann <arnd@arndb.de>
> diff --git a/drivers/soc/imx/Kconfig b/drivers/soc/imx/Kconfig
> index 70019cefa617..0b69024296d5 100644
> --- a/drivers/soc/imx/Kconfig
> +++ b/drivers/soc/imx/Kconfig
> @@ -21,6 +21,7 @@ config SOC_IMX8M
> bool "i.MX8M SoC family support"
> depends on ARCH_MXC || COMPILE_TEST
> default ARCH_MXC && ARM64
> + select ARM_GIC_V3
It would seem sensible to also drop the dependency on the 'default'
Arnd
^ permalink raw reply [flat|nested] 13+ messages in thread
* RE: [PATCH 3/3] soc: imx: select ARM_GIC_V3 for i.MX8M
2020-03-13 10:11 ` Arnd Bergmann
@ 2020-03-16 6:43 ` Peng Fan
2020-03-16 8:05 ` Arnd Bergmann
0 siblings, 1 reply; 13+ messages in thread
From: Peng Fan @ 2020-03-16 6:43 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Aisheng Dong, Abel Vesa, Anson Huang, Stephen Boyd, Shawn Guo,
Sascha Hauer, linux-kernel, Stefan Agner, linux-clk,
open list:GPIO SUBSYSTEM, dl-linux-imx, Sascha Hauer,
Fabio Estevam, Linus Walleij, Linux ARM
Hi Arnd,
> Subject: Re: [PATCH 3/3] soc: imx: select ARM_GIC_V3 for i.MX8M
>
> On Fri, Mar 13, 2020 at 4:34 AM <peng.fan@nxp.com> wrote:
> >
> > From: Peng Fan <peng.fan@nxp.com>
> >
> > Select ARM_GIC_V3, then it is able to use gic v3 driver in aarch32
> > mode linux on aarch64 hardware. For aarch64 mode, it not hurts to
> > select ARM_GIC_V3.
> >
> > Signed-off-by: Peng Fan <peng.fan@nxp.com>
> > ---
>
> Acked-by: Arnd Bergmann <arnd@arndb.de>
>
> > diff --git a/drivers/soc/imx/Kconfig b/drivers/soc/imx/Kconfig index
> > 70019cefa617..0b69024296d5 100644
> > --- a/drivers/soc/imx/Kconfig
> > +++ b/drivers/soc/imx/Kconfig
> > @@ -21,6 +21,7 @@ config SOC_IMX8M
> > bool "i.MX8M SoC family support"
> > depends on ARCH_MXC || COMPILE_TEST
> > default ARCH_MXC && ARM64
> > + select ARM_GIC_V3
>
> It would seem sensible to also drop the dependency on the 'default'
If drop default, we need enable this config option in ARM64 defconfig,
I would leave it as is for now.
Thanks,
Peng.
>
> Arnd
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Flists.infr
> adead.org%2Fmailman%2Flistinfo%2Flinux-arm-kernel&data=02%7C01
> %7Cpeng.fan%40nxp.com%7Ca28680bc024f478a8c4008d7c736ef64%7C686
> ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C637196911074143592&a
> mp;sdata=jNEcrVlci7UEAhAftKBNSVc4b6%2F0Sm2aOYPIA9ajZl8%3D&re
> served=0
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 3/3] soc: imx: select ARM_GIC_V3 for i.MX8M
2020-03-16 6:43 ` Peng Fan
@ 2020-03-16 8:05 ` Arnd Bergmann
2020-03-17 1:43 ` Peng Fan
0 siblings, 1 reply; 13+ messages in thread
From: Arnd Bergmann @ 2020-03-16 8:05 UTC (permalink / raw)
To: Peng Fan
Cc: Aisheng Dong, Abel Vesa, Anson Huang, Stephen Boyd, Shawn Guo,
Sascha Hauer, linux-kernel, Stefan Agner, linux-clk,
open list:GPIO SUBSYSTEM, dl-linux-imx, Sascha Hauer,
Fabio Estevam, Linus Walleij, Linux ARM
On Mon, Mar 16, 2020 at 7:43 AM Peng Fan <peng.fan@nxp.com> wrote:
> > Subject: Re: [PATCH 3/3] soc: imx: select ARM_GIC_V3 for i.MX8M
> >
> > On Fri, Mar 13, 2020 at 4:34 AM <peng.fan@nxp.com> wrote:
> > >
> > > From: Peng Fan <peng.fan@nxp.com>
> > >
> > > Select ARM_GIC_V3, then it is able to use gic v3 driver in aarch32
> > > mode linux on aarch64 hardware. For aarch64 mode, it not hurts to
> > > select ARM_GIC_V3.
> > >
> > > Signed-off-by: Peng Fan <peng.fan@nxp.com>
> > > ---
> >
> > Acked-by: Arnd Bergmann <arnd@arndb.de>
> >
> > > diff --git a/drivers/soc/imx/Kconfig b/drivers/soc/imx/Kconfig index
> > > 70019cefa617..0b69024296d5 100644
> > > --- a/drivers/soc/imx/Kconfig
> > > +++ b/drivers/soc/imx/Kconfig
> > > @@ -21,6 +21,7 @@ config SOC_IMX8M
> > > bool "i.MX8M SoC family support"
> > > depends on ARCH_MXC || COMPILE_TEST
> > > default ARCH_MXC && ARM64
> > > + select ARM_GIC_V3
> >
> > It would seem sensible to also drop the dependency on the 'default'
>
> If drop default, we need enable this config option in ARM64 defconfig,
> I would leave it as is for now.
I meant making it 'default ARCH_MXC' so it gets enabled for both
32-bit and 64-bit i.MX configurations, not just 64-bit.
Arnd
^ permalink raw reply [flat|nested] 13+ messages in thread
* RE: [PATCH 3/3] soc: imx: select ARM_GIC_V3 for i.MX8M
2020-03-16 8:05 ` Arnd Bergmann
@ 2020-03-17 1:43 ` Peng Fan
0 siblings, 0 replies; 13+ messages in thread
From: Peng Fan @ 2020-03-17 1:43 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Aisheng Dong, Abel Vesa, Anson Huang, Stephen Boyd, Shawn Guo,
Sascha Hauer, linux-kernel, Stefan Agner, linux-clk,
open list:GPIO SUBSYSTEM, dl-linux-imx, Sascha Hauer,
Fabio Estevam, Linus Walleij, Linux ARM
Hi Arnd,
> Subject: Re: [PATCH 3/3] soc: imx: select ARM_GIC_V3 for i.MX8M
>
> On Mon, Mar 16, 2020 at 7:43 AM Peng Fan <peng.fan@nxp.com> wrote:
> > > Subject: Re: [PATCH 3/3] soc: imx: select ARM_GIC_V3 for i.MX8M
> > >
> > > On Fri, Mar 13, 2020 at 4:34 AM <peng.fan@nxp.com> wrote:
> > > >
> > > > From: Peng Fan <peng.fan@nxp.com>
> > > >
> > > > Select ARM_GIC_V3, then it is able to use gic v3 driver in aarch32
> > > > mode linux on aarch64 hardware. For aarch64 mode, it not hurts to
> > > > select ARM_GIC_V3.
> > > >
> > > > Signed-off-by: Peng Fan <peng.fan@nxp.com>
> > > > ---
> > >
> > > Acked-by: Arnd Bergmann <arnd@arndb.de>
> > >
> > > > diff --git a/drivers/soc/imx/Kconfig b/drivers/soc/imx/Kconfig
> > > > index
> > > > 70019cefa617..0b69024296d5 100644
> > > > --- a/drivers/soc/imx/Kconfig
> > > > +++ b/drivers/soc/imx/Kconfig
> > > > @@ -21,6 +21,7 @@ config SOC_IMX8M
> > > > bool "i.MX8M SoC family support"
> > > > depends on ARCH_MXC || COMPILE_TEST
> > > > default ARCH_MXC && ARM64
> > > > + select ARM_GIC_V3
> > >
> > > It would seem sensible to also drop the dependency on the 'default'
> >
> > If drop default, we need enable this config option in ARM64 defconfig,
> > I would leave it as is for now.
>
> I meant making it 'default ARCH_MXC' so it gets enabled for both 32-bit and
> 64-bit i.MX configurations, not just 64-bit.
Understand. I'll drop ARM64 and post v2 with your A-b.
Thanks,
Peng.
>
> Arnd
^ permalink raw reply [flat|nested] 13+ messages in thread