public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Scott Wood <scottwood@freescale.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [ARM] Environment variables not available during console initialisation?
Date: Tue, 03 Feb 2009 10:15:15 -0600	[thread overview]
Message-ID: <49886D93.7000908@freescale.com> (raw)
In-Reply-To: <Pine.LNX.4.64.0902030842390.4902@axis700.grange>

Guennadi Liakhovetski wrote:
> On Tue, 3 Feb 2009, Wolfgang Denk wrote:
>> I don't see what nand(_spl) maintainers should comment here  -  their
>> code  is  so  far  completley  unaffected.  We just agreed on needing
>> another call to the load routine.
> 
> We still have the other possibility - properly initialise NAND before 
> calling env_init() and then read out env? This would be a preferred 
> solution IMHO, because it leverages restrictions like "only the boot 
> NAND," simplifies the choice of a location in RAM, leaves nand_spl code 
> intact, thus avoiding even minimal size increment. NAND maintainer's 
> opinion as to how feasible / difficult this would be would be appreciated. 
> Or even better - a patch:-) I think, we could even move mem_malloc_init() 
> before the initcall loop and use a malloc()ed buffer just as 
> env_relocate() would do.

Besides Wolfgang's comment that this would preclude serial output during 
NAND init, this won't work if you're booting from NOR flash and thus 
don't have RAM initialized yet.  And even with NAND boot, this is prior 
to relocation to the top of RAM, which might complicate things.

We could leave nand_spl code completely untouched if we pad the NAND 
image to a block boundary, and then append the environment.  nand_spl 
would just see a larger image.

-Scott

  parent reply	other threads:[~2009-02-03 16:15 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-01-15  7:33 [U-Boot] [ARM] Environment variables not available during console initialisation? Guennadi Liakhovetski
2009-01-15 18:37 ` Wolfgang Denk
2009-01-15 19:42 ` Ulf Samuelsson
2009-01-15 22:43   ` Wolfgang Denk
2009-02-02 16:35     ` Guennadi Liakhovetski
2009-02-02 20:07       ` Wolfgang Denk
2009-02-02 20:55         ` Guennadi Liakhovetski
2009-02-02 21:16           ` Wolfgang Denk
2009-02-02 22:37             ` Guennadi Liakhovetski
2009-02-02 23:55               ` Wolfgang Denk
2009-02-03  8:14                 ` Guennadi Liakhovetski
2009-02-03  8:52                   ` Wolfgang Denk
2009-02-03 16:15                   ` Scott Wood [this message]
  -- strict thread matches above, loose matches on Subject: below --
2009-02-02 19:05 [U-Boot] [ARM] Environment variables not available during, " Derek Ou

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=49886D93.7000908@freescale.com \
    --to=scottwood@freescale.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