On Thursday, July 14, 2011, David Long dave.long@linaro.org wrote:
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 that out to me too. I can generate another update if you wish, but I'll let you handle that unless you say otherwise.
FYI: In case it wasn't clear, this all came about because: 1) the Pandaboard has 1GB of RAM; 2) the presence of an fdt causes u-boot to relocate the fdt and ramdisk to the end of ram by default; 3) and the Linux kernel does not like having to access memory beyond about 3/4G (HIGHMEM) during early boot.
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.
g.
-dl