Skip to content

Commit

Permalink
ARM: EXYNOS: Disable cpuidle for exynos5440
Browse files Browse the repository at this point in the history
There is no point to register the cpuidle driver for the 5440 as it has only
one WFI state which is the default idle function when the cpuidle driver is
disabled.

By disabling cpuidle we prevent to enter to the governor computation for
nothing, thus saving a lot of processing time.

The only drawback is the statistic via sysfs on this state which is lost but
it is meaningless and it could be retrieved from the ftrace easily.

Signed-off-by: Daniel Lezcano <[email protected]>
Reviewed-by: Tomasz Figa <[email protected]>
Acked-by: Amit Kucheria <[email protected]>
Signed-off-by: Kukjin Kim <[email protected]>
  • Loading branch information
dlezcano authored and kgene committed May 25, 2014
1 parent dcef663 commit 70ecb84
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
3 changes: 0 additions & 3 deletions arch/arm/mach-exynos/cpuidle.c
Original file line number Diff line number Diff line change
Expand Up @@ -134,9 +134,6 @@ static int exynos_cpuidle_probe(struct platform_device *pdev)
{
int ret;

if (soc_is_exynos5440())
exynos_idle_driver.state_count = 1;

ret = cpuidle_register(&exynos_idle_driver, NULL);
if (ret) {
dev_err(&pdev->dev, "failed to register cpuidle driver\n");
Expand Down
3 changes: 3 additions & 0 deletions arch/arm/mach-exynos/exynos.c
Original file line number Diff line number Diff line change
Expand Up @@ -177,6 +177,9 @@ static struct platform_device exynos_cpuidle = {

void __init exynos_cpuidle_init(void)
{
if (soc_is_exynos5440())
return;

platform_device_register(&exynos_cpuidle);
}

Expand Down

0 comments on commit 70ecb84

Please sign in to comment.