ARM: Kirkwood: fix kirkwood_pm_init() declaration/type
authorBen Dooks <ben.dooks@codethink.co.uk>
Thu, 9 Jun 2016 11:57:12 +0000 (12:57 +0100)
committerGregory CLEMENT <gregory.clement@free-electrons.com>
Thu, 16 Jun 2016 10:01:37 +0000 (12:01 +0200)
The kirkwood-pm.c was missing the include of kirkwood-pm.h to
define the kirkwood_pm_init() function. However once this is
included, the types do not match.

Fixup the include, and then the prototype to avoid the following
warning:

arch/arm/mach-mvebu/kirkwood-pm.c:69:12: warning: symbol 'kirkwood_pm_init' was not declared. Should it be static?

Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: Gregory CLEMENT <gregory.clement@free-electrons.com>
arch/arm/mach-mvebu/kirkwood-pm.c

index cbb816f2120c90794b5ee00faff238b5192363f0..1e1f879f16ce0a4fc411a2f6e2c932ed58e31337 100644 (file)
@@ -18,6 +18,7 @@
 #include <linux/suspend.h>
 #include <linux/io.h>
 #include "kirkwood.h"
+#include "kirkwood-pm.h"
 
 static void __iomem *ddr_operation_base;
 static void __iomem *memory_pm_ctrl;
@@ -66,11 +67,10 @@ static const struct platform_suspend_ops kirkwood_suspend_ops = {
        .valid = kirkwood_pm_valid_standby,
 };
 
-int __init kirkwood_pm_init(void)
+void __init kirkwood_pm_init(void)
 {
        ddr_operation_base = ioremap(DDR_OPERATION_BASE, 4);
        memory_pm_ctrl = ioremap(MEMORY_PM_CTRL_PHYS, 4);
 
        suspend_set_ops(&kirkwood_suspend_ops);
-       return 0;
 }