* [PATCH] ARM: Kconfig: Open force maximum zone order for all SoCs
@ 2014-09-01 4:42 Xiubo Li
2014-09-26 5:30 ` Li.Xiubo at freescale.com
0 siblings, 1 reply; 6+ messages in thread
From: Xiubo Li @ 2014-09-01 4:42 UTC (permalink / raw)
To: linux-arm-kernel
Signed-off-by: Xiubo Li <Li.Xiubo@freescale.com>
---
arch/arm/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
index c49a775..bf8445c 100644
--- a/arch/arm/Kconfig
+++ b/arch/arm/Kconfig
@@ -1687,7 +1687,7 @@ config ARCH_WANT_GENERAL_HUGETLB
source "mm/Kconfig"
config FORCE_MAX_ZONEORDER
- int "Maximum zone order" if ARCH_SHMOBILE_LEGACY
+ int "Maximum zone order"
range 11 64 if ARCH_SHMOBILE_LEGACY
default "12" if SOC_AM33XX
default "9" if SA1111 || ARCH_EFM32
--
1.8.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH] ARM: Kconfig: Open force maximum zone order for all SoCs
2014-09-01 4:42 [PATCH] ARM: Kconfig: Open force maximum zone order for all SoCs Xiubo Li
@ 2014-09-26 5:30 ` Li.Xiubo at freescale.com
2014-09-26 8:20 ` Russell King - ARM Linux
0 siblings, 1 reply; 6+ messages in thread
From: Li.Xiubo at freescale.com @ 2014-09-26 5:30 UTC (permalink / raw)
To: linux-arm-kernel
Hi Russell,
I'd like to know the status of this patch.
Our Platforms depend on it.
Thanks,
BRs
Xiubo
> -----Original Message-----
> From: Xiubo Li [mailto:Li.Xiubo at freescale.com]
> Sent: Monday, September 01, 2014 12:42 PM
> To: linux at arm.linux.org.uk; linux-arm-kernel at lists.infradead.org
> Cc: linux-kernel at vger.kernel.org; Xiubo Li-B47053
> Subject: [PATCH] ARM: Kconfig: Open force maximum zone order for all SoCs
>
> Signed-off-by: Xiubo Li <Li.Xiubo@freescale.com>
> ---
> arch/arm/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> index c49a775..bf8445c 100644
> --- a/arch/arm/Kconfig
> +++ b/arch/arm/Kconfig
> @@ -1687,7 +1687,7 @@ config ARCH_WANT_GENERAL_HUGETLB
> source "mm/Kconfig"
>
> config FORCE_MAX_ZONEORDER
> - int "Maximum zone order" if ARCH_SHMOBILE_LEGACY
> + int "Maximum zone order"
> range 11 64 if ARCH_SHMOBILE_LEGACY
> default "12" if SOC_AM33XX
> default "9" if SA1111 || ARCH_EFM32
> --
> 1.8.4
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] ARM: Kconfig: Open force maximum zone order for all SoCs
2014-09-26 5:30 ` Li.Xiubo at freescale.com
@ 2014-09-26 8:20 ` Russell King - ARM Linux
2014-09-26 8:39 ` Li.Xiubo at freescale.com
0 siblings, 1 reply; 6+ messages in thread
From: Russell King - ARM Linux @ 2014-09-26 8:20 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Sep 26, 2014 at 05:30:40AM +0000, Li.Xiubo at freescale.com wrote:
> Hi Russell,
>
> I'd like to know the status of this patch.
>
> Our Platforms depend on it.
I'm not applying it. This option should not be exposed for general
use. The usage pattern is clear from the existing users - propose
an alternative default value for your platform rather than making it
a user visible tweakable option.
--
FTTC broadband for 0.8mile line: currently at 9.5Mbps down 400kbps up
according to speedtest.net.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] ARM: Kconfig: Open force maximum zone order for all SoCs
2014-09-26 8:20 ` Russell King - ARM Linux
@ 2014-09-26 8:39 ` Li.Xiubo at freescale.com
2014-09-26 8:44 ` Russell King - ARM Linux
0 siblings, 1 reply; 6+ messages in thread
From: Li.Xiubo at freescale.com @ 2014-09-26 8:39 UTC (permalink / raw)
To: linux-arm-kernel
[...]
> On Fri, Sep 26, 2014 at 05:30:40AM +0000, Li.Xiubo at freescale.com wrote:
> > Hi Russell,
> >
> > I'd like to know the status of this patch.
> >
> > Our Platforms depend on it.
>
> I'm not applying it. This option should not be exposed for general
> use. The usage pattern is clear from the existing users - propose
> an alternative default value for your platform rather than making it
> a user visible tweakable option.
>
Okay.
Something like :
++++++++++
diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
index 1ad6fb6..5729a2c 100644
--- a/arch/arm/Kconfig
+++ b/arch/arm/Kconfig
@@ -1802,7 +1802,7 @@ source "mm/Kconfig"
config FORCE_MAX_ZONEORDER
int "Maximum zone order" if ARCH_SHMOBILE
range 11 64 if ARCH_SHMOBILE
- default "12" if SOC_AM33XX
+ default "12" if SOC_AM33XX || SOC_LS1021A
default "9" if SA1111
default "11"
help
----------
Is okay ?
Thanks,
BRs
Xiubo
> --
> FTTC broadband for 0.8mile line: currently at 9.5Mbps down 400kbps up
> according to speedtest.net.
^ permalink raw reply related [flat|nested] 6+ messages in thread* [PATCH] ARM: Kconfig: Open force maximum zone order for all SoCs
2014-09-26 8:39 ` Li.Xiubo at freescale.com
@ 2014-09-26 8:44 ` Russell King - ARM Linux
2014-09-26 8:46 ` Li.Xiubo at freescale.com
0 siblings, 1 reply; 6+ messages in thread
From: Russell King - ARM Linux @ 2014-09-26 8:44 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Sep 26, 2014 at 08:39:04AM +0000, Li.Xiubo at freescale.com wrote:
> [...]
> > On Fri, Sep 26, 2014 at 05:30:40AM +0000, Li.Xiubo at freescale.com wrote:
> > > Hi Russell,
> > >
> > > I'd like to know the status of this patch.
> > >
> > > Our Platforms depend on it.
> >
> > I'm not applying it. This option should not be exposed for general
> > use. The usage pattern is clear from the existing users - propose
> > an alternative default value for your platform rather than making it
> > a user visible tweakable option.
> >
>
> Okay.
>
> Something like :
> ++++++++++
> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> index 1ad6fb6..5729a2c 100644
> --- a/arch/arm/Kconfig
> +++ b/arch/arm/Kconfig
> @@ -1802,7 +1802,7 @@ source "mm/Kconfig"
> config FORCE_MAX_ZONEORDER
> int "Maximum zone order" if ARCH_SHMOBILE
> range 11 64 if ARCH_SHMOBILE
> - default "12" if SOC_AM33XX
> + default "12" if SOC_AM33XX || SOC_LS1021A
> default "9" if SA1111
> default "11"
> help
> ----------
>
> Is okay ?
It's much better than exposing the option, but I'd like this patch to
appear as a properly submitted change (including a change log which
provides the reason why this is necessary.)
Thanks.
--
FTTC broadband for 0.8mile line: currently at 9.5Mbps down 400kbps up
according to speedtest.net.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] ARM: Kconfig: Open force maximum zone order for all SoCs
2014-09-26 8:44 ` Russell King - ARM Linux
@ 2014-09-26 8:46 ` Li.Xiubo at freescale.com
0 siblings, 0 replies; 6+ messages in thread
From: Li.Xiubo at freescale.com @ 2014-09-26 8:46 UTC (permalink / raw)
To: linux-arm-kernel
> Subject: Re: [PATCH] ARM: Kconfig: Open force maximum zone order for all SoCs
>
> On Fri, Sep 26, 2014 at 08:39:04AM +0000, Li.Xiubo at freescale.com wrote:
> > [...]
> > > On Fri, Sep 26, 2014 at 05:30:40AM +0000, Li.Xiubo at freescale.com wrote:
> > > > Hi Russell,
> > > >
> > > > I'd like to know the status of this patch.
> > > >
> > > > Our Platforms depend on it.
> > >
> > > I'm not applying it. This option should not be exposed for general
> > > use. The usage pattern is clear from the existing users - propose
> > > an alternative default value for your platform rather than making it
> > > a user visible tweakable option.
> > >
> >
> > Okay.
> >
> > Something like :
> > ++++++++++
> > diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> > index 1ad6fb6..5729a2c 100644
> > --- a/arch/arm/Kconfig
> > +++ b/arch/arm/Kconfig
> > @@ -1802,7 +1802,7 @@ source "mm/Kconfig"
> > config FORCE_MAX_ZONEORDER
> > int "Maximum zone order" if ARCH_SHMOBILE
> > range 11 64 if ARCH_SHMOBILE
> > - default "12" if SOC_AM33XX
> > + default "12" if SOC_AM33XX || SOC_LS1021A
> > default "9" if SA1111
> > default "11"
> > help
> > ----------
> >
> > Is okay ?
>
> It's much better than exposing the option, but I'd like this patch to
> appear as a properly submitted change (including a change log which
> provides the reason why this is necessary.)
>
Okay, I will.
Thanks,
BRs
Xiubo
> Thanks.
>
> --
> FTTC broadband for 0.8mile line: currently at 9.5Mbps down 400kbps up
> according to speedtest.net.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2014-09-26 8:46 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-09-01 4:42 [PATCH] ARM: Kconfig: Open force maximum zone order for all SoCs Xiubo Li
2014-09-26 5:30 ` Li.Xiubo at freescale.com
2014-09-26 8:20 ` Russell King - ARM Linux
2014-09-26 8:39 ` Li.Xiubo at freescale.com
2014-09-26 8:44 ` Russell King - ARM Linux
2014-09-26 8:46 ` Li.Xiubo at freescale.com
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).