From: Bin Meng <bin.m...@windriver.com>

The FDT blob might not have sufficient space to hold a copy of
reserved memory node. Expand it before the copy.

Reported-by: Rick Chen <r...@andestech.com>
Signed-off-by: Bin Meng <bin.m...@windriver.com>
Reviewed-by: Atish Patra <atish.pa...@wdc.com>
---

(no changes since v3)

Changes in v3:
- Extend the FDT size by 1024 bytes

 arch/riscv/lib/fdt_fixup.c | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/arch/riscv/lib/fdt_fixup.c b/arch/riscv/lib/fdt_fixup.c
index 5f523f0..160ccca 100644
--- a/arch/riscv/lib/fdt_fixup.c
+++ b/arch/riscv/lib/fdt_fixup.c
@@ -41,6 +41,18 @@ int riscv_fdt_copy_resv_mem_node(const void *src, void *dst)
                return 0;
        }
 
+       /*
+        * Extend the FDT by the following estimated size:
+        *
+        * Each PMP memory region entry occupies 64 bytes.
+        * With 16 PMP memory regions we need 64 * 16 = 1024 bytes.
+        */
+       err = fdt_open_into(dst, dst, fdt_totalsize(dst) + 1024);
+       if (err < 0) {
+               printf("Device Tree can't be expanded to accommodate new node");
+               return err;
+       }
+
        fdt_for_each_subnode(node, src, offset) {
                name = fdt_get_name(src, node, NULL);
 
-- 
2.7.4

Reply via email to