All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dave Peverley <dpeverley@mpc-data.co.uk>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] v1.1.0, omap boards and udelay()
Date: Mon, 19 Apr 2004 12:59:32 +0100	[thread overview]
Message-ID: <4083BF24.9030500@mpc-data.co.uk> (raw)

Hi all,

  I'm just merging my omap730p2 port into the 1.1.0 codebase, and have hit
a snag that I would have thought would affect the omap1510 and 1610 code as
well ; can anyone confirm this? The problem is as follows :
 
  in boards/omapXXXX/omapXXXX.c, the function ether__init() uses the 
udelay() call. Prior to version 1.1.0, this was implemented using :
  #if defined(CONFIG_INNOVATOROMAP1610) || defined(CONFIG_P2OMAP730)
  #define LOOPS_PER_MSEC 100		/* tuned on omap1610 */
      volatile int i, time_remaining = LOOPS_PER_MSEC * usec;
      for (i = time_remaining; i > 0; i--) {
  }
  #else
so that the omap's wouldn't use the MPU timer, as this does not work (at 
least at that point in the startup) This means that u-boot hangs on my 
board during ether__init() now as the timer never gets incremented. 

  Can anyone tell me why this code was removed? Looking at it, I can't see
a reason why it shouldn't break the 1510/1610 omap support as well. I've 
tested moving the code back as it was (using the 1.1.0 CONFIG_OMAP instead)
and this fixes the problem once more. 

Best Wishes,

Dave Peverley

---------------------------------------------------------------------------
             Dave Peverley, Software Engineer, MPC Data Limited.
Phone : [+44] (0) 1225 868 228              Web : http://www.mpc-data.co.uk

             reply	other threads:[~2004-04-19 11:59 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-04-19 11:59 Dave Peverley [this message]
  -- strict thread matches above, loose matches on Subject: below --
2004-04-20 16:30 [U-Boot-Users] v1.1.0, omap boards and udelay() Kamat, Nishant
2004-04-21  7:23 Gupta, Kshitij
2004-04-25 13:39 ` Wolfgang Denk
2004-04-26  9:31   ` Dave Peverley
2004-04-26 11:03     ` Wolfgang Denk
2004-04-26 11:15       ` Dave Peverley
2004-04-26 12:47         ` Wolfgang Denk

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=4083BF24.9030500@mpc-data.co.uk \
    --to=dpeverley@mpc-data.co.uk \
    --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.