* [PATCH 1/2] ARM: exynos_defconfig: Don't attempt to enable schedutil governor as module
@ 2016-08-19 13:12 Javier Martinez Canillas
2016-08-19 13:12 ` [PATCH 2/2] ARM: multi_v7_defconfig: " Javier Martinez Canillas
2016-08-19 13:41 ` [PATCH 1/2] ARM: exynos_defconfig: " Krzysztof Kozlowski
0 siblings, 2 replies; 8+ messages in thread
From: Javier Martinez Canillas @ 2016-08-19 13:12 UTC (permalink / raw)
To: linux-arm-kernel
The schedutil CPUFreq governor could be built as a module but the commit
58919e83c85c ("cpufreq / sched: Pass flags to cpufreq_update_util()")
made it non-modular so attempting to enable as module leads to a warning:
warning: symbol value 'm' invalid for CPU_FREQ_GOV_SCHEDUTIL
Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
---
arch/arm/configs/exynos_defconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/configs/exynos_defconfig b/arch/arm/configs/exynos_defconfig
index 5476be3be686..4e484f406419 100644
--- a/arch/arm/configs/exynos_defconfig
+++ b/arch/arm/configs/exynos_defconfig
@@ -29,7 +29,7 @@ CONFIG_CPU_FREQ_DEFAULT_GOV_ONDEMAND=y
CONFIG_CPU_FREQ_GOV_POWERSAVE=m
CONFIG_CPU_FREQ_GOV_USERSPACE=m
CONFIG_CPU_FREQ_GOV_CONSERVATIVE=m
-CONFIG_CPU_FREQ_GOV_SCHEDUTIL=m
+CONFIG_CPU_FREQ_GOV_SCHEDUTIL=y
CONFIG_CPUFREQ_DT=y
CONFIG_CPU_IDLE=y
CONFIG_ARM_EXYNOS_CPUIDLE=y
--
2.5.5
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH 2/2] ARM: multi_v7_defconfig: Don't attempt to enable schedutil governor as module 2016-08-19 13:12 [PATCH 1/2] ARM: exynos_defconfig: Don't attempt to enable schedutil governor as module Javier Martinez Canillas @ 2016-08-19 13:12 ` Javier Martinez Canillas 2016-08-19 13:40 ` Krzysztof Kozlowski 2016-08-19 13:41 ` [PATCH 1/2] ARM: exynos_defconfig: " Krzysztof Kozlowski 1 sibling, 1 reply; 8+ messages in thread From: Javier Martinez Canillas @ 2016-08-19 13:12 UTC (permalink / raw) To: linux-arm-kernel The schedutil CPUFreq governor could be built as a module but the commit 58919e83c85c ("cpufreq / sched: Pass flags to cpufreq_update_util()") made it non-modular so attempting to enable as module leads to a warning: warning: symbol value 'm' invalid for CPU_FREQ_GOV_SCHEDUTIL Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> --- arch/arm/configs/multi_v7_defconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig index 0ccaa519f588..65b5b6172e83 100644 --- a/arch/arm/configs/multi_v7_defconfig +++ b/arch/arm/configs/multi_v7_defconfig @@ -136,7 +136,7 @@ CONFIG_CPU_FREQ_DEFAULT_GOV_ONDEMAND=y CONFIG_CPU_FREQ_GOV_POWERSAVE=m CONFIG_CPU_FREQ_GOV_USERSPACE=m CONFIG_CPU_FREQ_GOV_CONSERVATIVE=m -CONFIG_CPU_FREQ_GOV_SCHEDUTIL=m +CONFIG_CPU_FREQ_GOV_SCHEDUTIL=y CONFIG_ARM_IMX6Q_CPUFREQ=y CONFIG_QORIQ_CPUFREQ=y CONFIG_CPU_IDLE=y -- 2.5.5 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH 2/2] ARM: multi_v7_defconfig: Don't attempt to enable schedutil governor as module 2016-08-19 13:12 ` [PATCH 2/2] ARM: multi_v7_defconfig: " Javier Martinez Canillas @ 2016-08-19 13:40 ` Krzysztof Kozlowski 2016-08-19 13:46 ` Javier Martinez Canillas 0 siblings, 1 reply; 8+ messages in thread From: Krzysztof Kozlowski @ 2016-08-19 13:40 UTC (permalink / raw) To: linux-arm-kernel On 08/19/2016 03:12 PM, Javier Martinez Canillas wrote: > The schedutil CPUFreq governor could be built as a module but the commit > 58919e83c85c ("cpufreq / sched: Pass flags to cpufreq_update_util()") > made it non-modular so attempting to enable as module leads to a warning: > > warning: symbol value 'm' invalid for CPU_FREQ_GOV_SCHEDUTIL > > Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> > > --- > > arch/arm/configs/multi_v7_defconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Looks good. Applied... wait, no, these are not in samsung-soc list so they miss also from patchwork. For reference: Reviewed-by: Krzysztof Kozlowski <k.kozlowski@samsung.com> Please post it to linux-samsung-soc, unless it goes through arm-soc or other tree. Best regards, Krzysztof ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 2/2] ARM: multi_v7_defconfig: Don't attempt to enable schedutil governor as module 2016-08-19 13:40 ` Krzysztof Kozlowski @ 2016-08-19 13:46 ` Javier Martinez Canillas 2016-08-19 13:52 ` Javier Martinez Canillas 0 siblings, 1 reply; 8+ messages in thread From: Javier Martinez Canillas @ 2016-08-19 13:46 UTC (permalink / raw) To: linux-arm-kernel Hello Krzysztof, On 08/19/2016 09:40 AM, Krzysztof Kozlowski wrote: > On 08/19/2016 03:12 PM, Javier Martinez Canillas wrote: >> The schedutil CPUFreq governor could be built as a module but the commit >> 58919e83c85c ("cpufreq / sched: Pass flags to cpufreq_update_util()") >> made it non-modular so attempting to enable as module leads to a warning: >> >> warning: symbol value 'm' invalid for CPU_FREQ_GOV_SCHEDUTIL >> >> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> >> >> --- >> >> arch/arm/configs/multi_v7_defconfig | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) > > Looks good. > > Applied... wait, no, these are not in samsung-soc list so they miss also > from patchwork. For reference: > Reviewed-by: Krzysztof Kozlowski <k.kozlowski@samsung.com> > Sigh, get_maintainers.pl suggested me an insanely large cc list so I added it manually... but I forgot linux-samsung-soc. I'll post v2. > Please post it to linux-samsung-soc, unless it goes through arm-soc or > other tree. > I think is OK if both patches go through your tree. > Best regards, > Krzysztof > Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 2/2] ARM: multi_v7_defconfig: Don't attempt to enable schedutil governor as module 2016-08-19 13:46 ` Javier Martinez Canillas @ 2016-08-19 13:52 ` Javier Martinez Canillas 2016-08-19 15:51 ` Krzysztof Kozlowski 0 siblings, 1 reply; 8+ messages in thread From: Javier Martinez Canillas @ 2016-08-19 13:52 UTC (permalink / raw) To: linux-arm-kernel Hello Krzysztof, On 08/19/2016 09:46 AM, Javier Martinez Canillas wrote: > > On 08/19/2016 09:40 AM, Krzysztof Kozlowski wrote: >> On 08/19/2016 03:12 PM, Javier Martinez Canillas wrote: >>> The schedutil CPUFreq governor could be built as a module but the commit >>> 58919e83c85c ("cpufreq / sched: Pass flags to cpufreq_update_util()") >>> made it non-modular so attempting to enable as module leads to a warning: >>> >>> warning: symbol value 'm' invalid for CPU_FREQ_GOV_SCHEDUTIL >>> >>> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> >>> >>> --- >>> >>> arch/arm/configs/multi_v7_defconfig | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> Looks good. >> >> Applied... wait, no, these are not in samsung-soc list so they miss also >> from patchwork. For reference: >> Reviewed-by: Krzysztof Kozlowski <k.kozlowski@samsung.com> >> > > Sigh, get_maintainers.pl suggested me an insanely large cc list so I added > it manually... but I forgot linux-samsung-soc. I'll post v2. > Damn... and now I forgot to include your collected tags in v2. I should not post patches before having enough coffee, sorry about that. Please let me know if you don't mind or if you prefer a resend. Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 2/2] ARM: multi_v7_defconfig: Don't attempt to enable schedutil governor as module 2016-08-19 13:52 ` Javier Martinez Canillas @ 2016-08-19 15:51 ` Krzysztof Kozlowski 2016-08-19 15:55 ` Javier Martinez Canillas 0 siblings, 1 reply; 8+ messages in thread From: Krzysztof Kozlowski @ 2016-08-19 15:51 UTC (permalink / raw) To: linux-arm-kernel On Fri, Aug 19, 2016 at 09:52:47AM -0400, Javier Martinez Canillas wrote: > Hello Krzysztof, > > On 08/19/2016 09:46 AM, Javier Martinez Canillas wrote: > > > > On 08/19/2016 09:40 AM, Krzysztof Kozlowski wrote: > >> On 08/19/2016 03:12 PM, Javier Martinez Canillas wrote: > >>> The schedutil CPUFreq governor could be built as a module but the commit > >>> 58919e83c85c ("cpufreq / sched: Pass flags to cpufreq_update_util()") > >>> made it non-modular so attempting to enable as module leads to a warning: > >>> > >>> warning: symbol value 'm' invalid for CPU_FREQ_GOV_SCHEDUTIL > >>> > >>> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> > >>> > >>> --- > >>> > >>> arch/arm/configs/multi_v7_defconfig | 2 +- > >>> 1 file changed, 1 insertion(+), 1 deletion(-) > >> > >> Looks good. > >> > >> Applied... wait, no, these are not in samsung-soc list so they miss also > >> from patchwork. For reference: > >> Reviewed-by: Krzysztof Kozlowski <k.kozlowski@samsung.com> > >> > > > > Sigh, get_maintainers.pl suggested me an insanely large cc list so I added > > it manually... but I forgot linux-samsung-soc. I'll post v2. > > > > Damn... and now I forgot to include your collected tags in v2. I should not > post patches before having enough coffee, sorry about that. > > Please let me know if you don't mind or if you prefer a resend. No problem, no need to resend. Applied to separate branch (not tagged yet) in case someone would like to base on top of it. Then please let me know, I'll add a tag. Mentioned commit is not yet in mainline and was applied recently, so I remove the SHA from commit msg. The SHA might change and it's better not to pollute the history in that case. Best regards, Krzysztof ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 2/2] ARM: multi_v7_defconfig: Don't attempt to enable schedutil governor as module 2016-08-19 15:51 ` Krzysztof Kozlowski @ 2016-08-19 15:55 ` Javier Martinez Canillas 0 siblings, 0 replies; 8+ messages in thread From: Javier Martinez Canillas @ 2016-08-19 15:55 UTC (permalink / raw) To: linux-arm-kernel Hello Krzysztof, On 08/19/2016 11:51 AM, Krzysztof Kozlowski wrote: > On Fri, Aug 19, 2016 at 09:52:47AM -0400, Javier Martinez Canillas wrote: >> Hello Krzysztof, >> >> On 08/19/2016 09:46 AM, Javier Martinez Canillas wrote: >>> >>> On 08/19/2016 09:40 AM, Krzysztof Kozlowski wrote: >>>> On 08/19/2016 03:12 PM, Javier Martinez Canillas wrote: >>>>> The schedutil CPUFreq governor could be built as a module but the commit >>>>> 58919e83c85c ("cpufreq / sched: Pass flags to cpufreq_update_util()") >>>>> made it non-modular so attempting to enable as module leads to a warning: >>>>> >>>>> warning: symbol value 'm' invalid for CPU_FREQ_GOV_SCHEDUTIL >>>>> >>>>> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> >>>>> >>>>> --- >>>>> >>>>> arch/arm/configs/multi_v7_defconfig | 2 +- >>>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>> >>>> Looks good. >>>> >>>> Applied... wait, no, these are not in samsung-soc list so they miss also >>>> from patchwork. For reference: >>>> Reviewed-by: Krzysztof Kozlowski <k.kozlowski@samsung.com> >>>> >>> >>> Sigh, get_maintainers.pl suggested me an insanely large cc list so I added >>> it manually... but I forgot linux-samsung-soc. I'll post v2. >>> >> >> Damn... and now I forgot to include your collected tags in v2. I should not >> post patches before having enough coffee, sorry about that. >> >> Please let me know if you don't mind or if you prefer a resend. > > No problem, no need to resend. > I resent some minutes ago figure getting your email. So feel free to ignore v3. > Applied to separate branch (not tagged yet) in case someone would like > to base on top of it. Then please let me know, I'll add a tag. > > Mentioned commit is not yet in mainline and was applied recently, so I > remove the SHA from commit msg. The SHA might change and it's better not > to pollute the history in that case. > Ok, thanks for doing that. > Best regards, > Krzysztof > Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 1/2] ARM: exynos_defconfig: Don't attempt to enable schedutil governor as module 2016-08-19 13:12 [PATCH 1/2] ARM: exynos_defconfig: Don't attempt to enable schedutil governor as module Javier Martinez Canillas 2016-08-19 13:12 ` [PATCH 2/2] ARM: multi_v7_defconfig: " Javier Martinez Canillas @ 2016-08-19 13:41 ` Krzysztof Kozlowski 1 sibling, 0 replies; 8+ messages in thread From: Krzysztof Kozlowski @ 2016-08-19 13:41 UTC (permalink / raw) To: linux-arm-kernel On 08/19/2016 03:12 PM, Javier Martinez Canillas wrote: > The schedutil CPUFreq governor could be built as a module but the commit > 58919e83c85c ("cpufreq / sched: Pass flags to cpufreq_update_util()") > made it non-modular so attempting to enable as module leads to a warning: > > warning: symbol value 'm' invalid for CPU_FREQ_GOV_SCHEDUTIL > > Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com> > --- > > arch/arm/configs/exynos_defconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Same as patch 2/2, for reference: Reviewed-by: Krzysztof Kozlowski <k.kozlowski@samsung.com> BR, Krzysztof ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2016-08-19 15:55 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2016-08-19 13:12 [PATCH 1/2] ARM: exynos_defconfig: Don't attempt to enable schedutil governor as module Javier Martinez Canillas 2016-08-19 13:12 ` [PATCH 2/2] ARM: multi_v7_defconfig: " Javier Martinez Canillas 2016-08-19 13:40 ` Krzysztof Kozlowski 2016-08-19 13:46 ` Javier Martinez Canillas 2016-08-19 13:52 ` Javier Martinez Canillas 2016-08-19 15:51 ` Krzysztof Kozlowski 2016-08-19 15:55 ` Javier Martinez Canillas 2016-08-19 13:41 ` [PATCH 1/2] ARM: exynos_defconfig: " Krzysztof Kozlowski
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).