From: nsekhar@ti.com (Sekhar Nori)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 03/18] ARM: davinci: Switch to sched_clock_register()
Date: Thu, 1 Aug 2013 11:16:14 +0530 [thread overview]
Message-ID: <51F9F626.80306@ti.com> (raw)
In-Reply-To: <1375309877-16617-4-git-send-email-sboyd@codeaurora.org>
On Thursday 01 August 2013 04:01 AM, Stephen Boyd wrote:
> The 32 bit sched_clock interface now supports 64 bits. Upgrade to
> the 64 bit function to allow us to remove the 32 bit registration
> interface.
>
> Cc: Sekhar Nori <nsekhar@ti.com>
> Cc: Kevin Hilman <khilman@deeprootsystems.com>
> Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
> ---
> arch/arm/mach-davinci/time.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/mach-davinci/time.c b/arch/arm/mach-davinci/time.c
> index 7a55b5c..6d5440a 100644
> --- a/arch/arm/mach-davinci/time.c
> +++ b/arch/arm/mach-davinci/time.c
> @@ -285,7 +285,7 @@ static struct clocksource clocksource_davinci = {
> /*
> * Overwrite weak default sched_clock with something more precise
> */
> -static u32 notrace davinci_read_sched_clock(void)
> +static u64 notrace davinci_read_sched_clock(void)
> {
> return timer32_read(&timers[TID_CLOCKSOURCE]);
> }
> @@ -392,7 +392,7 @@ void __init davinci_timer_init(void)
> davinci_clock_tick_rate))
> printk(err, clocksource_davinci.name);
>
> - setup_sched_clock(davinci_read_sched_clock, 32,
> + sched_clock_register(davinci_read_sched_clock, 32,
> davinci_clock_tick_rate);
Please fix the line break to align with open parens. Otherwise looks
good. I tested it on DA850 EVM as well. With above fixed:
Acked-by: Sekhar Nori <nsekhar@ti.com>
Thanks,
Sekhar
WARNING: multiple messages have this Message-ID (diff)
From: Sekhar Nori <nsekhar@ti.com>
To: Stephen Boyd <sboyd@codeaurora.org>
Cc: <arm@kernel.org>, <linux-kernel@vger.kernel.org>,
<linux-arm-kernel@lists.infradead.org>,
Kevin Hilman <khilman@deeprootsystems.com>
Subject: Re: [PATCH 03/18] ARM: davinci: Switch to sched_clock_register()
Date: Thu, 1 Aug 2013 11:16:14 +0530 [thread overview]
Message-ID: <51F9F626.80306@ti.com> (raw)
In-Reply-To: <1375309877-16617-4-git-send-email-sboyd@codeaurora.org>
On Thursday 01 August 2013 04:01 AM, Stephen Boyd wrote:
> The 32 bit sched_clock interface now supports 64 bits. Upgrade to
> the 64 bit function to allow us to remove the 32 bit registration
> interface.
>
> Cc: Sekhar Nori <nsekhar@ti.com>
> Cc: Kevin Hilman <khilman@deeprootsystems.com>
> Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
> ---
> arch/arm/mach-davinci/time.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/mach-davinci/time.c b/arch/arm/mach-davinci/time.c
> index 7a55b5c..6d5440a 100644
> --- a/arch/arm/mach-davinci/time.c
> +++ b/arch/arm/mach-davinci/time.c
> @@ -285,7 +285,7 @@ static struct clocksource clocksource_davinci = {
> /*
> * Overwrite weak default sched_clock with something more precise
> */
> -static u32 notrace davinci_read_sched_clock(void)
> +static u64 notrace davinci_read_sched_clock(void)
> {
> return timer32_read(&timers[TID_CLOCKSOURCE]);
> }
> @@ -392,7 +392,7 @@ void __init davinci_timer_init(void)
> davinci_clock_tick_rate))
> printk(err, clocksource_davinci.name);
>
> - setup_sched_clock(davinci_read_sched_clock, 32,
> + sched_clock_register(davinci_read_sched_clock, 32,
> davinci_clock_tick_rate);
Please fix the line break to align with open parens. Otherwise looks
good. I tested it on DA850 EVM as well. With above fixed:
Acked-by: Sekhar Nori <nsekhar@ti.com>
Thanks,
Sekhar
next prev parent reply other threads:[~2013-08-01 5:46 UTC|newest]
Thread overview: 58+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-07-30 18:50 [GIT PULL] 64bit friendly generic sched_clock() base for 3.12 John Stultz
2013-07-30 18:50 ` John Stultz
2013-07-31 22:30 ` [PATCH 00/18] Convert arm sched_clock users to sched_clock_register() Stephen Boyd
2013-07-31 22:30 ` Stephen Boyd
2013-07-31 22:31 ` [PATCH 01/18] ARM: timer-sp: Switch " Stephen Boyd
2013-07-31 22:31 ` Stephen Boyd
2013-07-31 22:31 ` [PATCH 02/18] ARM: clps711x: " Stephen Boyd
2013-07-31 22:31 ` Stephen Boyd
2013-07-31 22:31 ` [PATCH 03/18] ARM: davinci: " Stephen Boyd
2013-07-31 22:31 ` Stephen Boyd
2013-08-01 5:46 ` Sekhar Nori [this message]
2013-08-01 5:46 ` Sekhar Nori
2013-07-31 22:31 ` [PATCH 04/18] ARM: imx: " Stephen Boyd
2013-07-31 22:31 ` Stephen Boyd
2013-07-31 22:31 ` [PATCH 05/18] ARM: integrator: " Stephen Boyd
2013-07-31 22:31 ` Stephen Boyd
2013-08-07 13:52 ` Linus Walleij
2013-08-07 13:52 ` Linus Walleij
2013-07-31 22:31 ` [PATCH 06/18] ARM: IXP4xx: " Stephen Boyd
2013-07-31 22:31 ` Stephen Boyd
2013-08-09 7:49 ` Krzysztof Halasa
2013-08-09 7:49 ` Krzysztof Halasa
2013-07-31 22:31 ` [PATCH 07/18] ARM: mmp: " Stephen Boyd
2013-07-31 22:31 ` Stephen Boyd
2013-07-31 22:31 ` [PATCH 08/18] ARM: msm: " Stephen Boyd
2013-07-31 22:31 ` Stephen Boyd
2013-08-01 16:12 ` David Brown
2013-08-01 16:12 ` David Brown
2013-07-31 22:31 ` [PATCH 09/18] ARM: OMAP1: " Stephen Boyd
2013-07-31 22:31 ` Stephen Boyd
2013-07-31 22:31 ` [PATCH 10/18] ARM: OMAP2+: " Stephen Boyd
2013-07-31 22:31 ` Stephen Boyd
2013-08-01 0:43 ` Santosh Shilimkar
2013-08-01 0:43 ` Santosh Shilimkar
2013-07-31 22:31 ` [PATCH 11/18] ARM: pxa: " Stephen Boyd
2013-07-31 22:31 ` Stephen Boyd
2013-07-31 22:31 ` [PATCH 12/18] ARM: sa1100: " Stephen Boyd
2013-07-31 22:31 ` Stephen Boyd
2013-07-31 22:31 ` [PATCH 13/18] ARM: u300: " Stephen Boyd
2013-07-31 22:31 ` Stephen Boyd
2013-08-07 18:55 ` Linus Walleij
2013-08-07 18:55 ` Linus Walleij
2013-07-31 22:31 ` [PATCH 14/18] ARM: iop: " Stephen Boyd
2013-07-31 22:31 ` Stephen Boyd
2013-07-31 22:31 ` [PATCH 15/18] ARM: OMAP: " Stephen Boyd
2013-07-31 22:31 ` Stephen Boyd
2013-07-31 22:31 ` [PATCH 16/18] ARM: orion: " Stephen Boyd
2013-07-31 22:31 ` Stephen Boyd
2013-08-01 0:31 ` Jason Cooper
2013-08-01 0:31 ` Jason Cooper
2013-07-31 22:31 ` [PATCH 17/18] ARM: SAMSUNG: " Stephen Boyd
2013-07-31 22:31 ` Stephen Boyd
2013-08-05 17:22 ` Kukjin Kim
2013-08-05 17:22 ` Kukjin Kim
2013-07-31 22:31 ` [PATCH 18/18] ARM: versatile: " Stephen Boyd
2013-07-31 22:31 ` Stephen Boyd
2013-08-29 21:56 ` [GIT PULL] 64bit friendly generic sched_clock() base for 3.12 John Stultz
2013-08-29 21:56 ` John Stultz
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=51F9F626.80306@ti.com \
--to=nsekhar@ti.com \
--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.