All of lore.kernel.org
 help / color / mirror / Atom feed
From: Detlev Zundel <dzu@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] Start/stop of network devices (was: Re: [PATCH] UEC PHY: Speed up initial PHY neg.)
Date: Mon, 23 Aug 2010 17:19:40 +0200	[thread overview]
Message-ID: <m2wrrh73w3.fsf_-_@ohwell.denx.de> (raw)
In-Reply-To: <4C721E61.5000608@gmail.com> (Ben Warren's message of "Mon, 23 Aug 2010 00:08:17 -0700")

Hi Ben,

[Jocke deleted from CC as this is not about the patch anymore]

> Sorry for the delay on this.  I'm all for changing the existing
> behavior.  It seems to me that the only time we would ever want to
> wind an interface down is if we switch the active one (even then, I'm
> not sure).  My world view is limited, but I can't imagine that even
> changing interfaces happens much in real world U-boot usage, that is
> the non-lab, non-interactive use cases.  What would you think about
> adding something like ifup and ifdown commands so that users could
> explicitly start/stop interfaces?

Actually I was thinking of an automatic initialisiation once we do a
network transfer.  This would be more in line with the current command
line interface.  For this we would need a state variable per device and
start the device if it is down.

The "ifdown" _may_ be implemented as a command, but I think this should
be done automatically once we boot a kernel.  For USB we already have to
do something like this as USB host controllers tend to DMA frame numbers
into memory as long as they are running.  This was an especially
difficult problem to diagnose back then.  From this lesson I think we
should learn and call "stop" methods for every initialized device.

What do you think?

Once more this is going into the "device" model dicussion that we
already thought about a while ago [1].  Maybe someday somebody finds
some time and starts to work on it ;)

Cheers
  Detlev

[1] http://www.denx.de/wiki/view/U-Boot/OLSUbootBOF

-- 
A foolish consistency is the hobgoblin of little minds.
                                   -- Ralph Waldo Emerson             
--
DENX Software Engineering GmbH,      MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: dzu at denx.de

  parent reply	other threads:[~2010-08-23 15:19 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-08-10 14:36 [U-Boot] [PATCH] UEC PHY: Speed up initial PHY neg Joakim Tjernlund
2010-08-10 20:23 ` Mike Frysinger
2010-08-11  6:20   ` Joakim Tjernlund
2010-08-12 12:58 ` Detlev Zundel
2010-08-12 14:09   ` Joakim Tjernlund
2010-08-13  8:20     ` Detlev Zundel
2010-08-13 13:18       ` Joakim Tjernlund
2010-08-23  7:08       ` Ben Warren
2010-08-23  7:53         ` Joakim Tjernlund
2010-08-23 14:12           ` Ben Warren
2010-08-23 14:53             ` Joakim Tjernlund
2010-08-23 15:19         ` Detlev Zundel [this message]
2010-08-24 18:35           ` [U-Boot] Start/stop of network devices (was: Re: [PATCH] UEC PHY: Speed up initial PHY neg.) Joakim Tjernlund
2010-09-13  4:18 ` [U-Boot] [PATCH] UEC PHY: Speed up initial PHY neg Ben 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=m2wrrh73w3.fsf_-_@ohwell.denx.de \
    --to=dzu@denx.de \
    --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.