public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Stephan Linz <linz@li-pro.net>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 4/4] microblaze: generic: enable FDT support
Date: Thu, 24 Jun 2010 08:15:02 +0200	[thread overview]
Message-ID: <201006240815.02843.linz@li-pro.net> (raw)
In-Reply-To: <4C21A94A.8000801@monstr.eu>

Hi Michal,

I will work on your proposals today or tomorrow (European time). Please give 
me some time.

br,
Stephan

Am Mittwoch, 23. Juni 2010 08:27:22 schrieb Michal Simek:
> Stephan Linz wrote:
> > Am Dienstag, 22. Juni 2010 09:25:56 schrieb Michal Simek:
> >> Stephan Linz wrote:
> >>> Signed-off-by: Stephan Linz <linz@li-pro.net>
> >>> ---
> >>>  include/configs/microblaze-generic.h |    8 ++++++++
> >>>  1 files changed, 8 insertions(+), 0 deletions(-)
> >>>
> >>> diff --git a/include/configs/microblaze-generic.h
> >>> b/include/configs/microblaze-generic.h index 28cee47..89e6dbb 100644
> >>> --- a/include/configs/microblaze-generic.h
> >>> +++ b/include/configs/microblaze-generic.h
> >>> @@ -305,4 +305,12 @@
> >>>  #define CONFIG_SYS_PROMPT_HUSH_PS2 "> "
> >>>  #endif
> >>>
> >>> +/* pass open firmware flat tree */
> >>> +#define CONFIG_FIT		1
> >>> +#define CONFIG_OF_LIBFDT	1
> >>> +
> >>
> >> that's ok.
> >>
> >>> +/* Initial Memory map for Linux */
> >>> +#define CONFIG_SYS_BOOTMAPSZ	(8 << 20)
> >>
> >> Why is this value setup to 0x800000?
> >
> > First of all the flat device tree handling depends on
> > CONFIG_SYS_BOOTMAPSZ. So I have to define a valid value. As far as I
> > understand the initial memory map for Linux there have to be space for
> > the kernel (text+data), fdt_blob and kernel parameters (at least). I'm
> > unsure with an embedded init ramdisk as produced by simpleImage. The
> > simpleImage can quickly grow up over 6MB with embedded initrd.
> >
> > So I borrowed this configuration (8MB) from most other configurations
> > (mainly PowerPC).
>
> ok. Microblaze handles FDT blob in different way. Kernel just need to
> know where it is and then is copied to another place.
> We don't have any limits like 8MB or so that's why I prefer to setup
> ~0UL value or something like that.
>
> Can you please test it?
>
> Thanks,
> Michal

-- 
Viele Gr??e,
Stephan Linz
______________________________________________________________________________
OpenDCC: http://www.li-pro.net/opendcc.phtml
PC/M: http://www.li-pro.net/pcm.phtml
CDK4AVR: http://cdk4avr.sourceforge.net/
CDK4NIOS: http://cdk4nios.sourceforge.net/
CDK4MSP: http://cdk4msp.sourceforge.net/
CPM4L: http://download.opensuse.org/repositories/home:/rexut:/CPM4L

  reply	other threads:[~2010-06-24  6:15 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-06-21 20:58 [U-Boot] [PATCH 1/4] microblaze: generic: adding DHCP support Stephan Linz
2010-06-21 20:58 ` [U-Boot] [PATCH 2/4] microblaze: generic: rename MTD partition set to 'flash-0' Stephan Linz
2010-06-21 20:58   ` [U-Boot] [PATCH 3/4] microblaze: enable LMB support Stephan Linz
2010-06-21 20:58     ` [U-Boot] [PATCH 4/4] microblaze: generic: enable FDT support Stephan Linz
2010-06-22  7:25       ` Michal Simek
2010-06-22  9:30         ` Stephan Linz
2010-06-23  6:27           ` Michal Simek
2010-06-24  6:15             ` Stephan Linz [this message]
2010-06-22  7:24     ` [U-Boot] [PATCH 3/4] microblaze: enable LMB support Michal Simek
2010-06-22  9:46       ` Stephan Linz
2010-06-23  6:31         ` Michal Simek
2010-06-25 14:37           ` Stephan Linz
2010-06-22  7:26   ` [U-Boot] [PATCH 2/4] microblaze: generic: rename MTD partition set to 'flash-0' Michal Simek
2010-06-21 21:21 ` [U-Boot] [PATCH 1/4] microblaze: generic: adding DHCP support Mike Frysinger
2010-06-21 21:55   ` Stephan Linz
2010-06-22  7:27 ` 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=201006240815.02843.linz@li-pro.net \
    --to=linz@li-pro.net \
    --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