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] [PATCH] ATSTK1002: Remove default ethernet addresses
Date: Mon, 29 Oct 2007 12:45:54 +0100	[thread overview]
Message-ID: <010201c81a21$db9d6f30$01c4af0a@Glamdring> (raw)
In-Reply-To: 20071029122336.040782c4@dhcp-255-175.norway.atmel.com

> [re-adding mailing lists to Cc]
>
> On Mon, 29 Oct 2007 11:21:25 +0100
> "Ulf Samuelsson" <ulf@atmel.com> wrote:
>
>> Haavard,
>> For production use, you want to be able to set up a default address.
>
> I don't think this is necessary in our production setup at least.
>
>> All production PCs would have a dedicated Ethernet board, with a
>> predetemined IP address.
>> Nothing but the production PC and the board under production is connected 
>> to
>> this Ethernet.
>> Thus there is not a problem to have a fixed ethernet address on the AVR32
>> board.
>
> If you want this, feel free to patch it back into your production
> images. But as soon as someone erases the pre-programmed environment on
> two or more boards, you'll have chaos on the network they're connected
> to.

Not if you do the check I propose at the end.

> Btw, why don't you just pre-program a default environment sector right
> after you program the u-boot image? That way, you can have all the
> custom settings you want.
>

It is easier to do it inside U-boot, than outside U-boot.
It is easier to download one file, instead of downloading two files.
The effect is the same.
If you happen to change the address of the Environment in U-Boot
you need a mechanism to check that this is taken care of..

Also: I am not proposing a "default" address
just that the user can, by running a command, set the ethaddr
to something which is useful for production.
If the user does not run the command, then the address will remain unset.
Once the ethaddr is set to a different value, it should, as usual, not be
possible to change it.

>> This way you can define a bootcmd which downloads an Autoscript
>> which in its turn programs the linux kernel and the address.
>> When everything is finished, then the etherent address should
>> be programmed with its real value.
>
> So you need to define other environment variables anyway? Why can't you
> patch in a default ethernet address at the same time?
>

Because I want to download the setting over Ethernet, and
I cannot get access to the "serverip", without an Ethernet address.

>> To avoid problems, the linux kernel should not be bootable
>> if the kernel address is this "special" address.
>
> No way. I'm not adding bogus special-casing in the Linux kernel to cope
> with a problem I'm not even sure exists. Besides, you don't have to
> boot the kernel to experience problems; running "dhcp" from u-boot is
> enough to wreak havoc on the network.
>

This would be an U-Boot check.

If the Ethernet address is the predefined ethernet address
then the bootm command will give an error message.

> We've tried using a default ethernet address on both STK1000 and NGW100
> for a while now, and it causes a lot more problems than it solves. I'm
> turning it off unless someone can point to a specific, valid use-case
> that breaks.

Best Regards
Ulf Samuelsson

  reply	other threads:[~2007-10-29 11:45 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-10-29  7:45 [U-Boot-Users] [PATCH] ATSTK1002: Remove default ethernet addresses Haavard Skinnemoen
     [not found] ` <00b901c81a15$ae74e170$01c4af0a@Glamdring>
2007-10-29 11:23   ` Haavard Skinnemoen
2007-10-29 11:45     ` Ulf Samuelsson [this message]
2007-10-29 11:58       ` Haavard Skinnemoen
2007-10-29 15:29         ` Ulf Samuelsson
2007-10-29 15:55           ` Haavard Skinnemoen
2007-10-30 12:53             ` Robin Cutshaw
2007-10-30 13:25               ` Haavard Skinnemoen
2007-10-30 16:37               ` Scott Wood
2007-10-30 19:13               ` Ulf Samuelsson
2007-10-30 20:57                 ` Haavard Skinnemoen
2007-10-30 20:18                   ` Ulf Samuelsson
2007-10-29 15:29         ` 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='010201c81a21$db9d6f30$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