public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Peter Barada <peter.barada@logicpd.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [RFC] Preventing overriding of serverip when set in environment by user
Date: Wed, 18 Apr 2012 11:46:41 -0400	[thread overview]
Message-ID: <4F8EE1E1.1060201@logicpd.com> (raw)
In-Reply-To: <20120418152806.3AE51202B25@gemini.denx.de>

On 04/18/2012 11:28 AM, Wolfgang Denk wrote:
>
>> Is this the proper way to fix this issue and the right location to
>> make the change? The overall goal was to make sure that if I
>> specified a particular serverip that I wanted to use, then the DHCP
>> server should not be changing that.
> No, this is not an acceptable patch.
>
> What exactly _are_ you trying to fix?  What is wrong with using the
> DHCP server supplied address?  If it is incorrect, you should fix your
> DHCP server configuration.
I have the same type of problem.

In my case our company controls the DHCP server that hands out addresses
for the network, but I want to use a *local* TFTP server (the one on my
development machine).  I can't modify the DHCP server so if I use "dhcp"
to get and address u-boot also overwrites serverip.  At home my FiOS
router has a DHCP server (that I can't modify since its closed and is
needed to provide addresses to my TV set top box) that provides
addresses for my network, and has the same problem...

Would an option to dhcp (i.e. one that tells it to *only* modify ipaddr)
suffice? Or would it be better to provide an entirely a new command that
optionally updates the variables in the environment be better? Then
users have the option of using "dhcp" as is, or for environments like
mine (and apparently Chase) can just get an IP address and go from there...

-- 
Peter Barada
peter.barada at logicpd.com

  reply	other threads:[~2012-04-18 15:46 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-04-18 13:28 [U-Boot] [RFC] Preventing overriding of serverip when set in environment by user Maupin, Chase
2012-04-18 15:28 ` Wolfgang Denk
2012-04-18 15:46   ` Peter Barada [this message]
2012-04-18 19:14     ` Wolfgang Denk
2012-04-19  9:13     ` Stefano Babic
2012-04-23 13:39       ` Maupin, Chase
2012-04-18 18:46   ` Chase Maupin
     [not found]   ` <7D46E86EC0A8354091174257B2FED10138DD6FB8@DLEE12.ent.ti.com>
2012-04-18 19:29     ` Wolfgang Denk

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=4F8EE1E1.1060201@logicpd.com \
    --to=peter.barada@logicpd.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