Skip to contentSkip to conversation

multi_v7_defconfig: Remove CONFIG_SOC_LS1021A

PATCH New Fabio Estevam 3 participants 4 messages Dec 1, 2015 – Dec 2, 2015 Arm (32-bit)

lore
Share & tools
lore
#0 lore raw

PATCH v2 1/1 multi_v7_defconfig: Remove CONFIG_SOC_LS1021A

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

Remove this option.

---
Changes since v1:
- Improve commit log

 arch/arm/configs/multi_v7_defconfig | 1 -
 1 file changed, 1 deletion(-)
arch/arm/configs/multi_v7_defconfig +0 -1
index fa7ba92..410a699 100644
@@ -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
Signature · 1 line
1.9.1
#1 lore raw

PATCH v2 1/1 multi_v7_defconfig: Remove CONFIG_SOC_LS1021A

Quoted 6 lines · 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.

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

Arnd

#2 lore raw

PATCH v2 1/1 multi_v7_defconfig: Remove CONFIG_SOC_LS1021A

Quoted 12 lines · On Tuesday 01 December 2015 09:35:45 Fabio Estevam wrote:
Quoted 6 lines

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

Remove this option.

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

#3 lore raw

PATCH v2 1/1 multi_v7_defconfig: Remove CONFIG_SOC_LS1021A

Quoted 15 lines · On Wednesday 02 December 2015 02:12:18 Huan Wang wrote:
Quoted 11 lines · On Tuesday 01 December 2015 09:35:45 Fabio Estevam wrote:
Quoted 6 lines

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

Remove this option.

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.

Quoted 7 lines

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