From: daniel.lezcano@linaro.org (Daniel Lezcano)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH ] ARM: pxa: fix section mismatch warning for pxa_timer_nodt_init
Date: Mon, 28 Jul 2014 07:39:51 +0200 [thread overview]
Message-ID: <53D5E227.6090900@linaro.org> (raw)
In-Reply-To: <12220413.BdRleC0Xf5@wuerfel>
On 07/26/2014 08:50 PM, Arnd Bergmann wrote:
> commit a38b1f60b5245a3 ("ARM: pxa: Add non device-tree timer link to
> clocksource") introduced a harmless section mismatch warning for
> all pxa platforms, by introducing a new pxa_timer_init() function
> that is not marked __init but that calls pxa_timer_nodt_init(),
> which is. The function is only called at init time, so it is safe
> to also annotate it this way.
>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> ---
> Daniel, please apply on top of Robert's patch series.
Applied thanks!
-- Daniel
>
> diff --git a/arch/arm/mach-pxa/generic.c b/arch/arm/mach-pxa/generic.c
> index 630fa916bbc6..04b013fbc98f 100644
> --- a/arch/arm/mach-pxa/generic.c
> +++ b/arch/arm/mach-pxa/generic.c
> @@ -61,7 +61,7 @@ EXPORT_SYMBOL(get_clock_tick_rate);
> /*
> * For non device-tree builds, keep legacy timer init
> */
> -void pxa_timer_init(void)
> +void __init pxa_timer_init(void)
> {
> pxa_timer_nodt_init(IRQ_OST0, io_p2v(0x40a00000),
> get_clock_tick_rate());
>
--
<http://www.linaro.org/> Linaro.org ? Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog
WARNING: multiple messages have this Message-ID (diff)
From: Daniel Lezcano <daniel.lezcano-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
To: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>,
Robert Jarzmik <robert.jarzmik-GANU6spQydw@public.gmane.org>
Cc: Haojian Zhuang
<haojian.zhuang-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Thomas Gleixner <tglx-hfZtesqFncYOwBW4kG4KsQ@public.gmane.org>
Subject: Re: [PATCH ] ARM: pxa: fix section mismatch warning for pxa_timer_nodt_init
Date: Mon, 28 Jul 2014 07:39:51 +0200 [thread overview]
Message-ID: <53D5E227.6090900@linaro.org> (raw)
In-Reply-To: <12220413.BdRleC0Xf5@wuerfel>
On 07/26/2014 08:50 PM, Arnd Bergmann wrote:
> commit a38b1f60b5245a3 ("ARM: pxa: Add non device-tree timer link to
> clocksource") introduced a harmless section mismatch warning for
> all pxa platforms, by introducing a new pxa_timer_init() function
> that is not marked __init but that calls pxa_timer_nodt_init(),
> which is. The function is only called at init time, so it is safe
> to also annotate it this way.
>
> Signed-off-by: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>
> ---
> Daniel, please apply on top of Robert's patch series.
Applied thanks!
-- Daniel
>
> diff --git a/arch/arm/mach-pxa/generic.c b/arch/arm/mach-pxa/generic.c
> index 630fa916bbc6..04b013fbc98f 100644
> --- a/arch/arm/mach-pxa/generic.c
> +++ b/arch/arm/mach-pxa/generic.c
> @@ -61,7 +61,7 @@ EXPORT_SYMBOL(get_clock_tick_rate);
> /*
> * For non device-tree builds, keep legacy timer init
> */
> -void pxa_timer_init(void)
> +void __init pxa_timer_init(void)
> {
> pxa_timer_nodt_init(IRQ_OST0, io_p2v(0x40a00000),
> get_clock_tick_rate());
>
--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
WARNING: multiple messages have this Message-ID (diff)
From: Daniel Lezcano <daniel.lezcano@linaro.org>
To: Arnd Bergmann <arnd@arndb.de>, Robert Jarzmik <robert.jarzmik@free.fr>
Cc: Haojian Zhuang <haojian.zhuang@gmail.com>,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
Thomas Gleixner <tglx@linutronix.de>
Subject: Re: [PATCH ] ARM: pxa: fix section mismatch warning for pxa_timer_nodt_init
Date: Mon, 28 Jul 2014 07:39:51 +0200 [thread overview]
Message-ID: <53D5E227.6090900@linaro.org> (raw)
In-Reply-To: <12220413.BdRleC0Xf5@wuerfel>
On 07/26/2014 08:50 PM, Arnd Bergmann wrote:
> commit a38b1f60b5245a3 ("ARM: pxa: Add non device-tree timer link to
> clocksource") introduced a harmless section mismatch warning for
> all pxa platforms, by introducing a new pxa_timer_init() function
> that is not marked __init but that calls pxa_timer_nodt_init(),
> which is. The function is only called at init time, so it is safe
> to also annotate it this way.
>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> ---
> Daniel, please apply on top of Robert's patch series.
Applied thanks!
-- Daniel
>
> diff --git a/arch/arm/mach-pxa/generic.c b/arch/arm/mach-pxa/generic.c
> index 630fa916bbc6..04b013fbc98f 100644
> --- a/arch/arm/mach-pxa/generic.c
> +++ b/arch/arm/mach-pxa/generic.c
> @@ -61,7 +61,7 @@ EXPORT_SYMBOL(get_clock_tick_rate);
> /*
> * For non device-tree builds, keep legacy timer init
> */
> -void pxa_timer_init(void)
> +void __init pxa_timer_init(void)
> {
> pxa_timer_nodt_init(IRQ_OST0, io_p2v(0x40a00000),
> get_clock_tick_rate());
>
--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog
next prev parent reply other threads:[~2014-07-28 5:39 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-07-14 16:52 [PATCH v3 1/4] clocksource: move PXA timer to clocksource framework Robert Jarzmik
2014-07-14 16:52 ` Robert Jarzmik
2014-07-14 16:52 ` Robert Jarzmik
2014-07-14 16:52 ` [PATCH v3 2/4] clocksource: add device-tree support for PXA timer Robert Jarzmik
2014-07-14 16:52 ` Robert Jarzmik
2014-07-14 16:52 ` Robert Jarzmik
2014-07-14 16:52 ` [PATCH v3 3/4] arm: pxa: add CLKSRC_OF dependency Robert Jarzmik
2014-07-14 16:52 ` Robert Jarzmik
2014-07-14 16:52 ` [PATCH v3 4/4] arm: pxa: add non device-tree timer link to clocksource Robert Jarzmik
2014-07-14 16:52 ` Robert Jarzmik
2014-07-14 16:52 ` Robert Jarzmik
2014-07-21 14:37 ` Daniel Lezcano
2014-07-21 14:37 ` Daniel Lezcano
2014-07-21 14:37 ` Daniel Lezcano
2014-07-21 18:15 ` Robert Jarzmik
2014-07-21 18:15 ` Robert Jarzmik
2014-07-21 18:15 ` Robert Jarzmik
2014-07-26 18:50 ` [PATCH ] ARM: pxa: fix section mismatch warning for pxa_timer_nodt_init Arnd Bergmann
2014-07-26 18:50 ` Arnd Bergmann
2014-07-26 21:17 ` Robert Jarzmik
2014-07-26 21:17 ` Robert Jarzmik
2014-07-26 21:17 ` Robert Jarzmik
2014-07-28 5:39 ` Daniel Lezcano [this message]
2014-07-28 5:39 ` Daniel Lezcano
2014-07-28 5:39 ` Daniel Lezcano
2014-09-07 21:06 ` Arnd Bergmann
2014-09-07 21:06 ` Arnd Bergmann
2014-09-07 21:06 ` Arnd Bergmann
2014-09-08 2:21 ` Re : " robert.jarzmik at free.fr
2014-09-08 2:21 ` robert.jarzmik
2014-09-26 7:52 ` [PATCH v3 4/4] arm: pxa: add non device-tree timer link to clocksource Daniel Lezcano
2014-09-26 7:52 ` Daniel Lezcano
2014-09-26 17:08 ` Robert Jarzmik
2014-09-26 17:08 ` Robert Jarzmik
2014-09-26 17:08 ` Robert Jarzmik
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=53D5E227.6090900@linaro.org \
--to=daniel.lezcano@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.