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 configurenetwork	parameters
Date: Fri, 20 Apr 2007 23:51:11 +0200	[thread overview]
Message-ID: <028f01c78396$318ebe80$01c4af0a@Glamdring> (raw)
In-Reply-To: 1177102522.3904.28.camel@saruman.qstreams.net

> On Fri, 2007-04-20 at 15:40 -0500, Timur Tabi wrote:
>> Ulf Samuelsson wrote:
>> > U-Boot has  a number of network parameters like
>> >
>> > CONFIG_HOSTNAME
>> > CONFIG_IPADDR
>> > CONFIG_GATEWAYIP
>> > CONFIG_NETMASK
>> > etc.
>> >
>> > While these can be defined in an "include/configs/<board>.h",
>> > it would be nice to be able to define this in a separate file
>> > since they are really not board specific.
>>
>> I agree that these don't belong in the board config file.  In fact, I 
>> don't see why they
>> should be board-specific at all.  The U-Boot code in denx.de should have 
>> a config.h
>> somewhere that specifies these, but we should probably not encourage 
>> board maintainers to
>> have their own versions.
>>
> If these network parameters really need default values, it's probably
> better to initialize environment variables using
> CONFIG_EXTRA_ENV_SETTINGS, or something like that.  I expect that almost
> every board that U-boot gets loaded on will have these values
> overwritten anyway to fit in whatever network it ends up in.
>

I think everyone so far is missing the point.
My proposal is just to allow the u-boot developer to customize his/her build
with environment variables generated outside the u-boot source tree.

It does not specify what the user is doing with this new flexibility.

Doing it this way is the "clean" way in my opinion,
but If Wolfgang is going to complain, then I just
have to do it anyway, but outside U-boot.

It is very easy to just copy an "include/config/<board>.h" file
from outside u.-boot to "include/configs", adding the extra stuff
I need at the end of that file.

Even if WD does not like fixed IP configuration there are other use for this

If you have a board with multiple memories, and you can store
you environment in all of them, then you can use this function to
define where the environment is instead of having multiple board files.
It is really a pin to maintain multiple board files for this purpose alone.


> regards,
> Ben
>
>


Best Regards
Ulf Samuelsson                ulf at atmel.com
Atmel Nordic AB
Mail:  Box 2033, 174 02 Sundbyberg, Sweden
Visit:  Kavalleriv?gen 24, 174 58 Sundbyberg, Sweden
Phone +46 (8) 441 54 22     Fax +46 (8) 441 54 29
GSM    +46 (706) 22 44 57

Technical support when I am not available:
AT89 C51 Applications Group: mailto:micro.hotline at nto.atmel.com
AT90 AVR Applications Group: mailto:avr at atmel.com
AT91 ARM Applications Group: mailto:at91support at atmel.com
FPSLIC Application Group: mailto:fpslic at atmel.com Best AVR

  reply	other threads:[~2007-04-20 21:51 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     ` Ulf Samuelsson [this message]
2007-04-20 23:12       ` [U-Boot-Users] Proposal for patch to configurenetwork parameters 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
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='028f01c78396$318ebe80$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