linux-kernel - Re: [PATCH v2] clk: samsung: mark s3c...._clk_sleep_init() as _

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Thu, 26 Jan 2017 08:47:05 +0900
From:   Chanwoo Choi <cw00.choi@...sung.com>
To:     martin@...ser.cx, Krzysztof Kozlowski <krzk@...nel.org>,
        Stephen Boyd <sboyd@...eaurora.org>,
        Sylwester Nawrocki <s.nawrocki@...sung.com>,
        Tomasz Figa <tomasz.figa@...il.com>,
        Kukjin Kim <kgene@...nel.org>
Cc:     linux-arm-kernel@...ts.infradead.org,
        linux-samsung-soc@...r.kernel.org, linux-clk@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2] clk: samsung: mark s3c...._clk_sleep_init() as __init

Hi Martin,

The v2 patch is missing my reviewed-by tag[1].
[1] https://patchwork.kernel.org/patch/9535993/

Again, I add my reviewed-by tag.

Reviewed-by: Chanwoo Choi <cw00.choi@...sung.com>


On 2017년 01월 26일 06:42, martin@...ser.cx wrote:
> From: Martin Kaiser <martin@...ser.cx>
> 
> These functions are referencing s3c...._clk_regs[], which are marked as
> __initdata. When compiling with CONFIG_DEBUG_SECTION_MISMATCH=y, this
> produces 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)
> 
> Mark the s3c...._clk_sleep_init() functions as __init in
> order to fix this.
> 
> Signed-off-by: Martin Kaiser <martin@...ser.cx>
> ---
> v2:
> Fix the commit message, describe what the problem really is.
> (Thanks, Krzysztof)
> 
>  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(-)
> 
> diff --git a/drivers/clk/samsung/clk-s3c2410.c b/drivers/clk/samsung/clk-s3c2410.c
> index d7a1e77..e0650c3 100644
> --- a/drivers/clk/samsung/clk-s3c2410.c
> +++ b/drivers/clk/samsung/clk-s3c2410.c
> @@ -76,7 +76,7 @@ static void s3c2410_clk_resume(void)
>  	.resume = s3c2410_clk_resume,
>  };
>  
> -static void s3c2410_clk_sleep_init(void)
> +static void __init s3c2410_clk_sleep_init(void)
>  {
>  	s3c2410_save = samsung_clk_alloc_reg_dump(s3c2410_clk_regs,
>  						ARRAY_SIZE(s3c2410_clk_regs));
> @@ -90,7 +90,7 @@ static void s3c2410_clk_sleep_init(void)
>  	return;
>  }
>  #else
> -static void s3c2410_clk_sleep_init(void) {}
> +static void __init s3c2410_clk_sleep_init(void) {}
>  #endif
>  
>  PNAME(fclk_p) = { "mpll", "div_slow" };
> diff --git a/drivers/clk/samsung/clk-s3c2412.c b/drivers/clk/samsung/clk-s3c2412.c
> index ec873ee..b8340a4 100644
> --- a/drivers/clk/samsung/clk-s3c2412.c
> +++ b/drivers/clk/samsung/clk-s3c2412.c
> @@ -69,7 +69,7 @@ static void s3c2412_clk_resume(void)
>  	.resume = s3c2412_clk_resume,
>  };
>  
> -static void s3c2412_clk_sleep_init(void)
> +static void __init s3c2412_clk_sleep_init(void)
>  {
>  	s3c2412_save = samsung_clk_alloc_reg_dump(s3c2412_clk_regs,
>  						ARRAY_SIZE(s3c2412_clk_regs));
> @@ -83,7 +83,7 @@ static void s3c2412_clk_sleep_init(void)
>  	return;
>  }
>  #else
> -static void s3c2412_clk_sleep_init(void) {}
> +static void __init s3c2412_clk_sleep_init(void) {}
>  #endif
>  
>  static struct clk_div_table divxti_d[] = {
> diff --git a/drivers/clk/samsung/clk-s3c2443.c b/drivers/clk/samsung/clk-s3c2443.c
> index 5e24a17..abb935c 100644
> --- a/drivers/clk/samsung/clk-s3c2443.c
> +++ b/drivers/clk/samsung/clk-s3c2443.c
> @@ -89,7 +89,7 @@ static void s3c2443_clk_resume(void)
>  	.resume = s3c2443_clk_resume,
>  };
>  
> -static void s3c2443_clk_sleep_init(void)
> +static void __init s3c2443_clk_sleep_init(void)
>  {
>  	s3c2443_save = samsung_clk_alloc_reg_dump(s3c2443_clk_regs,
>  						ARRAY_SIZE(s3c2443_clk_regs));
> @@ -103,7 +103,7 @@ static void s3c2443_clk_sleep_init(void)
>  	return;
>  }
>  #else
> -static void s3c2443_clk_sleep_init(void) {}
> +static void __init s3c2443_clk_sleep_init(void) {}
>  #endif
>  
>  PNAME(epllref_p) = { "mpllref", "mpllref", "xti", "ext" };
> diff --git a/drivers/clk/samsung/clk-s3c64xx.c b/drivers/clk/samsung/clk-s3c64xx.c
> index a48bd5f..7306867 100644
> --- a/drivers/clk/samsung/clk-s3c64xx.c
> +++ b/drivers/clk/samsung/clk-s3c64xx.c
> @@ -121,7 +121,7 @@ static void s3c64xx_clk_resume(void)
>  	.resume = s3c64xx_clk_resume,
>  };
>  
> -static void s3c64xx_clk_sleep_init(void)
> +static void __init s3c64xx_clk_sleep_init(void)
>  {
>  	s3c64xx_save_common = samsung_clk_alloc_reg_dump(s3c64xx_clk_regs,
>  						ARRAY_SIZE(s3c64xx_clk_regs));
> @@ -145,7 +145,7 @@ static void s3c64xx_clk_sleep_init(void)
>  		__func__);
>  }
>  #else
> -static void s3c64xx_clk_sleep_init(void) {}
> +static void __init s3c64xx_clk_sleep_init(void) {}
>  #endif
>  
>  /* List of parent clocks common for all S3C64xx SoCs. */
> 


-- 
Best Regards,
Chanwoo Choi
Samsung Electronics