Re: [U-Boot] [u-boot PATCH] arm, lib/bootm.c: Exclude HIGHMEM from being used by u-boot

2011-08-20 Thread David Long
On Sat, 2011-08-20 at 08:01 +0200, Albert ARIBAUD wrote: > What exactly prevents ARM Linux from booting when FDT or initrd are > above 768MB? Can this limitation not be lifter on the Linux side? I don't think it reasonably can be. I have to start by saying that I am in no way a Linux VM expe

[U-Boot] [u-boot PATCH] arm, lib/bootm.c: Exclude HIGHMEM from being used by u-boot

2011-08-19 Thread David Long
Reserve any memory above 768MB to prevent u-boot from relocating fdt or initrd data into memory that Linux cannot reference during early boot. Code taken (with modifications) from the powerpc bootm.c. Signed-off-by: David A. Long --- arch/arm/lib/bootm.c | 24 +++- 1 files

Re: [U-Boot] [PATCH] Fix typo from 'mb_alloc' -> 'lmb_alloc'

2011-07-19 Thread David Long
On Tue, 2011-07-19 at 10:40 +0200, Wolfgang Denk wrote: > David, Jerry: I'm disappointed that obviously neither of you bothered > to compile-test this stuff. Please be more careful! > Sorry, looks like I have to take ownership of that mistake. I must have introduced that when I fixed the la

Re: [U-Boot] [uboot PATCH v2] Add uboot "fdt_high" enviroment variable

2011-07-14 Thread David Long
On Thu, 2011-07-14 at 20:53 -0600, Grant Likely wrote: >You should have everything you need to fix it. If > CONFIG_SYS_BOOTMAPSZ is defined, then U-Boot will not use memory > larger that that for the dtb or atags. > > Right now CONFIG_SYS_BOOTMAPSZ is not set by default, but we could > default i

Re: [U-Boot] [uboot PATCH v2] Add uboot "fdt_high" enviroment variable

2011-07-14 Thread David Long
On Thu, 2011-07-14 at 15:21 -0500, Scott Wood wrote: > You have memory below where the kernel is loaded? Our boot script loads the kernel 2MB into physical RAM. It loads the initrd and fdt from the same NAND flash file system into RAM below that. When we boot without specifying an FDT, u-boot

Re: [U-Boot] [uboot PATCH v2] Add uboot "fdt_high" enviroment variable

2011-07-14 Thread David Long
On Thu, 2011-07-14 at 14:43 -0500, Scott Wood wrote: > You need to use lmb_reserve() to exclude any memory regions that are not > suitable for boot images -- see powerpc's arch_lmb_reserve() and > get_effective_memsize()/CONFIG_SYS_LINUX_LOWMEM_MAX_SIZE. If one excludes HIGHMEM from the area u-

Re: [U-Boot] [uboot PATCH v2] Add uboot "fdt_high" enviroment variable

2011-07-14 Thread David Long
On Fri, 2011-07-15 at 03:50 +0900, Grant Likely wrote: > Regardless of this patch, the pandaboard uboot still needs to be > fixed. Setting an fdt_high variable is useful for debug, but it is not > a fix. > Then someone needs to own the issue of stopping the current u-boot default behavior of

Re: [U-Boot] [uboot PATCH v2] Add uboot "fdt_high" enviroment variable

2011-07-14 Thread David Long
On Thu, 2011-07-14 at 09:10 -0400, Jerry Van Baren wrote: > Hi Dave, > > This looks reasonable, with one minor nit... > > Need spaces around the "=" > > I will add the spaces before applying the patch unless you send an > updated patch. > OK, thanks much. Someone else recently pointed t