| Message ID | 1323661352-10291-7-git-send-email-richard.zhao@linaro.org |
|---|---|
| State | Changes Requested |
| Headers | show |
On Mon, Dec 12, 2011 at 11:42:31AM +0800, Richard Zhao wrote: > Signed-off-by: Richard Zhao <richard.zhao@linaro.org> > --- > arch/arm/mach-imx/Kconfig | 1 + > arch/arm/mach-imx/Makefile | 2 +- > arch/arm/mach-imx/cpu_op-imx6q.c | 78 ++++++++++++++++++++++++++++++++++++++ > 3 files changed, 80 insertions(+), 1 deletions(-) > create mode 100644 arch/arm/mach-imx/cpu_op-imx6q.c > > diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig > index fbd414b..7d813a3 100644 > --- a/arch/arm/mach-imx/Kconfig > +++ b/arch/arm/mach-imx/Kconfig > @@ -837,6 +837,7 @@ config SOC_IMX6Q > select ARM_GIC > select CACHE_L2X0 > select CPU_V7 > + select ARCH_HAS_CPUFREQ > select HAVE_ARM_SCU > select HAVE_IMX_GPC > select HAVE_IMX_MMDC > diff --git a/arch/arm/mach-imx/Makefile b/arch/arm/mach-imx/Makefile > index 9cf630a..2dfe4a7 100644 > --- a/arch/arm/mach-imx/Makefile > +++ b/arch/arm/mach-imx/Makefile > @@ -72,7 +72,7 @@ AFLAGS_head-v7.o :=-Wa,-march=armv7-a > obj-$(CONFIG_SMP) += platsmp.o > obj-$(CONFIG_HOTPLUG_CPU) += hotplug.o > obj-$(CONFIG_LOCAL_TIMERS) += localtimer.o > -obj-$(CONFIG_SOC_IMX6Q) += clock-imx6q.o mach-imx6q.o pm-imx6q.o > +obj-$(CONFIG_SOC_IMX6Q) += clock-imx6q.o mach-imx6q.o pm-imx6q.o cpu_op-imx6q.o > > # i.MX5 based machines > obj-$(CONFIG_MACH_MX51_BABBAGE) += mach-mx51_babbage.o > diff --git a/arch/arm/mach-imx/cpu_op-imx6q.c b/arch/arm/mach-imx/cpu_op-imx6q.c > new file mode 100644 > index 0000000..8807b96 > --- /dev/null > +++ b/arch/arm/mach-imx/cpu_op-imx6q.c > @@ -0,0 +1,78 @@ > +/* > + * Copyright (C) 2010-2011 Freescale Semiconductor, Inc. All Rights Reserved. > + */ > + > +/* > + * The code contained herein is licensed under the GNU General Public > + * License. You may obtain a copy of the GNU General Public License > + * Version 2 or later at the following locations: > + * > + * http://www.opensource.org/licenses/gpl-license.html > + * http://www.gnu.org/copyleft/gpl.html > + */ > + > +#include <linux/types.h> > +#include <linux/kernel.h> > +#include <linux/string.h> > +#include <mach/hardware.h> > + > +static u32 arm_max_freq; > +static int num_cpu_op; Unused variable. > + > +/* working point(wp): 0 - 1GHzMHz; 1 - 800MHz, 3 - 400MHz, 4 - 160MHz */ s/GHzMHz/GHz/ > +static struct cpu_op mx6q_cpu_op_1G[] = { > + { > + .cpu_rate = 996000000, > + .cpu_voltage = 1225000,}, > + { Strange indention. > + .cpu_rate = 792000000, > + .cpu_voltage = 1100000,}, > + { > + .cpu_rate = 396000000, > + .cpu_voltage = 950000,}, > + { > + .cpu_rate = 198000000, > + .cpu_voltage = 850000,}, > +}; > + > +static struct cpu_op mx6q_cpu_op[] = { > + { > + .cpu_rate = 792000000, > + .cpu_voltage = 1100000,}, > + { > + .cpu_rate = 396000000, > + .cpu_voltage = 950000,}, > + { > + .cpu_rate = 198000000, > + .cpu_voltage = 850000,}, > +}; > + > +struct cpu_op *mx6q_get_cpu_op(int *op) > +{ > + if (arm_max_freq == 1000) { > + *op = num_cpu_op = ARRAY_SIZE(mx6q_cpu_op_1G); > + return mx6q_cpu_op_1G; > + } else { > + *op = num_cpu_op = ARRAY_SIZE(mx6q_cpu_op); > + return mx6q_cpu_op; > + } Why not just return &mx6q_cpu_op[1] instead of creating a second array? > +} > + > +static int __init mx6q_cpu_op_init(void) > +{ > + get_cpu_op = mx6q_get_cpu_op; > + return 0; > +} > + > +core_initcall(mx6q_cpu_op_init); > + > +static int __init set_arm_max_freq(char *p) > +{ > + if (strncmp(p, "1000", 4) == 0) > + arm_max_freq = 1000; > + else if (strncmp(p, "800", 3) == 0) > + arm_max_freq = 800; > + return 0; > +} > + > +early_param("arm_freq", set_arm_max_freq); Do we need this? I suppose this should be passed by the device tree if necessary. Sascha
On Mon, Dec 12, 2011 at 01:44:36PM +0100, Sascha Hauer wrote: > On Mon, Dec 12, 2011 at 11:42:31AM +0800, Richard Zhao wrote: > > Signed-off-by: Richard Zhao <richard.zhao@linaro.org> > > --- > > arch/arm/mach-imx/Kconfig | 1 + > > arch/arm/mach-imx/Makefile | 2 +- > > arch/arm/mach-imx/cpu_op-imx6q.c | 78 ++++++++++++++++++++++++++++++++++++++ > > 3 files changed, 80 insertions(+), 1 deletions(-) > > create mode 100644 arch/arm/mach-imx/cpu_op-imx6q.c > > > > diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig > > index fbd414b..7d813a3 100644 > > --- a/arch/arm/mach-imx/Kconfig > > +++ b/arch/arm/mach-imx/Kconfig > > @@ -837,6 +837,7 @@ config SOC_IMX6Q > > select ARM_GIC > > select CACHE_L2X0 > > select CPU_V7 > > + select ARCH_HAS_CPUFREQ > > select HAVE_ARM_SCU > > select HAVE_IMX_GPC > > select HAVE_IMX_MMDC > > diff --git a/arch/arm/mach-imx/Makefile b/arch/arm/mach-imx/Makefile > > index 9cf630a..2dfe4a7 100644 > > --- a/arch/arm/mach-imx/Makefile > > +++ b/arch/arm/mach-imx/Makefile > > @@ -72,7 +72,7 @@ AFLAGS_head-v7.o :=-Wa,-march=armv7-a > > obj-$(CONFIG_SMP) += platsmp.o > > obj-$(CONFIG_HOTPLUG_CPU) += hotplug.o > > obj-$(CONFIG_LOCAL_TIMERS) += localtimer.o > > -obj-$(CONFIG_SOC_IMX6Q) += clock-imx6q.o mach-imx6q.o pm-imx6q.o > > +obj-$(CONFIG_SOC_IMX6Q) += clock-imx6q.o mach-imx6q.o pm-imx6q.o cpu_op-imx6q.o > > > > # i.MX5 based machines > > obj-$(CONFIG_MACH_MX51_BABBAGE) += mach-mx51_babbage.o > > diff --git a/arch/arm/mach-imx/cpu_op-imx6q.c b/arch/arm/mach-imx/cpu_op-imx6q.c > > new file mode 100644 > > index 0000000..8807b96 > > --- /dev/null > > +++ b/arch/arm/mach-imx/cpu_op-imx6q.c > > @@ -0,0 +1,78 @@ > > +/* > > + * Copyright (C) 2010-2011 Freescale Semiconductor, Inc. All Rights Reserved. > > + */ > > + > > +/* > > + * The code contained herein is licensed under the GNU General Public > > + * License. You may obtain a copy of the GNU General Public License > > + * Version 2 or later at the following locations: > > + * > > + * http://www.opensource.org/licenses/gpl-license.html > > + * http://www.gnu.org/copyleft/gpl.html > > + */ > > + > > +#include <linux/types.h> > > +#include <linux/kernel.h> > > +#include <linux/string.h> > > +#include <mach/hardware.h> > > + > > +static u32 arm_max_freq; > > +static int num_cpu_op; > > Unused variable. right, thanks. > > > + > > +/* working point(wp): 0 - 1GHzMHz; 1 - 800MHz, 3 - 400MHz, 4 - 160MHz */ > > s/GHzMHz/GHz/ ok > > > +static struct cpu_op mx6q_cpu_op_1G[] = { > > + { > > + .cpu_rate = 996000000, > > + .cpu_voltage = 1225000,}, > > + { > > Strange indention. { .cpu_rate = 996000000, .cpu_voltage = 1225000, }, I'll change to like above. > > > > > + .cpu_rate = 792000000, > > + .cpu_voltage = 1100000,}, > > + { > > + .cpu_rate = 396000000, > > + .cpu_voltage = 950000,}, > > + { > > + .cpu_rate = 198000000, > > + .cpu_voltage = 850000,}, > > +}; > > + > > +static struct cpu_op mx6q_cpu_op[] = { > > + { > > + .cpu_rate = 792000000, > > + .cpu_voltage = 1100000,}, > > + { > > + .cpu_rate = 396000000, > > + .cpu_voltage = 950000,}, > > + { > > + .cpu_rate = 198000000, > > + .cpu_voltage = 850000,}, > > +}; > > + > > +struct cpu_op *mx6q_get_cpu_op(int *op) > > +{ > > + if (arm_max_freq == 1000) { > > + *op = num_cpu_op = ARRAY_SIZE(mx6q_cpu_op_1G); > > + return mx6q_cpu_op_1G; > > + } else { > > + *op = num_cpu_op = ARRAY_SIZE(mx6q_cpu_op); > > + return mx6q_cpu_op; > > + } > > Why not just return &mx6q_cpu_op[1] instead of creating a second array? Good idea. > > > +} > > + > > +static int __init mx6q_cpu_op_init(void) > > +{ > > + get_cpu_op = mx6q_get_cpu_op; > > + return 0; > > +} > > + > > +core_initcall(mx6q_cpu_op_init); > > + > > +static int __init set_arm_max_freq(char *p) > > +{ > > + if (strncmp(p, "1000", 4) == 0) > > + arm_max_freq = 1000; > > + else if (strncmp(p, "800", 3) == 0) > > + arm_max_freq = 800; > > + return 0; > > +} > > + > > +early_param("arm_freq", set_arm_max_freq); > > Do we need this? I suppose this should be passed by the device tree if > necessary. Yes, we do. Some boards don't support 1G. Would you let it be a temp solution? Or I'll only support 1G boards. For long time, maybe we put all operating points to dts or opp. Which one will you prefer? It needs much other work to do. Thanks Richard > > Sascha > > -- > Pengutronix e.K. | | > Industrial Linux Solutions | http://www.pengutronix.de/ | > Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel >
On Mon, Dec 12, 2011 at 11:42:31AM +0800, Richard Zhao wrote: > Signed-off-by: Richard Zhao <richard.zhao@linaro.org> > --- > arch/arm/mach-imx/Kconfig | 1 + > arch/arm/mach-imx/Makefile | 2 +- > arch/arm/mach-imx/cpu_op-imx6q.c | 78 ++++++++++++++++++++++++++++++++++++++ > 3 files changed, 80 insertions(+), 1 deletions(-) > create mode 100644 arch/arm/mach-imx/cpu_op-imx6q.c > > diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig > index fbd414b..7d813a3 100644 > --- a/arch/arm/mach-imx/Kconfig > +++ b/arch/arm/mach-imx/Kconfig > @@ -837,6 +837,7 @@ config SOC_IMX6Q > select ARM_GIC > select CACHE_L2X0 > select CPU_V7 > + select ARCH_HAS_CPUFREQ ABCDEFGH...etc...
On Mon, Dec 12, 2011 at 02:10:31PM +0000, Russell King - ARM Linux wrote: > On Mon, Dec 12, 2011 at 11:42:31AM +0800, Richard Zhao wrote: > > Signed-off-by: Richard Zhao <richard.zhao@linaro.org> > > --- > > arch/arm/mach-imx/Kconfig | 1 + > > arch/arm/mach-imx/Makefile | 2 +- > > arch/arm/mach-imx/cpu_op-imx6q.c | 78 ++++++++++++++++++++++++++++++++++++++ > > 3 files changed, 80 insertions(+), 1 deletions(-) > > create mode 100644 arch/arm/mach-imx/cpu_op-imx6q.c > > > > diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig > > index fbd414b..7d813a3 100644 > > --- a/arch/arm/mach-imx/Kconfig > > +++ b/arch/arm/mach-imx/Kconfig > > @@ -837,6 +837,7 @@ config SOC_IMX6Q > > select ARM_GIC > > select CACHE_L2X0 > > select CPU_V7 > > + select ARCH_HAS_CPUFREQ > > ABCDEFGH...etc... Right. Thanks for your comments. Richard > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel >
On Mon, Dec 12, 2011 at 09:46:43PM +0800, Richard Zhao wrote: > > > +{ > > > + if (strncmp(p, "1000", 4) == 0) > > > + arm_max_freq = 1000; > > > + else if (strncmp(p, "800", 3) == 0) > > > + arm_max_freq = 800; > > > + return 0; > > > +} > > > + > > > +early_param("arm_freq", set_arm_max_freq); > > > > Do we need this? I suppose this should be passed by the device tree if > > necessary. > Yes, we do. Some boards don't support 1G. Would you let it be a temp solution? Or I'll > only support 1G boards. > For long time, maybe we put all operating points to dts or opp. Which one will > you prefer? It needs much other work to do. I see several boards already having a clock-frequency property in their cpu node, so you could use this as maximum frequency. Sascha
diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig index fbd414b..7d813a3 100644 --- a/arch/arm/mach-imx/Kconfig +++ b/arch/arm/mach-imx/Kconfig @@ -837,6 +837,7 @@ config SOC_IMX6Q select ARM_GIC select CACHE_L2X0 select CPU_V7 + select ARCH_HAS_CPUFREQ select HAVE_ARM_SCU select HAVE_IMX_GPC select HAVE_IMX_MMDC diff --git a/arch/arm/mach-imx/Makefile b/arch/arm/mach-imx/Makefile index 9cf630a..2dfe4a7 100644 --- a/arch/arm/mach-imx/Makefile +++ b/arch/arm/mach-imx/Makefile @@ -72,7 +72,7 @@ AFLAGS_head-v7.o :=-Wa,-march=armv7-a obj-$(CONFIG_SMP) += platsmp.o obj-$(CONFIG_HOTPLUG_CPU) += hotplug.o obj-$(CONFIG_LOCAL_TIMERS) += localtimer.o -obj-$(CONFIG_SOC_IMX6Q) += clock-imx6q.o mach-imx6q.o pm-imx6q.o +obj-$(CONFIG_SOC_IMX6Q) += clock-imx6q.o mach-imx6q.o pm-imx6q.o cpu_op-imx6q.o # i.MX5 based machines obj-$(CONFIG_MACH_MX51_BABBAGE) += mach-mx51_babbage.o diff --git a/arch/arm/mach-imx/cpu_op-imx6q.c b/arch/arm/mach-imx/cpu_op-imx6q.c new file mode 100644 index 0000000..8807b96 --- /dev/null +++ b/arch/arm/mach-imx/cpu_op-imx6q.c @@ -0,0 +1,78 @@ +/* + * Copyright (C) 2010-2011 Freescale Semiconductor, Inc. All Rights Reserved. + */ + +/* + * The code contained herein is licensed under the GNU General Public + * License. You may obtain a copy of the GNU General Public License + * Version 2 or later at the following locations: + * + * http://www.opensource.org/licenses/gpl-license.html + * http://www.gnu.org/copyleft/gpl.html + */ + +#include <linux/types.h> +#include <linux/kernel.h> +#include <linux/string.h> +#include <mach/hardware.h> + +static u32 arm_max_freq; +static int num_cpu_op; + +/* working point(wp): 0 - 1GHzMHz; 1 - 800MHz, 3 - 400MHz, 4 - 160MHz */ +static struct cpu_op mx6q_cpu_op_1G[] = { + { + .cpu_rate = 996000000, + .cpu_voltage = 1225000,}, + { + .cpu_rate = 792000000, + .cpu_voltage = 1100000,}, + { + .cpu_rate = 396000000, + .cpu_voltage = 950000,}, + { + .cpu_rate = 198000000, + .cpu_voltage = 850000,}, +}; + +static struct cpu_op mx6q_cpu_op[] = { + { + .cpu_rate = 792000000, + .cpu_voltage = 1100000,}, + { + .cpu_rate = 396000000, + .cpu_voltage = 950000,}, + { + .cpu_rate = 198000000, + .cpu_voltage = 850000,}, +}; + +struct cpu_op *mx6q_get_cpu_op(int *op) +{ + if (arm_max_freq == 1000) { + *op = num_cpu_op = ARRAY_SIZE(mx6q_cpu_op_1G); + return mx6q_cpu_op_1G; + } else { + *op = num_cpu_op = ARRAY_SIZE(mx6q_cpu_op); + return mx6q_cpu_op; + } +} + +static int __init mx6q_cpu_op_init(void) +{ + get_cpu_op = mx6q_get_cpu_op; + return 0; +} + +core_initcall(mx6q_cpu_op_init); + +static int __init set_arm_max_freq(char *p) +{ + if (strncmp(p, "1000", 4) == 0) + arm_max_freq = 1000; + else if (strncmp(p, "800", 3) == 0) + arm_max_freq = 800; + return 0; +} + +early_param("arm_freq", set_arm_max_freq);
Signed-off-by: Richard Zhao <richard.zhao@linaro.org> --- arch/arm/mach-imx/Kconfig | 1 + arch/arm/mach-imx/Makefile | 2 +- arch/arm/mach-imx/cpu_op-imx6q.c | 78 ++++++++++++++++++++++++++++++++++++++ 3 files changed, 80 insertions(+), 1 deletions(-) create mode 100644 arch/arm/mach-imx/cpu_op-imx6q.c