All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michal Simek <monstr@monstr.eu>
To: u-boot@lists.denx.de
Subject: [U-Boot] Environmentvar. "ipaddr" - microblaze
Date: Mon, 22 Mar 2010 18:24:09 +0100	[thread overview]
Message-ID: <4BA7A7B9.6000103@monstr.eu> (raw)
In-Reply-To: <m2hbo8s42l.fsf@ohwell.denx.de>

Detlev Zundel wrote:
> Hi Michal,
> 
>> Detlev Zundel wrote:
>>> Hi Horst,
>>>
>>>> in "lib_microblaze/board.c  board_init()" the function
>>>> "env_relocate()" should be called before "getenv_IPaddr()"
>>>> otherwise there is no chance to save an ip-address in the
>>>> environment (like in lib_ppc/board.c).
>> Horst: Give me step-by-step manual how to reach that fault.
> 
> As long as Horst doesn't answer, I belive he has his environment in SPI
> flash and was not able to change the "ipaddr" setting used by the
> network code.

First off all I am looking forward on that spi driver.

> 
> Now all I can think of is that in lib_microblaze/board.c:144 the call
> returns the environmetn setting of the compiled in environment rather
> than the SPI flash saved settings.  Thus without doing an explicit
> "setenv ipaddr" network will always use the compiled in ipaddr.
> 
> Does that make sense?

Anyway. I don't need to setup different IP address from command line.

The funny thing is that I have this change in my private repo I sync it 
with ppc some weeks/month ago.
I will have to invest some time to add all my changes to mainline.

If you want to add only this change, I am ok with it.

Michal


> 
> Cheers
>   Detlev
> 


-- 
Michal Simek, Ing. (M.Eng)
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel 2.6 Microblaze Linux - http://www.monstr.eu/fdt/
Microblaze U-BOOT custodian

  reply	other threads:[~2010-03-22 17:24 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-03-17 22:12 [U-Boot] Environmentvar. "ipaddr" - microblaze Horst Gall
2010-03-18  9:38 ` Detlev Zundel
2010-03-22 13:20   ` Michal Simek
2010-03-22 16:40     ` Detlev Zundel
2010-03-22 17:24       ` Michal Simek [this message]
2010-03-23  9:55         ` Detlev Zundel
2010-03-23 13:44           ` Michal Simek
2010-03-23 11:03     ` Horst Gall
2010-03-23 13:41       ` Michal Simek

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=4BA7A7B9.6000103@monstr.eu \
    --to=monstr@monstr.eu \
    --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.