All of lore.kernel.org
 help / color / mirror / Atom feed
From: Steven Scholz <steven.scholz@imc-berlin.de>
To: Dan Malek <dan@embeddededge.com>
Cc: Linuxppc-Embedded <linuxppc-embedded@lists.linuxppc.org>
Subject: Re: was: FEC on MPC860T & race condition
Date: Fri, 07 Feb 2003 08:35:30 +0100	[thread overview]
Message-ID: <3E4361C2.8010007@imc-berlin.de> (raw)
In-Reply-To: 3E401AC9.9090703@embeddededge.com


Dan Malek schrieb:
> Tom Rini wrote:
>
>> I believe this is wrong, in that trying to udelay() here is a bad idea.
>
>
> I agree.  All of the MII communcation is interrupt driven.  This is easy
> and the way everything should work on the 8xx.  The link interrupt is
> more challenging because the interrupt from that depends upon the phy
> type and the board design.  The link interrupts are either real interrupts
> or managed with a timed thread.
>
> If you need to wait before installing the link interrupt (which I still
> don't understand why), this should be done as part of the phy discovery
> interrupt.  For example, add an indirect function pointer and if it isn't
> NULL, call it at that time to do anything that must wait until the phy
> is discovered and initialized.

I understand what you mean. And certanly you're right.
I moved the request_irq into 3 but my board still hanges from time to time. At a
different place in a different way though.

I have to admit that I don't understand this mii_queue stuff.
Please, Dan, would you please give me a small hint!
I assume by "add an indirect function pointer" you mean "add something to
mii_queue"... Is that right?

Thanks!

Steven


** Sent via the linuxppc-embedded mail list. See http://lists.linuxppc.org/

  reply	other threads:[~2003-02-07  7:35 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-01-30 17:05 was: FEC on MPC860T & race condition Steven Scholz
2003-01-30 20:22 ` Dan Malek
2003-01-31  9:16   ` Steven Scholz
2003-02-03 21:09     ` Tom Rini
2003-02-04  8:39       ` Steven Scholz
2003-02-04 16:04         ` Tom Rini
2003-02-04 16:14           ` Steven Scholz
2003-02-04 19:32             ` Tom Rini
2003-02-04 19:55               ` Dan Malek
2003-02-07  7:35                 ` Steven Scholz [this message]
2003-02-07  7:37               ` Steven Scholz
2003-02-07 17:12                 ` Tom Rini
2003-02-07 17:15                   ` Steven Scholz

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=3E4361C2.8010007@imc-berlin.de \
    --to=steven.scholz@imc-berlin.de \
    --cc=dan@embeddededge.com \
    --cc=linuxppc-embedded@lists.linuxppc.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.