public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Ulf Samuelsson <ulf@atmel.com>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] Proposal for patch to configure networkparameters
Date: Sat, 21 Apr 2007 01:29:11 +0200	[thread overview]
Message-ID: <02cc01c783a4$3a489920$01c4af0a@Glamdring> (raw)
In-Reply-To: 20070420225959.58640352650@atlas.denx.de

link: www.avrfreaks.net
----- Original Message ----- 
From: "Wolfgang Denk" <wd@denx.de>
To: "Ulf Samuelsson" <ulf@atmel.com>
Cc: <u-boot-users@lists.sourceforge.net>
Sent: Saturday, April 21, 2007 12:59 AM
Subject: Re: [U-Boot-Users] Proposal for patch to configure 
networkparameters


> In message <027c01c78394$16500590$01c4af0a@Glamdring> you wrote:
>>
>> Why should you care if the customer sets a configuration value at compile
>> time or dynamically?
>
> I don't care about this.
>
>> By setting things at compile time, you allow the board to boot up,
>> download a script which gets executed by "autoscr"
>> This script will sets the REAL values for the network.
>> Obviously the values should only be used until manually changed.
>
> OK, in such a configuration you can  add  the  values  to  the  board
> config file, as there is no need to change them frequently, right?
>
>> Not sure, if this is your problem.
>
> No.
>
>> Assume you have no serial communication available...
>> How do you reconfigure the system?
>
> We are not talking about reconfiguration, but about how to initialize
> the default environment. These are two different things.
>
>> Can you explain how to download the environment using the network
>> if you are not allowed to compile in the network parameters?
>
> You don't have to download such parameters.
>
> Please  feel  free  to  set  these  values  somewhere  during   board
> production,  but  you  better  not embed them into the default U-Boot
> image. There are several methods to initialize such parameters  which
> are much cleaner. For example, some boards use some hardware ID block
> which  gets  written separately from the U-Boot image, which then can
> contain board specific parameters like serial  number,  MAC  address,
> and - if you like - additional default netowrk configuration.


Why do you need to separate something which NEVER is going to be written 
again?
There is a cost involved with doing multiple programming steps and those 
should be minimized.

Even if you have multiple servers you can still allocate a duplicated IP 
address
to a network port on each server which is a point to point connection to the
board to be produced.

If there is a need to change these values, a simple recompile will fix that.

And still: The proposed patch will not do anything with network parameters
It will allow people to generate whatever config variables outside U-boot.

How they use this, you will not be aware of so I really do not understand
why you are against the patch.

The only thing the patch does is to add
#include <custom.h>
in the config.h

and then an empty custom.h is created
to make sure that it does not crash compiles of existing boards.
.
The patch has really nothing to do with networking,
so I suggest we stop discussing networking configuration
and start discussing the patch.

Are you against giving developers freedom?


Best Regards
Ulf Samuelsson

  reply	other threads:[~2007-04-20 23:29 UTC|newest]

Thread overview: 51+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-04-20 19:22 [U-Boot-Users] Proposal for patch to configure network parameters Ulf Samuelsson
2007-04-20 20:40 ` Timur Tabi
2007-04-20 20:49   ` Wolfgang Denk
2007-04-20 20:55   ` Ben Warren
2007-04-20 21:51     ` [U-Boot-Users] Proposal for patch to configurenetwork parameters Ulf Samuelsson
2007-04-20 23:12       ` Wolfgang Denk
2007-04-20 23:39         ` Ulf Samuelsson
2007-04-21  0:31           ` Wolfgang Denk
2007-04-21  6:07             ` Ulf Samuelsson
2007-04-21  6:17             ` Ulf Samuelsson
2007-04-22 19:56           ` Timur Tabi
2007-04-22 20:25             ` Wolfgang Denk
2007-04-22 20:30               ` Timur Tabi
2007-04-22 21:27             ` Ulf Samuelsson
2007-04-20 21:59   ` [U-Boot-Users] Proposal for patch to configure network parameters Ulf Samuelsson
2007-04-20 23:14     ` Wolfgang Denk
2007-04-20 23:46       ` [U-Boot-Users] Proposal for patch to configure networkparameters Ulf Samuelsson
2007-04-20 20:47 ` [U-Boot-Users] Proposal for patch to configure network parameters Wolfgang Denk
2007-04-20 21:35   ` [U-Boot-Users] Proposal for patch to configure networkparameters Ulf Samuelsson
2007-04-20 22:59     ` Wolfgang Denk
2007-04-20 23:29       ` Ulf Samuelsson [this message]
2007-04-21  0:25         ` Wolfgang Denk
2007-04-21  6:05           ` Ulf Samuelsson
2007-04-21 10:37             ` Robert Schwebel
2007-04-22  5:30               ` Ulf Samuelsson
2007-04-22  7:27                 ` Robert Schwebel
2007-04-22  8:45                   ` Ulf Samuelsson
2007-04-22 11:05                     ` Wolfgang Denk
2007-04-22 13:16                       ` Ulf Samuelsson
2007-04-22 13:58                         ` Wolfgang Denk
2007-04-22 14:28                           ` Ulf Samuelsson
2007-04-22 18:48                             ` Wolfgang Denk
2007-04-22 10:30                 ` Wolfgang Denk
2007-04-20 23:03   ` [U-Boot-Users] Proposal for patch to configure network parameters Timur Tabi
2007-04-20 23:29     ` Wolfgang Denk
2007-04-20 23:58       ` [U-Boot-Users] Proposal for patch to configure networkparameters Ulf Samuelsson
2007-04-21 12:41       ` [U-Boot-Users] Proposal for patch to configure network parameters Andreas Schweigstill
2007-04-22 20:11       ` Timur Tabi
2007-04-22 20:31         ` Wolfgang Denk
2007-04-22 21:50           ` [U-Boot-Users] Proposal for patch to configure networkparameters Ulf Samuelsson
2007-04-22 23:57             ` Wolfgang Denk
2007-04-23  5:43               ` Ulf Samuelsson
2007-04-23  6:46                 ` [U-Boot-Users] MPC8343 I2C problem Monstr at seznam.cz
2007-04-23 14:46                   ` Timur Tabi
2007-04-23  7:00                 ` [U-Boot-Users] MPC8343 NET without PHY Monstr at seznam.cz
2007-04-23 14:51                   ` Timur Tabi
2007-04-23  8:59                 ` [U-Boot-Users] Proposal for patch to configure networkparameters Wolfgang Denk
2007-04-23  9:24                   ` Ulf Samuelsson
2007-04-23 10:50                     ` Wolfgang Denk
2007-04-23  9:01                 ` Ulf Samuelsson
2007-04-22 21:38         ` Ulf Samuelsson

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='02cc01c783a4$3a489920$01c4af0a@Glamdring' \
    --to=ulf@atmel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox