[PATCH v2] multi_v7_defconfig: Remove CONFIG_SOC_LS1021A

[PATCH v2] multi_v7_defconfig: Remove CONFIG_SOC_LS1021A

[Fabio Estevam]

multi_v7_defconfig is not suitable for booting LS1021 as it needs
LPAE.
    
Remove this option.

Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
Changes since v1:
- Improve commit log

 arch/arm/configs/multi_v7_defconfig | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig
index fa7ba92..410a699 100644
--- a/arch/arm/configs/multi_v7_defconfig
+++ b/arch/arm/configs/multi_v7_defconfig
@@ -54,7 +54,6 @@ CONFIG_SOC_IMX6SX=y
 CONFIG_SOC_IMX6UL=y
 CONFIG_SOC_IMX7D=y
 CONFIG_SOC_VF610=y
-CONFIG_SOC_LS1021A=y
 CONFIG_ARCH_OMAP3=y
 CONFIG_ARCH_OMAP4=y
 CONFIG_SOC_OMAP5=y
-- 
1.9.1

[PATCH v2] multi_v7_defconfig: Remove CONFIG_SOC_LS1021A

[Arnd Bergmann]

On Tuesday 01 December 2015 09:35:45 Fabio Estevam wrote:
> multi_v7_defconfig is not suitable for booting LS1021 as it needs
> LPAE.
>     
> Remove this option.
> 
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>

This needs a better explanation. Why do we need LPAE for this
platform?

	Arnd

[PATCH v2] multi_v7_defconfig: Remove CONFIG_SOC_LS1021A

[Huan Wang]

> On Tuesday 01 December 2015 09:35:45 Fabio Estevam wrote:
> > multi_v7_defconfig is not suitable for booting LS1021 as it needs
> > LPAE.
> >
> > Remove this option.
> >
> > Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> 
> This needs a better explanation. Why do we need LPAE for this platform?
> 
>
[Alison Wang] Because PCIE, IFC region 2, DRAM2 and other IPs on LS1021A
are in the 40-bit address space, so we need to enable LPAE to access those
memory beyond the 4GB limit.

Let me summarize the current status. I have made multi_v7_lpae_defconfig to
support LS1021A according to the suggestion from Russell King. Olof Johansson
said they have discussed it before and they have chosen to just have a
multi_v7_defconfig. Attached is the discussion. There is not further
solution about the discussion.

So now, I suggest to remain LS1021A in multi_v7_defconfig at least.

Best Regards,
Alison Wang
-------------- next part --------------
An embedded message was scrubbed...
From: Huan Wang <alison.wang@freescale.com>
Subject: RE: [RFC PATCH] ARM: add v7 LPAE multi-platform defconfig
Date: Tue, 10 Nov 2015 07:19:00 +0000
Size: 7804
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20151202/63f3598b/attachment.mht>

[PATCH v2] multi_v7_defconfig: Remove CONFIG_SOC_LS1021A

[Arnd Bergmann]

On Wednesday 02 December 2015 02:12:18 Huan Wang wrote:
> > On Tuesday 01 December 2015 09:35:45 Fabio Estevam wrote:
> > > multi_v7_defconfig is not suitable for booting LS1021 as it needs
> > > LPAE.
> > >
> > > Remove this option.
> > >
> > > Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> > 
> > This needs a better explanation. Why do we need LPAE for this platform?
> > 
> >
> [Alison Wang] Because PCIE, IFC region 2, DRAM2 and other IPs on LS1021A
> are in the 40-bit address space, so we need to enable LPAE to access those
> memory beyond the 4GB limit.
> 
> Let me summarize the current status. I have made multi_v7_lpae_defconfig to
> support LS1021A according to the suggestion from Russell King. Olof Johansson
> said they have discussed it before and they have chosen to just have a
> multi_v7_defconfig. Attached is the discussion. There is not further
> solution about the discussion.
> 
> So now, I suggest to remain LS1021A in multi_v7_defconfig at least.

I think that's fine for the time being. We are still looking for
a good way to deal with LPAE configurations in general.

	Arnd