From: marc.zyngier@arm.com (Marc Zyngier)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 1/3] ARM: davinci: convert to common sched_clock() implementation
Date: Wed, 18 Jan 2012 18:50:46 +0000 [thread overview]
Message-ID: <4F171486.80105@arm.com> (raw)
In-Reply-To: <DF0F476B391FA8409C78302C7BA518B631794A9F@DBDE01.ent.ti.com>
Hi Sekhar,
On 18/01/12 18:36, Nori, Sekhar wrote:
> On Wed, Jan 18, 2012 at 21:44:50, Marc Zyngier wrote:
>> Davinci has its own sched_clock() implementation, which gets in the
>> way of a single zImage. Moving to the common sched_clock framework
>> makes the code slightly cleaner.
>>
>> Cc: Sekhar Nori <nsekhar@ti.com>
>> Cc: Kevin Hilman <khilman@ti.com>
>> Signed-off-by: Marc Zyngier <marc.zyngier@arm.com>
>
> Tested this on DaVinci DM365 and works well.
Thanks for giving it a go.
>> diff --git a/arch/arm/mach-davinci/time.c b/arch/arm/mach-davinci/time.c
>> index e1969ce..187fcda 100644
>> --- a/arch/arm/mach-davinci/time.c
>> +++ b/arch/arm/mach-davinci/time.c
>> @@ -20,6 +20,7 @@
>> #include <linux/platform_device.h>
>>
>> #include <mach/hardware.h>
>> +#include <asm/sched_clock.h>
>> #include <asm/mach/irq.h>
>> #include <asm/mach/time.h>
>> #include <mach/cputype.h>
>
> I did not like the fact that the asm/ and mach/ headers
> are intermixed in this file so I created a patch to
> fix that while I was doing the common sched_clock() conversion.
>
> http://www.spinics.net/lists/arm-kernel/msg153506.html
>
> Can you please rebase your patch on that?
Sure. I can probably merge that patch altogether in mine if you want.
> If you are planning to take this from outside of DaVinci tree,
> feel free to add my:
>
> Acked-by: Sekhar Nori <nsekhar@ti.com>
Thanks. Ideally, I'd like all 3 patches to be kept together, otherwise
we may end up with patch 3 being applied before patch 1, breaking your
platform.
Cheers,
M.
--
Jazz is not dead. It just smells funny...
next prev parent reply other threads:[~2012-01-18 18:50 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-01-18 16:14 [PATCH v2 0/3] Convert remaining platforms to common sched_clock() Marc Zyngier
2012-01-18 16:14 ` [PATCH v2 1/3] ARM: davinci: convert to common sched_clock() implementation Marc Zyngier
2012-01-18 18:36 ` Nori, Sekhar
2012-01-18 18:50 ` Marc Zyngier [this message]
2012-01-18 20:16 ` Nori, Sekhar
2012-01-18 16:14 ` [PATCH v2 2/3] ARM: prima2: " Marc Zyngier
2012-01-18 16:14 ` [PATCH v2 3/3] ARM: Make the sched_clock framework mandatory Marc Zyngier
2012-01-18 16:26 ` Russell King - ARM Linux
2012-01-18 16:29 ` Marc Zyngier
2012-01-18 20:14 ` Nicolas Pitre
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=4F171486.80105@arm.com \
--to=marc.zyngier@arm.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).