lkml.org 
[lkml]   [2013]   [Sep]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] drivers: misc: bmp085: remove '__init' from mp085_get_of_properties()

It seems the "./scripts/get_maintainer.pl" is not quite precise, it
lead me to send this mail to kernel mailing list, not to drivers
related mailing list.

[root@gchenlinux linux-next]# ./scripts/get_maintainer.pl /tmp/0001-drivers-misc-bmp085-remove-__init-from-mp085_get_of_.patch
Arnd Bergmann <arnd@arndb.de> (supporter:CHAR and MISC DRI...)
Greg Kroah-Hartman <gregkh@linuxfoundation.org> (supporter:CHAR and MISC DRI...)
linux-kernel@vger.kernel.org (open list)

Thanks.

On 09/02/2013 03:54 PM, Chen Gang wrote:
> bmp085_get_of_properties() is called by bmp085_init_client() which is
> called by bmp085_probe() which is an EXPORT_SYMBOL_GPL function.
>
> bmp085_probe() is really used as a probe function by another modules
> (e.g. bmp085-i2c.c, bmp085-spi.c).
>
> Except bmp085_get_of_properties(), all functions have no '__init', so
> need remove '__init' from bmp085_get_of_properties() too, or at least,
> it will report related warning:
>
> WARNING: vmlinux.o(.text+0x4c8a07): Section mismatch in reference from the variable .LM171 to the variable .init.text:_bmp085_get_of_properties
> The function .LM171() references
> the variable __init _bmp085_get_of_properties.
> This is often because .LM171 lacks a __init
> annotation or the annotation of _bmp085_get_of_properties is wrong.
>
>
> Signed-off-by: Chen Gang <gang.chen@asianux.com>
> ---
> drivers/misc/bmp085.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/misc/bmp085.c b/drivers/misc/bmp085.c
> index 849e2fe..2704d88 100644
> --- a/drivers/misc/bmp085.c
> +++ b/drivers/misc/bmp085.c
> @@ -374,7 +374,7 @@ int bmp085_detect(struct device *dev)
> }
> EXPORT_SYMBOL_GPL(bmp085_detect);
>
> -static void __init bmp085_get_of_properties(struct bmp085_data *data)
> +static void bmp085_get_of_properties(struct bmp085_data *data)
> {
> #ifdef CONFIG_OF
> struct device_node *np = data->dev->of_node;
>


--
Chen Gang


\
 
 \ /
  Last update: 2013-09-02 10:21    [W:0.029 / U:0.112 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site