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

2011-07-17 Thread Grant Likely
On Sat, Jul 16, 2011 at 12:06:33PM -0400, Jerry Van Baren wrote: > On 07/09/2011 04:40 PM, David A. Long wrote: > >From: David A. Long > > > >Add a new "fdt_high" enviroment variable. This can be used to control (or > >prevent) the > >relocation of the flattened device tree on boot. It can be used

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

2011-07-16 Thread Jerry Van Baren
On 07/09/2011 04:40 PM, David A. Long wrote: From: David A. Long Add a new "fdt_high" enviroment variable. This can be used to control (or prevent) the relocation of the flattened device tree on boot. It can be used to prevent relocation of the fdt into highmem. The variable behaves similarly

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

2011-07-15 Thread Scott Wood
On Thu, 14 Jul 2011 16:09:16 -0400 David Long wrote: > 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_SY

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

2011-07-15 Thread Scott Wood
On Thu, 14 Jul 2011 17:20:53 -0400 David Long wrote: > When we boot without specifying an FDT, u-boot does not relocate the > initrd. When we specify an FDT address in RAM, u-boot relocates both. > We do not need that relocation (in this case at least). Well, that does sound strange. I'd think

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 Jerry Van Baren
Hi Dave, This looks reasonable, with one minor nit... On 07/09/2011 04:40 PM, David A. Long wrote: From: David A. Long Add a new "fdt_high" enviroment variable. This can be used to control (or prevent) the relocation of the flattened device tree on boot. It can be used to prevent relocation

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

2011-07-14 Thread Scott Wood
On Thu, 14 Jul 2011 15:12:25 -0400 David Long wrote: > 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

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