All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thierry Reding <treding@nvidia.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] armv8: Allow SoCs to override the generic timer
Date: Tue, 28 Jul 2015 11:33:48 +0200	[thread overview]
Message-ID: <20150728093347.GC18202@ulmo.nvidia.com> (raw)
In-Reply-To: <1437605770-5462-1-git-send-email-swarren@wwwdotorg.org>

On Wed, Jul 22, 2015 at 04:56:10PM -0600, Stephen Warren wrote:
> From: Thierry Reding <treding@nvidia.com>
> 
> Some SoCs come with a custom timer interface, so allow them to use that
> instead.
> 
> swarren notes: I did consider reworking this patch so the Makefile only
> compiles generic_timer.c ifndef CONFIG_SYS_TIMER_COUNTER. However, I can
> foresee a time where generic_timer.c contains both routines to implement
> U-Boot's internal timer, and to perform other operations related to the
> generic timer, such as initializing the CNTFRQ register for a booted OS.
> If that happens, this version of the change will result in simpler future
> patches.
> 
> Signed-off-by: Thierry Reding <treding@nvidia.com>
> Signed-off-by: Tom Warren <twarren@nvidia.com>
> Signed-off-by: Stephen Warren <swarren@nvidia.com>
> ---
>  arch/arm/cpu/armv8/generic_timer.c | 2 ++
>  1 file changed, 2 insertions(+)

Marc Zyngier and David Feng had objections to this when I posted[0]. I
ended up solving this differently, by initializing the architectural
timer earlier and then using the architected timer on ARM64.

I'll post the two patches I have in my tree as a replacement for this
and sorry for causing such a mess.

Thierry

[0]: http://lists.denx.de/pipermail/u-boot/2015-March/208505.html
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150728/74770271/attachment.sig>

  parent reply	other threads:[~2015-07-28  9:33 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-22 22:56 [U-Boot] [PATCH] armv8: Allow SoCs to override the generic timer Stephen Warren
2015-07-23 20:18 ` Simon Glass
2015-07-23 20:54 ` Tom Rini
2015-07-28  9:33 ` Thierry Reding [this message]
2015-07-28  9:35 ` [U-Boot] [PATCH 1/2] ARM: tegra: Initialize timer earlier Thierry Reding
2015-07-28  9:35   ` [U-Boot] [PATCH 2/2] ARM: tegra: Use architected timer on ARMv8 Thierry Reding
2015-08-02 21:29     ` Simon Glass
2015-07-28 16:08   ` [U-Boot] [PATCH 1/2] ARM: tegra: Initialize timer earlier Stephen Warren

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=20150728093347.GC18202@ulmo.nvidia.com \
    --to=treding@nvidia.com \
    --cc=u-boot@lists.denx.de \
    /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.