diff mbox

ARM: exynos_defconfig: Enable bus frequency scaling with DEVFREQ/DEVFREQ-Event

Message ID 1468546426-4482-1-git-send-email-cw00.choi@samsung.com (mailing list archive)
State New, archived
Headers show

Commit Message

Chanwoo Choi July 15, 2016, 1:33 a.m. UTC
This patch enables the bus frequency scaling driver with DEVFREQ and
DEVFREQ-Event framework. This patches already was merged for Odroid-U3/XU3
,Trats2 and Rinato/Monk boards.

Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com>
---
 arch/arm/configs/exynos_defconfig | 10 ++++++++++
 1 file changed, 10 insertions(+)

Comments

Javier Martinez Canillas July 15, 2016, 11:43 a.m. UTC | #1
Hello Chanwoo,

On 07/14/2016 09:33 PM, Chanwoo Choi wrote:
> This patch enables the bus frequency scaling driver with DEVFREQ and
> DEVFREQ-Event framework. This patches already was merged for Odroid-U3/XU3
> ,Trats2 and Rinato/Monk boards.
> 
> Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com>
> ---

The patch makes sense to me. Should we do the same for multi_v7_defconfig?

Reviewed-by: Javier Martinez Canillas <javier@osg.samsung.com>

Best regards,
Chanwoo Choi July 21, 2016, 8:34 a.m. UTC | #2
Hi Krzysztof,

Could you please pick this patch?

Regards,
Chanwoo Choi

On 2016년 07월 15일 10:33, Chanwoo Choi wrote:
> This patch enables the bus frequency scaling driver with DEVFREQ and
> DEVFREQ-Event framework. This patches already was merged for Odroid-U3/XU3
> ,Trats2 and Rinato/Monk boards.
> 
> Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com>
> ---
>  arch/arm/configs/exynos_defconfig | 10 ++++++++++
>  1 file changed, 10 insertions(+)
> 
> diff --git a/arch/arm/configs/exynos_defconfig b/arch/arm/configs/exynos_defconfig
> index a5aa44c56bfe..37052b115a40 100644
> --- a/arch/arm/configs/exynos_defconfig
> +++ b/arch/arm/configs/exynos_defconfig
> @@ -219,6 +219,16 @@ CONFIG_CROS_EC_CHARDEV=y
>  CONFIG_COMMON_CLK_MAX77686=y
>  CONFIG_COMMON_CLK_MAX77802=y
>  CONFIG_COMMON_CLK_S2MPS11=y
> +CONFIG_PM_DEVFREQ=y
> +CONFIG_DEVFREQ_GOV_SIMPLE_ONDEMAND=y
> +CONFIG_DEVFREQ_GOV_PERFORMANCE=y
> +CONFIG_DEVFREQ_GOV_POWERSAVE=y
> +CONFIG_DEVFREQ_GOV_USERSPACE=y
> +CONFIG_DEVFREQ_GOV_PASSIVE=y
> +CONFIG_ARM_EXYNOS_BUS_DEVFREQ=y
> +CONFIG_PM_DEVFREQ_EVENT=y
> +CONFIG_DEVFREQ_EVENT_EXYNOS_NOCP=y
> +CONFIG_DEVFREQ_EVENT_EXYNOS_PPMU=y
>  CONFIG_EXTCON=y
>  CONFIG_EXTCON_MAX14577=y
>  CONFIG_EXTCON_MAX77693=y
>
Krzysztof Kozlowski July 21, 2016, 8:52 a.m. UTC | #3
On 07/21/2016 10:34 AM, Chanwoo Choi wrote:
> Hi Krzysztof,
> 
> Could you please pick this patch?
> 
> Regards,
> Chanwoo Choi

Hi,

The patchset looks good, only the commit message needs some minor
improvements (I can do that while applying). However it came too late
for this cycle - after my last pull request for v4.8 ten days ago. I
won't be able to push anything more so it has to wait for v4.9. I'll
pick it up after merge window.

For the record:
Reviewed-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>

Best regards,
Krzysztof
Krzysztof Kozlowski Aug. 10, 2016, 9:46 a.m. UTC | #4
On 07/15/2016 03:33 AM, Chanwoo Choi wrote:
> This patch enables the bus frequency scaling driver with DEVFREQ and
> DEVFREQ-Event framework. This patches already was merged for Odroid-U3/XU3
> ,Trats2 and Rinato/Monk boards.
> 
> Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com>
> ---
>  arch/arm/configs/exynos_defconfig | 10 ++++++++++
>  1 file changed, 10 insertions(+)
> 
> diff --git a/arch/arm/configs/exynos_defconfig b/arch/arm/configs/exynos_defconfig
> index a5aa44c56bfe..37052b115a40 100644
> --- a/arch/arm/configs/exynos_defconfig
> +++ b/arch/arm/configs/exynos_defconfig
> @@ -219,6 +219,16 @@ CONFIG_CROS_EC_CHARDEV=y
>  CONFIG_COMMON_CLK_MAX77686=y
>  CONFIG_COMMON_CLK_MAX77802=y
>  CONFIG_COMMON_CLK_S2MPS11=y
> +CONFIG_PM_DEVFREQ=y
> +CONFIG_DEVFREQ_GOV_SIMPLE_ONDEMAND=y
> +CONFIG_DEVFREQ_GOV_PERFORMANCE=y
> +CONFIG_DEVFREQ_GOV_POWERSAVE=y
> +CONFIG_DEVFREQ_GOV_USERSPACE=y
> +CONFIG_DEVFREQ_GOV_PASSIVE=y
> +CONFIG_ARM_EXYNOS_BUS_DEVFREQ=y
> +CONFIG_PM_DEVFREQ_EVENT=y
> +CONFIG_DEVFREQ_EVENT_EXYNOS_NOCP=y
> +CONFIG_DEVFREQ_EVENT_EXYNOS_PPMU=y

Applied with small changes in commit message and removal of symbols
already selected (defconfig).

Best regards,
Krzysztof
diff mbox

Patch

diff --git a/arch/arm/configs/exynos_defconfig b/arch/arm/configs/exynos_defconfig
index a5aa44c56bfe..37052b115a40 100644
--- a/arch/arm/configs/exynos_defconfig
+++ b/arch/arm/configs/exynos_defconfig
@@ -219,6 +219,16 @@  CONFIG_CROS_EC_CHARDEV=y
 CONFIG_COMMON_CLK_MAX77686=y
 CONFIG_COMMON_CLK_MAX77802=y
 CONFIG_COMMON_CLK_S2MPS11=y
+CONFIG_PM_DEVFREQ=y
+CONFIG_DEVFREQ_GOV_SIMPLE_ONDEMAND=y
+CONFIG_DEVFREQ_GOV_PERFORMANCE=y
+CONFIG_DEVFREQ_GOV_POWERSAVE=y
+CONFIG_DEVFREQ_GOV_USERSPACE=y
+CONFIG_DEVFREQ_GOV_PASSIVE=y
+CONFIG_ARM_EXYNOS_BUS_DEVFREQ=y
+CONFIG_PM_DEVFREQ_EVENT=y
+CONFIG_DEVFREQ_EVENT_EXYNOS_NOCP=y
+CONFIG_DEVFREQ_EVENT_EXYNOS_PPMU=y
 CONFIG_EXTCON=y
 CONFIG_EXTCON_MAX14577=y
 CONFIG_EXTCON_MAX77693=y