Re: [PATCH] clk: samsung: mark s3c...._clk_sleep_init() as __init

From: Chanwoo Choi
Date: Tue Jan 24 2017 - 19:36:43 EST


Hi Martin,

On 2017ë 01ì 25ì 06:33, Martin Kaiser wrote:
> These functions are called only by other __init functions.
> Compiling for s3c2410 with CONFIG_DEBUG_SECTION_MISMATCH=y,
> I get warnings like
>
> WARNING: vmlinux.o(.text+0x198350):
> Section mismatch in reference from the function s3c2410_clk_sleep_init()
> to the (unknown reference) .init.data:(unknown)
>
> Marking the s3c...._clk_sleep() functions as __init fixes this.
>
> Signed-off-by: Martin Kaiser <martin@xxxxxxxxx>
> ---
> drivers/clk/samsung/clk-s3c2410.c | 4 ++--
> drivers/clk/samsung/clk-s3c2412.c | 4 ++--
> drivers/clk/samsung/clk-s3c2443.c | 4 ++--
> drivers/clk/samsung/clk-s3c64xx.c | 4 ++--
> 4 files changed, 8 insertions(+), 8 deletions(-)


Looks good to me.
Reviewed-by: Chanwoo Choi <cw00.choi@xxxxxxxxxxx>

[snip]

--
Best Regards,
Chanwoo Choi
Samsung Electronics