All of lore.kernel.org
 help / color / mirror / Atom feed
From: Graeme Russ <graeme.russ@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] [Timer]Remove calls to [get, reset]_timer outside arch/
Date: Tue, 24 May 2011 21:13:28 +1000	[thread overview]
Message-ID: <4DDB92D8.9090300@gmail.com> (raw)
In-Reply-To: <1306234440-3486-1-git-send-email-graeme.russ@gmail.com>

On 24/05/11 20:54, Graeme Russ wrote:
> There is no need to use set_timer() and reset_timer() and there are build
> breakages occuring because of them (specifically cfi_flash). Remove any
> usage outside arch/ to fix build breakages and to prepare for complete
> removal
> 
> The Nios2 timer implentation updates every 10ms (increases in 10ms steps)
> and required a reset_timer() call before timing operations. For Nios2, call
> reset_timer() when get_timer() is called with a parameter value of 0
> 
> Signed-off-by: Graeme Russ <graeme.russ@gmail.com>
> 
> ---
> checkpatch complains about long lines and brace usage in the board specific
> flash.c files - They are deprecated and not worth fixing for style
> 
> Changes since V1:
>  - Fix typo in commit message
>  - Add reset_timer() to Nios2 get_timer(0)
> ---
>  arch/nios2/cpu/interrupts.c    |    4 ++++
>  board/BuS/EB+MCF-EV123/flash.c |   10 ++++++----
>  board/cobra5272/flash.c        |   10 ++++++----
>  board/idmr/flash.c             |   10 ++++++----
>  drivers/block/mg_disk.c        |    1 -
>  drivers/mtd/cfi_flash.c        |    2 --
>  6 files changed, 22 insertions(+), 15 deletions(-)

Argh - Sorry about the subject - It should be [PATCH v2]

Do I need to resend or will that make matters worse?

Regards,

Graeme

  reply	other threads:[~2011-05-24 11:13 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-05-23 11:54 [U-Boot] [PATCH] [Timer]Remove calls to [get, reset]_timer outside arch/ Graeme Russ
2011-05-23 12:19 ` Scott McNutt
2011-05-23 12:32   ` Graeme Russ
2011-05-23 13:12     ` Scott McNutt
     [not found]       ` <4DDA9CDA.1080401@comcast.net>
     [not found]         ` <4DDAA77F.7020708@psyent.com>
2011-05-23 20:10           ` Graeme Russ
2011-05-23 20:49             ` J. William Campbell
2011-05-23 21:02               ` Graeme Russ
2011-05-23 21:15                 ` J. William Campbell
2011-05-24  5:13                 ` Graeme Russ
2011-05-24 15:41                   ` J. William Campbell
2011-05-23 21:53               ` Wolfgang Denk
2011-05-23 22:44                 ` Graeme Russ
2011-05-24  3:20                   ` Mike Frysinger
2011-05-23 12:31 ` Jens Scharsig
2011-05-23 13:22   ` Wolfgang Denk
2011-05-23 14:05     ` [U-Boot] Formal acks and patchwork (was: [PATCH] [Timer]Remove calls to [get, reset]_timer outside arch/) Albert ARIBAUD
2011-05-23 18:44       ` Wolfgang Denk
2011-05-23 19:19         ` [U-Boot] Formal acks and patchwork Albert ARIBAUD
2011-05-23 19:42           ` Wolfgang Denk
2011-05-23 20:15             ` Albert ARIBAUD
2011-05-23 13:34 ` [U-Boot] [PATCH] [Timer]Remove calls to [get, reset]_timer outside arch/ Jens Scharsig
     [not found] ` <BANLkTik3W7WJbRuiKsC8m0f8iof7JL-ZMg@mail.gmail.com>
2011-05-23 20:09   ` Graeme Russ
2011-05-24 10:54 ` Graeme Russ
2011-05-24 11:13   ` Graeme Russ [this message]
2011-05-25 12:17   ` Graeme Russ

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=4DDB92D8.9090300@gmail.com \
    --to=graeme.russ@gmail.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.