From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jerry Van Baren Date: Thu, 14 Jul 2011 09:10:13 -0400 Subject: [U-Boot] [uboot PATCH v2] Add uboot "fdt_high" enviroment variable In-Reply-To: <20110709204019.593.76357.stgit@dave-Dell-System-XPS-L502X> References: <20110709204019.593.76357.stgit@dave-Dell-System-XPS-L502X> Message-ID: <4E1EEAB5.5020907@gmail.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 > of the fdt into highmem. The variable behaves similarly to the existing "initrd_high" > variable. > > Signed-off-by: David A. Long > --- > README | 9 ++++++++ > common/image.c | 60 ++++++++++++++++++++++++++++++++++++++++++++++---------- > 2 files changed, 58 insertions(+), 11 deletions(-) > > diff --git a/README b/README > index 8bb9c8d..5b95246 100644 > --- a/README > +++ b/README > @@ -3281,6 +3281,15 @@ List of environment variables (most likely not complete): [snip] > diff --git a/common/image.c b/common/image.c > index e542a57..7853de0 100644 > --- a/common/image.c > +++ b/common/image.c > @@ -1234,8 +1234,10 @@ int boot_relocate_fdt (struct lmb *lmb, char **of_flat_tree, ulong *of_size) > { > void *fdt_blob = *of_flat_tree; > void *of_start = 0; > + char *fdt_high; > ulong of_len = 0; > int err; > + int disable_relocation=0; Need spaces around the "=" I will add the spaces before applying the patch unless you send an updated patch. Thanks, gvb