public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Wolfgang Denk <wd@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 1/2] fdt: Add fdt_fixup_nor_flash_size() to fixup NOR FLASH size in dtb
Date: Fri, 23 Oct 2009 10:33:10 +0200	[thread overview]
Message-ID: <20091023083310.39B6DE398E1@gemini.denx.de> (raw)
In-Reply-To: <4AE0D860.7030305@gmail.com>

Dear Jerry Van Baren,

In message <4AE0D860.7030305@gmail.com> you wrote:
> 
> Stefan Roese wrote:
> > This function can be used to update the size in the "reg" property
> > of the NOR FLASH device nodes. This is necessary for boards with
> > non-fixed NOR FLASH sizes.
> > 
> > Signed-off-by: Stefan Roese <sr@denx.de>
> > Cc: Wolfgang Denk <wd@denx.de>
> > Cc: Gerald Van Baren <vanbaren@cideas.com>
> 
> Looking at the code, I'm OK with it.
> 
> Are you looking to get this into the current release (v2009.11) as a 
> bugfix or the next release (v2010.02)?

I consider it a bug fix, as failing to adjust the flash size prevents
booting on some boards where the real flash size deviates from the
entry in the default DT in the Linux kernel.

> If you need it in the v2009.11 release, you have my
>    Acked-by Gerald Van Baren <vanbaren@cideas.com>
> and I suggest you run it through the ppc4xx repository with its sister 
> patch.

Thanks.

Stefan - also

Acked-by: Wolfgang Denk <wd@denx.de> 

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
If it went on at this rate, in several billion  years  he'd  be  rich
beyond his wildest dreams!            - Terry Pratchett, _Soul Music_

  reply	other threads:[~2009-10-23  8:33 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-10-21 11:25 [U-Boot] [PATCH 1/2] fdt: Add fdt_fixup_nor_flash_size() to fixup NOR FLASH size in dtb Stefan Roese
2009-10-22 22:10 ` Jerry Van Baren
2009-10-23  8:33   ` Wolfgang Denk [this message]
2009-10-23  9:22     ` Stefan Roese
2009-10-23 13:54 ` Stefan Roese

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=20091023083310.39B6DE398E1@gemini.denx.de \
    --to=wd@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