From: Stefano Babic <sbabic@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [RFC] Preventing overriding of serverip when set in environment by user
Date: Thu, 19 Apr 2012 11:13:44 +0200 [thread overview]
Message-ID: <4F8FD748.80808@denx.de> (raw)
In-Reply-To: <4F8EE1E1.1060201@logicpd.com>
On 18/04/2012 17:46, Peter Barada wrote:
> 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...
There is already such as option. You can pass the tftp server in the
"tftp"command, see code:
tftp [loadAddress] [[hostIPaddr:]bootfilename]
for example, something like
tftp 0x80008000 192.168.1.1:uImage
Is it not enough ?
Best regards,
Stefano Babic
--
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================
next prev parent reply other threads:[~2012-04-19 9:13 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
2012-04-18 19:14 ` Wolfgang Denk
2012-04-19 9:13 ` Stefano Babic [this message]
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=4F8FD748.80808@denx.de \
--to=sbabic@denx.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox