public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Ira W. Snyder <iws@ovro.caltech.edu>
To: u-boot@lists.denx.de
Subject: [U-Boot] Summary of my patches for v2010.12 merge window
Date: Thu, 9 Sep 2010 14:03:56 -0700	[thread overview]
Message-ID: <20100909210355.GH3496@ovro.caltech.edu> (raw)
In-Reply-To: <20100909194413.798AA153798@gemini.denx.de>

On Thu, Sep 09, 2010 at 09:44:13PM +0200, Wolfgang Denk wrote:
> Dear Timur Tabi,
> 
> In message <4C893536.6020109@freescale.com> you wrote:
> > 
> > 1. [v2] p1022ds: add audclk hwconfig setting to enable codec reference clock
> > 2. fsl: verify writes to the MAC address EEPROM
> > 3. always relocate fdt into an lmb-allocated memory block
> > 4. logos: add Freescale logo
> > 5. [v3] fsl: refactor MPC8610 and MPC5121 DIU code to use existing bitmap
> > and logo features
> > 6. p1022ds: add video support
> 
> Patches 5 and 6 are video related (and eventually # 4 as well); they
> need to go through Anatolij.
> 
> > Please note that Wolfgang has asked for someone else to test "always
> > relocate fdt into an lmb-allocated memory block", but no one has done so
> > yet, and you have no indicated that anyone will.
> 
> Indeed this needs to be tested.
> 

Per a conversation with Timur on linuxppc-dev, I have tested this patch
on an MPC8349EMDS board. It caused no problems. In order to force the
new code path to have an effect, I loaded my FIT image below
CONFIG_SYS_BOOTMAPSZ. Here is the relevant output from both runs:

Vanilla U-Boot (FIT image loaded to 0xa00000 == 10MB):
   Verifying Hash Integrity ... crc32+ OK
   Booting using the fdt blob at 0xc6a278
   Uncompressing Kernel Image ... OK
   Loading Ramdisk to 0fe9f000, end 0ff75699 ... OK

With Timur's patch:
   Verifying Hash Integrity ... crc32+ OK
   Booting using the fdt blob at 0xc42d6c
   Uncompressing Kernel Image ... OK
   Loading Ramdisk to 0fe9f000, end 0ff75699 ... OK
   Loading Device Tree to 00ff8000, end 00fff84f ... OK

You'll notice that Timur's patch now relocates the FDT, whereas vanilla
U-Boot did not.

If I load my FIT image above CONFIG_SYS_BOOTMAPSZ, both U-Boots (with
and without the patch) behave identically.

Hope it helps,
Ira

  reply	other threads:[~2010-09-09 21:03 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-09 19:27 [U-Boot] Summary of my patches for v2010.12 merge window Timur Tabi
2010-09-09 19:44 ` Wolfgang Denk
2010-09-09 21:03   ` Ira W. Snyder [this message]
2010-09-09 19:48 ` Anatolij Gustschin

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=20100909210355.GH3496@ovro.caltech.edu \
    --to=iws@ovro.caltech.edu \
    --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