[PATCH] ARM: ux500: cpuidle: fix section mismatch

[PATCH] ARM: ux500: cpuidle: fix section mismatch

[Daniel Lezcano]

The dbx500_cpuidle_probe is tagged as an __init section but the variable
dbx500_cpuidle_plat_driver is not.

The dbx500_cpuidle_probe could not be declared as __init because of macro
module_platform_driver builds the exit function, tags as __exit and this one
refers to the dbx500_cpuidle_plat_driver which is an __initdata.

That leads to a section mismatch.

Fix it by removing the __init tag for the probe function.

Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
---
 drivers/cpuidle/cpuidle-ux500.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/cpuidle/cpuidle-ux500.c b/drivers/cpuidle/cpuidle-ux500.c
index e056465..5e35804 100644
--- a/drivers/cpuidle/cpuidle-ux500.c
+++ b/drivers/cpuidle/cpuidle-ux500.c
@@ -111,7 +111,7 @@ static struct cpuidle_driver ux500_idle_driver = {
 	.state_count = 2,
 };
 
-static int __init dbx500_cpuidle_probe(struct platform_device *pdev)
+static int dbx500_cpuidle_probe(struct platform_device *pdev)
 {
 	/* Configure wake up reasons */
 	prcmu_enable_wakeups(PRCMU_WAKEUP(ARM) | PRCMU_WAKEUP(RTC) |
-- 
1.7.9.5

[PATCH] ARM: ux500: cpuidle: fix section mismatch

[Linus Walleij]

On Fri, Sep 27, 2013 at 7:16 PM, Daniel Lezcano
<daniel.lezcano@linaro.org> wrote:

> The dbx500_cpuidle_probe is tagged as an __init section but the variable
> dbx500_cpuidle_plat_driver is not.
>
> The dbx500_cpuidle_probe could not be declared as __init because of macro
> module_platform_driver builds the exit function, tags as __exit and this one
> refers to the dbx500_cpuidle_plat_driver which is an __initdata.
>
> That leads to a section mismatch.
>
> Fix it by removing the __init tag for the probe function.
>
> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>

Acked-by: Linus Walleij <linus.walleij@linaro.org>

Sorry if I caused this and thanks for fixing it up!

Yours,
Linus Walleij