Hi Dave,

This looks reasonable, with one minor nit...

On 07/09/2011 04:40 PM, David A. Long wrote:
From: David A. Long<dave.l...@linaro.org>

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<dave.l...@linaro.org>
---
  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

_______________________________________________
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev

Reply via email to