linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
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...

  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).