[PATCH] arm: make FORCE_MAX_ZONEORDER configurable for TI AM33XX

On 25.09.2012 02:37, Tony Lindgren wrote:

* Russell King - ARM Linux [off-list ref] [120924 16:17]:

quoted

On Mon, Sep 24, 2012 at 09:05:11PM +0200, Yegor Yefremov wrote:

quoted

On Mon, Sep 24, 2012 at 7:18 PM, Tony Lindgren [off-list ref] wrote:

quoted

* yegorslists at googlemail.com [off-list ref] [120703 07:26]:

quoted

From: Yegor Yefremov <redacted>

FORCE_MAX_ZONEORDER of 12 is needed to allocation more than 4MB
of consistent DMA memory (da8xx frame buffer driver).

Sorry for the delay on this one, looks like this one is
still valid. I'll apply it.

Thanks.

Yegor

quoted

quoted

Signed-off-by: Dejan Gacnik <redacted>
Signed-off-by: Yegor Yefremov <redacted>
---
 arch/arm/Kconfig |    5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
index e876819..ff14c1e 100644
--- a/arch/arm/Kconfig
+++ b/arch/arm/Kconfig

@@ -1725,8 +1725,9 @@ config HW_PERF_EVENTS
 source "mm/Kconfig"

 config FORCE_MAX_ZONEORDER
-     int "Maximum zone order" if ARCH_SHMOBILE
-     range 11 64 if ARCH_SHMOBILE
+     int "Maximum zone order"
+     depends on ARCH_SHMOBILE || SOC_AM33XX
+     range 11 64 if ARCH_SHMOBILE || SOC_AM33XX
      default "9" if SA1111
      default "11"

NAK.  This patch breaks SA1111 platforms.  To see why, read the patch.

OK let's drop this.

How should I change the patch to make it proper? SA1111 is broken anyway:

config FORCE_MAX_ZONEORDER
        int "Maximum zone order" if ARCH_SHMOBILE
        range 11 64 if ARCH_SHMOBILE
        default "9" if SA1111
        default "11"

AFAIK if ARCH_SHMOBILE defines dependency on ARCH_SHMOBILE, so SA1111 won't be evaluated (at least if I select SA1111 include/generated/autoconf.h shows "11"). If I add SA1111 to dependency list like this:

depends on ARCH_SHMOBILE || SOC_AM33XX || SA1111

the prompt in "Kernel features" becomes visible, but it doesn't have the default value of "9", but "11".

Am I missing something?

Yegor