diff mbox series

[2/5] ARM: rockchip: pm: Mark init functions __init

Message ID 20190411232157.55125-2-dianders@chromium.org (mailing list archive)
State New, archived
Headers show
Series [1/5] clk: rockchip: Turn on "aclk_dmac1" for suspend | expand

Commit Message

Douglas Anderson April 11, 2019, 11:21 p.m. UTC
The functions rk3288_config_bootdata() and rk3288_suspend_init() are
only called in the context of rockchip_suspend_init() which is already
marked __init.  We can mark them __init too.

Signed-off-by: Douglas Anderson <dianders@chromium.org>
---

 arch/arm/mach-rockchip/pm.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Heiko Stuebner April 12, 2019, 11:16 a.m. UTC | #1
Am Freitag, 12. April 2019, 01:21:54 CEST schrieb Douglas Anderson:
> The functions rk3288_config_bootdata() and rk3288_suspend_init() are
> only called in the context of rockchip_suspend_init() which is already
> marked __init.  We can mark them __init too.
> 
> Signed-off-by: Douglas Anderson <dianders@chromium.org>

applied for 5.2

Thanks
Heiko
diff mbox series

Patch

diff --git a/arch/arm/mach-rockchip/pm.c b/arch/arm/mach-rockchip/pm.c
index 0592534e0b88..065b09e6f1eb 100644
--- a/arch/arm/mach-rockchip/pm.c
+++ b/arch/arm/mach-rockchip/pm.c
@@ -59,7 +59,7 @@  static inline u32 rk3288_l2_config(void)
 	return l2ctlr;
 }
 
-static void rk3288_config_bootdata(void)
+static void __init rk3288_config_bootdata(void)
 {
 	rkpm_bootdata_cpusp = rk3288_bootram_phy + (SZ_4K - 8);
 	rkpm_bootdata_cpu_code = __pa_symbol(cpu_resume);
@@ -230,7 +230,7 @@  static void rk3288_suspend_finish(void)
 		pr_err("%s: Suspend finish failed\n", __func__);
 }
 
-static int rk3288_suspend_init(struct device_node *np)
+static int __init rk3288_suspend_init(struct device_node *np)
 {
 	struct device_node *sram_np;
 	struct resource res;