Re: [PATCH] mfd: Mark clocks_init as non-init in twl4030-core.c

[Rakib Mullick]

On 4/19/09, Rakib Mullick <rakib.mullick@xxxxxxxxx> wrote:
>  Impact: Fix section mismatch.
>
> clocks_init() has been called from twl4030_probe() which is a non-init
> function. Since probing can be done anytime so clocks_init will be
> called anytime too. So we mark clock_init() as non-init.
>
> LD      drivers/mfd/built-in.o
> WARNING: drivers/mfd/built-in.o(.text+0x8dd9): Section mismatch in
> reference from the function twl4030_probe() to the function
> .init.text:clocks_init()
> The function twl4030_probe() references
> the function __init clocks_init().
> This is often because twl4030_probe lacks a __init
> annotation or the annotation of clocks_init is wrong.
>
> ---
> Signed-off-by: Rakib Mullick <rakib.mullick@xxxxxxxxx>
>
> --- linus/drivers/mfd/twl4030-core.c	2009-03-26 06:26:44.000000000 +0600
> +++ rakib/drivers/mfd/twl4030-core.c	2009-03-26 21:51:53.604077256 +0600
> @@ -649,7 +649,7 @@ static inline int __init unprotect_pm_ma
>  	return e;
>  }
>
> -static void __init clocks_init(struct device *dev)
> +static void clocks_init(struct device *dev)
>  {
>  	int e = 0;
>  	struct clk *osc;
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/