All of lore.kernel.org
 help / color / mirror / Atom feed
From: Steven Scholz <steven.scholz@imc-berlin.de>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] [Fwd: [PPCBoot-users] DHCP updating environment???]
Date: Tue, 05 Nov 2002 14:09:59 +0100	[thread overview]
Message-ID: <3DC7C327.2CA471E@imc-berlin.de> (raw)
In-Reply-To: 20021105125715.08F471015F@denx.denx.de

Wolfgang Denk wrote:

> > is there a reason why BootpCopyNetParams() in bootp.c does _not_ setenv the
> > ipaddr?
> 
> IIRC this was  requested  by  one  /  some  users  with  nobody  else
> complaining about this. [But this discussion may have happned about 2
> years ago, so I may be wrong.]
> 
> > If I set autoload=n I want to use the dhcp command just to get the IP address
> > from the server. But PPCBoot only sets the "bootfile" variable and nothing more!
> 
> No. With DHCP you cannot "just get the IP address"; you  will  always
> get  what the DHCP server has to offer, and you default configuration
> will most certainly ask for more than just the IP address.
> 
> > Wouldn't it make sense to set all the params we got from the DHCP server?
> 
> Maybe, maybe not. This depends on a lot of things.
> 
> For example: assume I manually set a parameter (maybe  ipaddr,  maybe
> gatewayip); in this case I probably want to make sure that my pre-set
> values   take   precedence   over   any  DHCP  supplied  (additional)
> information. But then, I cannot simply implement some code which sets
> the params gotten via DHCP if they are not present in the environment
> - I think I remember there was an argument that in such  a  situation
> you  would  not be able to get rid of a faulty setting suppllied once
> by a misconfigured DHCP server.

Fair enough!
That's why I mentioned CONFIG_BOOTP_MASK. This could be use to decide/filter
which setting supplied by a DHCP server to use.
If you have to deal with a faulty server and you don't want/can update PPCBoot,
maybe we can introduce an environment variable like "dhcp_mask" ...

> > Well, at least the ones we're interessted in (see CONFIG_BOOTP_MASK).
> 
> I understand what you mean, but I would like to  hear  more  feedback
> from others...
Yes please! :o)

Again: What's the use of setting autoload to "n"?
I thought it's to get the settings from a DHCP server without immediately
loading a file, so I can use the settings (IP, server ip etc) for something
else, e.g. tftp'ing an FPGA file or ppcboot (ehm sorry U-Boot) image.

Thanks,

Steven

  reply	other threads:[~2002-11-05 13:09 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-11-05 12:22 [U-Boot-Users] [Fwd: [PPCBoot-users] DHCP updating environment???] Steven Scholz
2002-11-05 12:57 ` Wolfgang Denk
2002-11-05 13:09   ` Steven Scholz [this message]
2002-11-05 13:22     ` Wolfgang Denk
2002-11-05 13:33       ` Steven Scholz
2002-11-05 14:03         ` Wolfgang Denk
2002-11-05 15:07           ` Steven Scholz
2002-11-05 15:30             ` Steven Scholz
2002-11-05 15:44               ` Steven Scholz
2002-11-05 15:49                 ` Wolfgang Denk
2002-11-05 17:02                   ` 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=3DC7C327.2CA471E@imc-berlin.de \
    --to=steven.scholz@imc-berlin.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.