All of lore.kernel.org
 help / color / mirror / Atom feed
From: nicolas.ferre@atmel.com (Nicolas Ferre)
To: linux-arm-kernel@lists.infradead.org
Subject: Timekeeping on at91rm9200 [Was: [PATCH v2] clockevents: Sanitize ticks to nsec conversion]
Date: Fri, 4 Oct 2013 12:00:53 +0200	[thread overview]
Message-ID: <524E91D5.5020804@atmel.com> (raw)
In-Reply-To: <20130924211139.GE16106@pengutronix.de>

On 24/09/2013 23:11, Uwe Kleine-K?nig :
> Hello Nicolas, hello Jean-Christophe,
>
> I expect you should be able to revert
>
> 	b7a8ca5 (ARM: at91: rm9200 fix time support)
>
> which reverted
>
> 	838a2ae (ARM: use clockevents_config_and_register() where possible)
>
> for at91rm9200 on top of the patch this mail is a reply to.
>
> Can you please test that?

I have just tested that on at91rm9200-ek:
- revert the revert (!)
- experience the crash
- then apply the v2 patch
- and finally see it working nicely!

Thanks a lot for having corrected this. Now, do you want that I stack 
this revert-of-revert on the AT91 git tree or you are able to apply it 
through the clock git tree together with the patch that is fixing the 
issue (it may prevent synchronization difficulties)? (BTW, I do not see 
it yet in "next").

Anyway, if it is required, you can add my:

Tested-by: Nicolas Ferre <nicolas.ferre@atmel.com>

Thanks, bye,
-- 
Nicolas Ferre

WARNING: multiple messages have this Message-ID (diff)
From: Nicolas Ferre <nicolas.ferre@atmel.com>
To: "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>,
	"Jean-Christophe PLAGNIOL-VILLARD" <plagnioj@jcrosoft.com>,
	"Thomas Gleixner" <tglx@linutronix.de>,
	LAK <linux-arm-kernel@lists.infradead.org>
Cc: Russell King - ARM Linux <linux@arm.linux.org.uk>,
	<linux-kernel@vger.kernel.org>, Marc Pignat <marc.pignat@hevs.ch>,
	"Ludovic Desroches" <ludovic.desroches@atmel.com>,
	<john.stultz@linaro.org>, <kernel@pengutronix.de>,
	Marc Kleine-Budde <mkl@pengutronix.de>,
	Ronald Wahl <ronald.wahl@raritan.com>
Subject: Re: Timekeeping on at91rm9200 [Was: [PATCH v2] clockevents: Sanitize ticks to nsec conversion]
Date: Fri, 4 Oct 2013 12:00:53 +0200	[thread overview]
Message-ID: <524E91D5.5020804@atmel.com> (raw)
In-Reply-To: <20130924211139.GE16106@pengutronix.de>

On 24/09/2013 23:11, Uwe Kleine-König :
> Hello Nicolas, hello Jean-Christophe,
>
> I expect you should be able to revert
>
> 	b7a8ca5 (ARM: at91: rm9200 fix time support)
>
> which reverted
>
> 	838a2ae (ARM: use clockevents_config_and_register() where possible)
>
> for at91rm9200 on top of the patch this mail is a reply to.
>
> Can you please test that?

I have just tested that on at91rm9200-ek:
- revert the revert (!)
- experience the crash
- then apply the v2 patch
- and finally see it working nicely!

Thanks a lot for having corrected this. Now, do you want that I stack 
this revert-of-revert on the AT91 git tree or you are able to apply it 
through the clock git tree together with the patch that is fixing the 
issue (it may prevent synchronization difficulties)? (BTW, I do not see 
it yet in "next").

Anyway, if it is required, you can add my:

Tested-by: Nicolas Ferre <nicolas.ferre@atmel.com>

Thanks, bye,
-- 
Nicolas Ferre

  reply	other threads:[~2013-10-04 10:00 UTC|newest]

Thread overview: 62+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-09-13 13:02 [PATCH] [PATCH] clocksource: tcb: fix min_delta calculation Marc Kleine-Budde
2013-09-13 13:02 ` Marc Kleine-Budde
2013-09-17  9:56 ` Ludovic Desroches
2013-09-17 10:04   ` Russell King - ARM Linux
2013-09-17 10:04     ` Russell King - ARM Linux
2013-09-17 11:26     ` Thomas Gleixner
2013-09-17 11:26       ` Thomas Gleixner
2013-09-17 13:01       ` Ludovic Desroches
2013-09-17 21:15         ` [PATCH] clockevents: Sanitize ticks to nsec conversion Thomas Gleixner
2013-09-17 21:15           ` Thomas Gleixner
2013-09-17 22:25           ` Marc Kleine-Budde
2013-09-17 22:25             ` Marc Kleine-Budde
2013-09-17 23:20             ` Thomas Gleixner
2013-09-17 23:20               ` Thomas Gleixner
2013-09-18  7:33           ` Ludovic Desroches
2013-09-18  8:56           ` Uwe Kleine-König
2013-09-18  8:56             ` Uwe Kleine-König
2013-09-18  9:38             ` Thomas Gleixner
2013-09-18  9:38               ` Thomas Gleixner
2013-09-18 15:09               ` Uwe Kleine-König
2013-09-18 15:09                 ` Uwe Kleine-König
2013-09-18 22:01                 ` Thomas Gleixner
2013-09-18 22:01                   ` Thomas Gleixner
2013-09-19 10:02                   ` Uwe Kleine-König
2013-09-19 10:02                     ` Uwe Kleine-König
2013-09-19 10:15                     ` Thomas Gleixner
2013-09-19 10:15                       ` Thomas Gleixner
2013-09-19 12:48                       ` Uwe Kleine-König
2013-09-19 12:48                         ` Uwe Kleine-König
2013-09-19 13:12                         ` Thomas Gleixner
2013-09-19 13:12                           ` Thomas Gleixner
2013-09-19 14:30                         ` Thomas Gleixner
2013-09-19 14:30                           ` Thomas Gleixner
2013-09-19 20:03                           ` Uwe Kleine-König
2013-09-19 20:03                             ` Uwe Kleine-König
2013-09-20  9:56                             ` Thomas Gleixner
2013-09-20  9:56                               ` Thomas Gleixner
2013-09-20 20:41                               ` Uwe Kleine-König
2013-09-20 20:41                                 ` Uwe Kleine-König
2013-09-20 21:30                                 ` Thomas Gleixner
2013-09-20 21:30                                   ` Thomas Gleixner
2013-09-24 19:50                           ` [PATCH v2] " Uwe Kleine-König
2013-09-24 19:50                             ` Uwe Kleine-König
2013-09-24 21:11                             ` Timekeeping on at91rm9200 [Was: [PATCH v2] clockevents: Sanitize ticks to nsec conversion] Uwe Kleine-König
2013-09-24 21:11                               ` Uwe Kleine-König
2013-10-04 10:00                               ` Nicolas Ferre [this message]
2013-10-04 10:00                                 ` Nicolas Ferre
2013-09-24 21:16                             ` [PATCH v2] clockevents: Sanitize ticks to nsec conversion Uwe Kleine-König
2013-09-24 21:16                               ` Uwe Kleine-König
2013-10-08 10:08                             ` Marc Kleine-Budde
2013-10-08 10:08                               ` Marc Kleine-Budde
2013-10-08 15:31                               ` [GIT PULL] fixes for integer rounding in timer core (Was: [PATCH v2] clockevents: Sanitize ticks to nsec conversion) Uwe Kleine-König
2013-10-08 15:31                                 ` Uwe Kleine-König
2013-10-14  7:34                                 ` [GIT PULL] fixes for integer rounding in timer core Uwe Kleine-König
2013-10-14  7:34                                   ` Uwe Kleine-König
2013-10-16 14:19                                   ` Nicolas Ferre
2013-10-16 14:19                                     ` Nicolas Ferre
2013-10-21  7:12                                   ` Uwe Kleine-König
2013-10-21  7:12                                     ` Uwe Kleine-König
2013-10-21 20:53                                     ` Daniel Lezcano
2013-10-21 20:53                                       ` Daniel Lezcano
2013-10-23 10:56                             ` [tip:timers/urgent] clockevents: Sanitize ticks to nsec conversion tip-bot for Thomas Gleixner

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=524E91D5.5020804@atmel.com \
    --to=nicolas.ferre@atmel.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.