[U-Boot] [PATCH 1/3] udoo_neo: Add fdt_addr_r environment variable

According to doc/README.distro: "fdt_addr_r: Mandatory. The location in RAM where the DTB will be loaded or copied to when processing the fdtdir/devicetreedir or fdt/devicetree options in extlinux.conf."
So add the fdt_addr_r environment variable.
Suggested-by: Andreas Färber afaerber@suse.de Signed-off-by: Breno Lima breno.lima@nxp.com --- include/configs/udoo_neo.h | 1 + 1 file changed, 1 insertion(+)
diff --git a/include/configs/udoo_neo.h b/include/configs/udoo_neo.h index 915d0f0..f1e1c93 100644 --- a/include/configs/udoo_neo.h +++ b/include/configs/udoo_neo.h @@ -36,6 +36,7 @@ "initrd_high=0xffffffff\0" \ "fdtfile=undefined\0" \ "fdt_addr=0x83000000\0" \ + "fdt_addr_r=0x83000000\0" \ "ip_dyn=yes\0" \ "mmcdev=0\0" \ "mmcrootfstype=ext4\0" \

Remove the trailing semicolon and space. It's not necessary to have it on the last condition.
Suggested-by: Andreas Färber afaerber@suse.de Signed-off-by: Breno Lima breno.lima@nxp.com --- include/configs/udoo_neo.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/configs/udoo_neo.h b/include/configs/udoo_neo.h index f1e1c93..a7d1eac 100644 --- a/include/configs/udoo_neo.h +++ b/include/configs/udoo_neo.h @@ -50,7 +50,7 @@ "if test $board_name = EXTENDED; then " \ "setenv fdtfile imx6sx-udoo-neo-extended.dtb; fi; " \ "if test $fdtfile = UNDEFINED; then " \ - "echo WARNING: Could not determine dtb to use; fi; \0" \ + "echo WARNING: Could not determine dtb to use; fi\0" \ "kernel_addr_r=" __stringify(CONFIG_LOADADDR) "\0" \ "pxefile_addr_r=" __stringify(CONFIG_LOADADDR) "\0" \ "ramdisk_addr_r=0x83000000\0" \

On 10/01/2017 19:32, Breno Lima wrote:
Remove the trailing semicolon and space. It's not necessary to have it on the last condition.
Suggested-by: Andreas Färber afaerber@suse.de Signed-off-by: Breno Lima breno.lima@nxp.com
include/configs/udoo_neo.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/configs/udoo_neo.h b/include/configs/udoo_neo.h index f1e1c93..a7d1eac 100644 --- a/include/configs/udoo_neo.h +++ b/include/configs/udoo_neo.h @@ -50,7 +50,7 @@ "if test $board_name = EXTENDED; then " \ "setenv fdtfile imx6sx-udoo-neo-extended.dtb; fi; " \ "if test $fdtfile = UNDEFINED; then " \
"echo WARNING: Could not determine dtb to use; fi; \0" \
"kernel_addr_r=" __stringify(CONFIG_LOADADDR) "\0" \ "pxefile_addr_r=" __stringify(CONFIG_LOADADDR) "\0" \ "ramdisk_addr_r=0x83000000\0" \"echo WARNING: Could not determine dtb to use; fi\0" \
Applied to u-boot-imx, thanks !
Best regards, Stefano Babic

Remove unused ramdiskaddr environment variable.
Suggested-by: Andreas Färber afaerber@suse.de Signed-off-by: Breno Lima breno.lima@nxp.com --- include/configs/udoo_neo.h | 1 - 1 file changed, 1 deletion(-)
diff --git a/include/configs/udoo_neo.h b/include/configs/udoo_neo.h index a7d1eac..8d67ff3 100644 --- a/include/configs/udoo_neo.h +++ b/include/configs/udoo_neo.h @@ -54,7 +54,6 @@ "kernel_addr_r=" __stringify(CONFIG_LOADADDR) "\0" \ "pxefile_addr_r=" __stringify(CONFIG_LOADADDR) "\0" \ "ramdisk_addr_r=0x83000000\0" \ - "ramdiskaddr=0x83000000\0" \ "scriptaddr=" __stringify(CONFIG_LOADADDR) "\0" \ BOOTENV

On 10/01/2017 19:32, Breno Lima wrote:
Remove unused ramdiskaddr environment variable.
Suggested-by: Andreas Färber afaerber@suse.de Signed-off-by: Breno Lima breno.lima@nxp.com
include/configs/udoo_neo.h | 1 - 1 file changed, 1 deletion(-)
diff --git a/include/configs/udoo_neo.h b/include/configs/udoo_neo.h index a7d1eac..8d67ff3 100644 --- a/include/configs/udoo_neo.h +++ b/include/configs/udoo_neo.h @@ -54,7 +54,6 @@ "kernel_addr_r=" __stringify(CONFIG_LOADADDR) "\0" \ "pxefile_addr_r=" __stringify(CONFIG_LOADADDR) "\0" \ "ramdisk_addr_r=0x83000000\0" \
- "ramdiskaddr=0x83000000\0" \ "scriptaddr=" __stringify(CONFIG_LOADADDR) "\0" \ BOOTENV
Applied to u-boot-imx, thanks !
Best regards, Stefano Babic

On 10/01/2017 19:32, Breno Lima wrote:
According to doc/README.distro: "fdt_addr_r: Mandatory. The location in RAM where the DTB will be loaded or copied to when processing the fdtdir/devicetreedir or fdt/devicetree options in extlinux.conf."
So add the fdt_addr_r environment variable.
Suggested-by: Andreas Färber afaerber@suse.de Signed-off-by: Breno Lima breno.lima@nxp.com
include/configs/udoo_neo.h | 1 + 1 file changed, 1 insertion(+)
diff --git a/include/configs/udoo_neo.h b/include/configs/udoo_neo.h index 915d0f0..f1e1c93 100644 --- a/include/configs/udoo_neo.h +++ b/include/configs/udoo_neo.h @@ -36,6 +36,7 @@ "initrd_high=0xffffffff\0" \ "fdtfile=undefined\0" \ "fdt_addr=0x83000000\0" \
- "fdt_addr_r=0x83000000\0" \ "ip_dyn=yes\0" \ "mmcdev=0\0" \ "mmcrootfstype=ext4\0" \
Applied to u-boot-imx, thanks !
Best regards, Stefano Babic
participants (2)
-
Breno Lima
-
Stefano Babic