[PATCH v2 00/13] bugfix/improvements for IMX-based modules
1. Enable relocation of fdt and initrd for all Toradex modules; 2. Add nfsboot wrapper to env for Verdin i.MX8M Mini 3. i.MX8M Mini EVK/Verdin: adjust dram size in case bl32 is used 4. Set gpr1 ENET_CLK_SEL for Colibri iMX6S/DL, which fixes PHY negotiation issues. 5. Colibri i.MX8QXP/QM pd_dma nodes should be accessible before U-Boot relocation.
v2: - Applied R-b tags - Put all pending patches into one patch-series to easier track stuff
Igor Opaniuk (13): ARM: dts: imx8qm-apalis: replace dm-spl with dm-pre-proper ARM: dts: imx8qxp-colibri: replace dm-spl with dm-pre-proper ARM: dts: imx8qm-apalis: dm-pre-proper for pd_dma nodes ARM: dts: imx8qxp-colibri: dm-pre-proper for pd_dma nodes colibri_imx6: set gpr1 ENET_CLK_SEL verdin-imx8mm: adjust dram size in case bl32 is used imx8mm_evk: adjust dram size in case bl32 is used verdin-imx8mm: add nfsboot wrapper to env colibri_vf: enable relocation of fdt and initrd colibri_imx7: enable relocation of fdt and initrd apalis_imx6: enable relocation of fdt and initrd colibri-imx6ull: enable relocation of fdt and initrd colibri_imx6: enable relocation of fdt and initrd
arch/arm/dts/fsl-imx8qm-apalis-u-boot.dtsi | 70 +++++++++++--------- arch/arm/dts/fsl-imx8qxp-colibri-u-boot.dtsi | 68 +++++++++++-------- board/freescale/imx8mm_evk/imx8mm_evk.c | 6 +- board/toradex/colibri_imx6/colibri_imx6.c | 3 + board/toradex/verdin-imx8mm/verdin-imx8mm.c | 7 +- include/configs/apalis_imx6.h | 2 - include/configs/colibri-imx6ull.h | 2 - include/configs/colibri_imx6.h | 2 - include/configs/colibri_imx7.h | 2 - include/configs/colibri_vf.h | 2 - include/configs/verdin-imx8mm.h | 6 ++ 11 files changed, 99 insertions(+), 71 deletions(-)
From: Igor Opaniuk igor.opaniuk@toradex.com
For non-SPL/TPL setups dm-spl, dm-tpl, dm-pre-proper, dm-pre-reloc are handled equally, forcing the nodes with these properties to be accessible and device being probed before pre-relocation of U-Boot proper (drivers/core/util.c):
bool ofnode_pre_reloc(ofnode node) { /* for SPL and TPL the remaining nodes after the fdtgrep 1st pass * had property dm-pre-reloc or u-boot,dm-spl/tpl. * They are removed in final dtb (fdtgrep 2nd pass) */ return true; if (ofnode_read_bool(node, "u-boot,dm-pre-reloc")) return true; if (ofnode_read_bool(node, "u-boot,dm-pre-proper")) return true;
/* * In regular builds individual spl and tpl handling both * count as handled pre-relocation for later second init. */ if (ofnode_read_bool(node, "u-boot,dm-spl") || ofnode_read_bool(node, "u-boot,dm-tpl")) return true;
return false; }
Howewer, to avoid confusion in future, replace dm-spl `%s/dm-spl/dm-pre-proper/g` properties to dm-pre-proper to explicitly state that they are handled during pre-relocation stage of U-Boot proper.
Signed-off-by: Igor Opaniuk igor.opaniuk@toradex.com Reviewed-by: Oleksandr Suvorov oleksandr.suvorov@toradex.com ---
arch/arm/dts/fsl-imx8qm-apalis-u-boot.dtsi | 62 +++++++++++----------- 1 file changed, 31 insertions(+), 31 deletions(-)
diff --git a/arch/arm/dts/fsl-imx8qm-apalis-u-boot.dtsi b/arch/arm/dts/fsl-imx8qm-apalis-u-boot.dtsi index 7b1a9550e4..6cdf58c8ad 100644 --- a/arch/arm/dts/fsl-imx8qm-apalis-u-boot.dtsi +++ b/arch/arm/dts/fsl-imx8qm-apalis-u-boot.dtsi @@ -4,125 +4,125 @@ */
&mu { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&clk { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&iomuxc { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&pd_lsio { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&pd_lsio_gpio0 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&pd_lsio_gpio1 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&pd_lsio_gpio2 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&pd_lsio_gpio3 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&pd_lsio_gpio4 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&pd_lsio_gpio5 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&pd_lsio_gpio6 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&pd_lsio_gpio7 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&pd_conn { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&pd_conn_sdch0 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&pd_conn_sdch1 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&pd_conn_sdch2 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&gpio0 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&gpio1 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&gpio2 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&gpio3 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&gpio4 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&gpio5 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&gpio6 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&gpio7 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&lpuart0 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&lpuart1 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&lpuart2 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&lpuart3 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&usdhc1 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&usdhc2 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&usdhc3 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
From: Igor Opaniuk igor.opaniuk@toradex.com
For non-SPL/TPL setups dm-spl, dm-tpl, dm-pre-proper, dm-pre-reloc are handled equally, forcing the nodes with these properties to be accessible and device being probed before pre-relocation of U-Boot proper (drivers/core/util.c):
bool ofnode_pre_reloc(ofnode node) { /* for SPL and TPL the remaining nodes after the fdtgrep 1st pass * had property dm-pre-reloc or u-boot,dm-spl/tpl. * They are removed in final dtb (fdtgrep 2nd pass) */ return true; if (ofnode_read_bool(node, "u-boot,dm-pre-reloc")) return true; if (ofnode_read_bool(node, "u-boot,dm-pre-proper")) return true;
/* * In regular builds individual spl and tpl handling both * count as handled pre-relocation for later second init. */ if (ofnode_read_bool(node, "u-boot,dm-spl") || ofnode_read_bool(node, "u-boot,dm-tpl")) return true;
return false; }
Howewer, to avoid confusion in future, replace dm-spl `%s/dm-spl/dm-pre-proper/g` properties to dm-pre-proper to explicitly state that they are handled during pre-relocation stage of U-Boot proper.
Signed-off-by: Igor Opaniuk igor.opaniuk@toradex.com Reviewed-by: Oleksandr Suvorov oleksandr.suvorov@toradex.com ---
arch/arm/dts/fsl-imx8qxp-colibri-u-boot.dtsi | 56 ++++++++++---------- 1 file changed, 28 insertions(+), 28 deletions(-)
diff --git a/arch/arm/dts/fsl-imx8qxp-colibri-u-boot.dtsi b/arch/arm/dts/fsl-imx8qxp-colibri-u-boot.dtsi index 5b061f94ba..87de9b5653 100644 --- a/arch/arm/dts/fsl-imx8qxp-colibri-u-boot.dtsi +++ b/arch/arm/dts/fsl-imx8qxp-colibri-u-boot.dtsi @@ -5,113 +5,113 @@
&{/imx8qx-pm} {
- u-boot,dm-spl; + u-boot,dm-pre-proper; };
&mu { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&clk { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&iomuxc { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&pd_lsio { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&pd_lsio_gpio0 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&pd_lsio_gpio1 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&pd_lsio_gpio2 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&pd_lsio_gpio3 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&pd_lsio_gpio4 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&pd_lsio_gpio5 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&pd_lsio_gpio6 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&pd_lsio_gpio7 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&pd_conn { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&pd_conn_sdch0 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&pd_conn_sdch1 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&pd_conn_sdch2 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&gpio0 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&gpio1 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&gpio2 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&gpio3 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&gpio4 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&gpio5 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&gpio6 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&gpio7 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&lpuart3 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&usdhc1 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
&usdhc2 { - u-boot,dm-spl; + u-boot,dm-pre-proper; };
From: Igor Opaniuk igor.opaniuk@toradex.com
pd_dma_* nodes should be accessible during pre-relocation stage of U-Boot proper for properly handling power domains.
This fixes the issue with permanent failing of invocation of power_domain_get_by_index() in the common code of DM power domain uclass (drivers/power/domain/power-domain-uclass.c).
Fixes: f0cc4eae9a ("core: device: use dev_power_domain_on") Signed-off-by: Igor Opaniuk igor.opaniuk@toradex.com Reviewed-by: Oleksandr Suvorov oleksandr.suvorov@toradex.com ---
arch/arm/dts/fsl-imx8qm-apalis-u-boot.dtsi | 8 ++++++++ 1 file changed, 8 insertions(+)
diff --git a/arch/arm/dts/fsl-imx8qm-apalis-u-boot.dtsi b/arch/arm/dts/fsl-imx8qm-apalis-u-boot.dtsi index 6cdf58c8ad..956d724979 100644 --- a/arch/arm/dts/fsl-imx8qm-apalis-u-boot.dtsi +++ b/arch/arm/dts/fsl-imx8qm-apalis-u-boot.dtsi @@ -51,6 +51,14 @@ u-boot,dm-pre-proper; };
+&pd_dma { + u-boot,dm-pre-proper; +}; + +&pd_dma_lpuart1 { + u-boot,dm-pre-proper; +}; + &pd_conn { u-boot,dm-pre-proper; };
From: Igor Opaniuk igor.opaniuk@toradex.com
pd_dma_* nodes should be accessible during pre-relocation stage of U-Boot proper for properly handling power domains.
This fixes the issue with permanent failing of invocation of power_domain_get_by_index() in the common code of DM power domain uclass (drivers/power/domain/power-domain-uclass.c).
Fixes: f0cc4eae9a ("core: device: use dev_power_domain_on") Signed-off-by: Igor Opaniuk igor.opaniuk@toradex.com Reviewed-by: Oleksandr Suvorov oleksandr.suvorov@toradex.com ---
arch/arm/dts/fsl-imx8qxp-colibri-u-boot.dtsi | 12 ++++++++++++ 1 file changed, 12 insertions(+)
diff --git a/arch/arm/dts/fsl-imx8qxp-colibri-u-boot.dtsi b/arch/arm/dts/fsl-imx8qxp-colibri-u-boot.dtsi index 87de9b5653..322429a98a 100644 --- a/arch/arm/dts/fsl-imx8qxp-colibri-u-boot.dtsi +++ b/arch/arm/dts/fsl-imx8qxp-colibri-u-boot.dtsi @@ -56,6 +56,18 @@ u-boot,dm-pre-proper; };
+&pd_dma { + u-boot,dm-pre-proper; +}; + +&pd_dma_lpuart0 { + u-boot,dm-pre-proper; +}; + +&pd_dma_lpuart3 { + u-boot,dm-pre-proper; +}; + &pd_conn { u-boot,dm-pre-proper; };
From: Igor Opaniuk igor.opaniuk@toradex.com
This fixes the issue when PHY auto negotiation never completes.
Fixes: 431cd76dd8("colibri_imx6: migrate to DM_ETH") Signed-off-by: Igor Opaniuk igor.opaniuk@toradex.com Reviewed-by: Oleksandr Suvorov oleksandr.suvorov@toradex.com ---
board/toradex/colibri_imx6/colibri_imx6.c | 3 +++ 1 file changed, 3 insertions(+)
diff --git a/board/toradex/colibri_imx6/colibri_imx6.c b/board/toradex/colibri_imx6/colibri_imx6.c index f04b749311..d11401ecab 100644 --- a/board/toradex/colibri_imx6/colibri_imx6.c +++ b/board/toradex/colibri_imx6/colibri_imx6.c @@ -354,12 +354,15 @@ int board_phy_config(struct phy_device *phydev) int setup_fec(void) { int ret; + struct iomuxc *iomuxc_regs = (struct iomuxc *)IOMUXC_BASE_ADDR;
/* provide the PHY clock from the i.MX 6 */ ret = enable_fec_anatop_clock(0, ENET_50MHZ); if (ret) return ret;
+ setbits_le32(&iomuxc_regs->gpr[1], IOMUXC_GPR1_ENET_CLK_SEL_MASK); + return 0; }
From: Igor Opaniuk igor.opaniuk@toradex.com
Adjust DRAM size in case BL32 secure payload is loaded (OP-TEE/Trusty), so during MMU initialization U-Boot won't touch this mem area.
BL32 is loaded to the end of DRAM, bl32 payload size is read from rom_pointer[1]. This relates to the issue described in 59efa6b52b("imx8m: Fix MMU table issue for OPTEE memory").
Signed-off-by: Igor Opaniuk igor.opaniuk@toradex.com ---
board/toradex/verdin-imx8mm/verdin-imx8mm.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/board/toradex/verdin-imx8mm/verdin-imx8mm.c b/board/toradex/verdin-imx8mm/verdin-imx8mm.c index 16b9fa1ec1..cb9b4e3b0a 100644 --- a/board/toradex/verdin-imx8mm/verdin-imx8mm.c +++ b/board/toradex/verdin-imx8mm/verdin-imx8mm.c @@ -5,6 +5,7 @@
#include <common.h> #include <asm/arch/clock.h> +#include <asm/arch/sys_proto.h> #include <asm/io.h> #include <miiphy.h> #include <netdev.h> @@ -13,7 +14,11 @@ DECLARE_GLOBAL_DATA_PTR;
int dram_init(void) { - gd->ram_size = get_ram_size((void *)PHYS_SDRAM, PHYS_SDRAM_SIZE); + /* rom_pointer[1] contains the size of TEE occupies */ + if (rom_pointer[1]) + gd->ram_size = PHYS_SDRAM_SIZE - rom_pointer[1]; + else + gd->ram_size = PHYS_SDRAM_SIZE;
return 0; }
From: Igor Opaniuk igor.opaniuk@toradex.com Adjust DRAM size in case BL32 secure payload is loaded (OP-TEE/Trusty), so during MMU initialization U-Boot won't touch this mem area. BL32 is loaded to the end of DRAM, bl32 payload size is read from rom_pointer[1]. This relates to the issue described in 59efa6b52b("imx8m: Fix MMU table issue for OPTEE memory"). Signed-off-by: Igor Opaniuk igor.opaniuk@toradex.com
Applied to u-boot-imx, master, thanks !
Best regards, Stefano Babic
From: Igor Opaniuk igor.opaniuk@toradex.com
Adjust DRAM size in case BL32 secure payload is loaded (OP-TEE/Trusty), so during MMU initialization U-Boot won't touch this mem area.
BL32 is loaded to the end of DRAM, bl32 payload size is read from rom_pointer[1]. This relates to the issue described in 59efa6b52b("imx8m: Fix MMU table issue for OPTEE memory").
Signed-off-by: Igor Opaniuk igor.opaniuk@toradex.com ---
board/freescale/imx8mm_evk/imx8mm_evk.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/board/freescale/imx8mm_evk/imx8mm_evk.c b/board/freescale/imx8mm_evk/imx8mm_evk.c index c5fd940eeb..53ebb949df 100644 --- a/board/freescale/imx8mm_evk/imx8mm_evk.c +++ b/board/freescale/imx8mm_evk/imx8mm_evk.c @@ -15,7 +15,11 @@ DECLARE_GLOBAL_DATA_PTR;
int dram_init(void) { - gd->ram_size = PHYS_SDRAM_SIZE; + /* rom_pointer[1] contains the size of TEE occupies */ + if (rom_pointer[1]) + gd->ram_size = PHYS_SDRAM_SIZE - rom_pointer[1]; + else + gd->ram_size = PHYS_SDRAM_SIZE;
return 0; }
On Fri, Mar 27, 2020 at 12:29 PM Igor Opaniuk igor.opaniuk@gmail.com wrote:
From: Igor Opaniuk igor.opaniuk@toradex.com
Adjust DRAM size in case BL32 secure payload is loaded (OP-TEE/Trusty), so during MMU initialization U-Boot won't touch this mem area.
BL32 is loaded to the end of DRAM, bl32 payload size is read from rom_pointer[1]. This relates to the issue described in 59efa6b52b("imx8m: Fix MMU table issue for OPTEE memory").
Signed-off-by: Igor Opaniuk igor.opaniuk@toradex.com
board/freescale/imx8mm_evk/imx8mm_evk.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/board/freescale/imx8mm_evk/imx8mm_evk.c b/board/freescale/imx8mm_evk/imx8mm_evk.c index c5fd940eeb..53ebb949df 100644 --- a/board/freescale/imx8mm_evk/imx8mm_evk.c +++ b/board/freescale/imx8mm_evk/imx8mm_evk.c @@ -15,7 +15,11 @@ DECLARE_GLOBAL_DATA_PTR;
int dram_init(void) {
gd->ram_size = PHYS_SDRAM_SIZE;
/* rom_pointer[1] contains the size of TEE occupies */if (rom_pointer[1])gd->ram_size = PHYS_SDRAM_SIZE - rom_pointer[1];elsegd->ram_size = PHYS_SDRAM_SIZE;
Is not this code equivalent to just one line? gd->ram_size = PHYS_SDRAM_SIZE - rom_pointer[1];
If rom_pointer[1] == 0, so the gd->ram_size gets the PHYS_SDRAM_SIZE value.
return 0;}
2.17.1
From: Igor Opaniuk igor.opaniuk@toradex.com Adjust DRAM size in case BL32 secure payload is loaded (OP-TEE/Trusty), so during MMU initialization U-Boot won't touch this mem area. BL32 is loaded to the end of DRAM, bl32 payload size is read from rom_pointer[1]. This relates to the issue described in 59efa6b52b("imx8m: Fix MMU table issue for OPTEE memory"). Signed-off-by: Igor Opaniuk igor.opaniuk@toradex.com
Applied to u-boot-imx, master, thanks !
Best regards, Stefano Babic
From: Igor Opaniuk igor.opaniuk@toradex.com
Add nfsboot wrapper to env to boot Linux kernel from TFTP/NFS.
Signed-off-by: Igor Opaniuk igor.opaniuk@toradex.com ---
include/configs/verdin-imx8mm.h | 6 ++++++ 1 file changed, 6 insertions(+)
diff --git a/include/configs/verdin-imx8mm.h b/include/configs/verdin-imx8mm.h index dc0a2efec6..82bff3608c 100644 --- a/include/configs/verdin-imx8mm.h +++ b/include/configs/verdin-imx8mm.h @@ -66,6 +66,12 @@ "initrd_addr=0x43800000\0" \ "initrd_high=0xffffffffffffffff\0" \ "kernel_image=Image\0" \ + "netargs=setenv bootargs console=${console},${baudrate} " \ + "root=/dev/nfs ip=dhcp nfsroot=${serverip}:${nfsroot},v3,tcp" \ + "\0" \ + "nfsboot=run netargs; dhcp ${loadaddr} ${kernel_image}; " \ + "tftp ${fdt_addr} verdin/${fdtfile}; " \ + "booti ${loadaddr} - ${fdt_addr}\0" \ "setup=setenv setupargs console=${console},${baudrate} " \ "console=tty1 consoleblank=0 earlycon\0" \ "update_uboot=askenv confirm Did you load flash.bin (y/N)?; " \
On Fri, Mar 27, 2020 at 12:30 PM Igor Opaniuk igor.opaniuk@gmail.com wrote:
From: Igor Opaniuk igor.opaniuk@toradex.com
Add nfsboot wrapper to env to boot Linux kernel from TFTP/NFS.
Signed-off-by: Igor Opaniuk igor.opaniuk@toradex.com
Reviewed-by: Oleksandr Suvorov oleksandr.suvorov@toradex.com
include/configs/verdin-imx8mm.h | 6 ++++++ 1 file changed, 6 insertions(+)
diff --git a/include/configs/verdin-imx8mm.h b/include/configs/verdin-imx8mm.h index dc0a2efec6..82bff3608c 100644 --- a/include/configs/verdin-imx8mm.h +++ b/include/configs/verdin-imx8mm.h @@ -66,6 +66,12 @@ "initrd_addr=0x43800000\0" \ "initrd_high=0xffffffffffffffff\0" \ "kernel_image=Image\0" \
"netargs=setenv bootargs console=${console},${baudrate} " \"root=/dev/nfs ip=dhcp nfsroot=${serverip}:${nfsroot},v3,tcp" \"\0" \"nfsboot=run netargs; dhcp ${loadaddr} ${kernel_image}; " \"tftp ${fdt_addr} verdin/${fdtfile}; " \"booti ${loadaddr} - ${fdt_addr}\0" \ "setup=setenv setupargs console=${console},${baudrate} " \ "console=tty1 consoleblank=0 earlycon\0" \ "update_uboot=askenv confirm Did you load flash.bin (y/N)?; " \-- 2.17.1
From: Igor Opaniuk igor.opaniuk@toradex.com Add nfsboot wrapper to env to boot Linux kernel from TFTP/NFS. Signed-off-by: Igor Opaniuk igor.opaniuk@toradex.com Reviewed-by: Oleksandr Suvorov oleksandr.suvorov@toradex.com
Applied to u-boot-imx, master, thanks !
Best regards, Stefano Babic
From: Igor Opaniuk igor.opaniuk@toradex.com
Remove 'fdt_high' and 'initrd_high' environment variables (set to 0xFFFFFFFF) from default environment which prevents relocation of FDT and initrd. Rely on 'bootm_size' value instead to safely relocate kernel, device tree and initrd.
Signed-off-by: Igor Opaniuk igor.opaniuk@toradex.com Reviewed-by: Oleksandr Suvorov oleksandr.suvorov@toradex.com ---
include/configs/colibri_vf.h | 2 -- 1 file changed, 2 deletions(-)
diff --git a/include/configs/colibri_vf.h b/include/configs/colibri_vf.h index 1478ea844e..b03ccaf094 100644 --- a/include/configs/colibri_vf.h +++ b/include/configs/colibri_vf.h @@ -51,8 +51,6 @@ #define MEM_LAYOUT_ENV_SETTINGS \ "bootm_size=0x10000000\0" \ "fdt_addr_r=0x82000000\0" \ - "fdt_high=0xffffffff\0" \ - "initrd_high=0xffffffff\0" \ "kernel_addr_r=0x81000000\0" \ "pxefile_addr_r=0x87100000\0" \ "ramdisk_addr_r=0x82100000\0" \
From: Igor Opaniuk igor.opaniuk@toradex.com Remove 'fdt_high' and 'initrd_high' environment variables (set to 0xFFFFFFFF) from default environment which prevents relocation of FDT and initrd. Rely on 'bootm_size' value instead to safely relocate kernel, device tree and initrd. Signed-off-by: Igor Opaniuk igor.opaniuk@toradex.com Reviewed-by: Oleksandr Suvorov oleksandr.suvorov@toradex.com
Applied to u-boot-imx, master, thanks !
Best regards, Stefano Babic
From: Igor Opaniuk igor.opaniuk@toradex.com
Remove 'fdt_high' and 'initrd_high' environment variables (set to 0xFFFFFFFF) from default environment which prevents relocation of FDT and initrd. Rely on 'bootm_size' value instead to safely relocate kernel, device tree and initrd.
Signed-off-by: Igor Opaniuk igor.opaniuk@toradex.com Reviewed-by: Oleksandr Suvorov oleksandr.suvorov@toradex.com ---
include/configs/colibri_imx7.h | 2 -- 1 file changed, 2 deletions(-)
diff --git a/include/configs/colibri_imx7.h b/include/configs/colibri_imx7.h index 603ea3a053..7c00f78ef1 100644 --- a/include/configs/colibri_imx7.h +++ b/include/configs/colibri_imx7.h @@ -108,8 +108,6 @@ #define MEM_LAYOUT_ENV_SETTINGS \ "bootm_size=0x10000000\0" \ "fdt_addr_r=0x82000000\0" \ - "fdt_high=0xffffffff\0" \ - "initrd_high=0xffffffff\0" \ "kernel_addr_r=0x81000000\0" \ "ramdisk_addr_r=0x82100000\0" \ "scriptaddr=0x82500000\0"
From: Igor Opaniuk igor.opaniuk@toradex.com Remove 'fdt_high' and 'initrd_high' environment variables (set to 0xFFFFFFFF) from default environment which prevents relocation of FDT and initrd. Rely on 'bootm_size' value instead to safely relocate kernel, device tree and initrd. Signed-off-by: Igor Opaniuk igor.opaniuk@toradex.com Reviewed-by: Oleksandr Suvorov oleksandr.suvorov@toradex.com
Applied to u-boot-imx, master, thanks !
Best regards, Stefano Babic
From: Igor Opaniuk igor.opaniuk@toradex.com
Remove 'fdt_high' and 'initrd_high' environment variables (set to 0xFFFFFFFF) from default environment which prevents relocation of FDT and initrd. Rely on 'bootm_size' value instead to safely relocate kernel, device tree and initrd.
Signed-off-by: Igor Opaniuk igor.opaniuk@toradex.com Reviewed-by: Oleksandr Suvorov oleksandr.suvorov@toradex.com ---
include/configs/apalis_imx6.h | 2 -- 1 file changed, 2 deletions(-)
diff --git a/include/configs/apalis_imx6.h b/include/configs/apalis_imx6.h index d2ff7e9534..fb0037444f 100644 --- a/include/configs/apalis_imx6.h +++ b/include/configs/apalis_imx6.h @@ -146,8 +146,6 @@ #define MEM_LAYOUT_ENV_SETTINGS \ "bootm_size=0x20000000\0" \ "fdt_addr_r=0x12100000\0" \ - "fdt_high=0xffffffff\0" \ - "initrd_high=0xffffffff\0" \ "kernel_addr_r=0x11000000\0" \ "pxefile_addr_r=0x17100000\0" \ "ramdisk_addr_r=0x12200000\0" \
From: Igor Opaniuk igor.opaniuk@toradex.com Remove 'fdt_high' and 'initrd_high' environment variables (set to 0xFFFFFFFF) from default environment which prevents relocation of FDT and initrd. Rely on 'bootm_size' value instead to safely relocate kernel, device tree and initrd. Signed-off-by: Igor Opaniuk igor.opaniuk@toradex.com Reviewed-by: Oleksandr Suvorov oleksandr.suvorov@toradex.com
Applied to u-boot-imx, master, thanks !
Best regards, Stefano Babic
From: Igor Opaniuk igor.opaniuk@toradex.com
Remove 'fdt_high' and 'initrd_high' environment variables (set to 0xFFFFFFFF) from default environment which prevents relocation of FDT and initrd. Rely on 'bootm_size' value instead to safely relocate kernel, device tree and initrd.
Signed-off-by: Igor Opaniuk igor.opaniuk@toradex.com Reviewed-by: Oleksandr Suvorov oleksandr.suvorov@toradex.com ---
include/configs/colibri-imx6ull.h | 2 -- 1 file changed, 2 deletions(-)
diff --git a/include/configs/colibri-imx6ull.h b/include/configs/colibri-imx6ull.h index ea5ba6bfce..2a76f576a8 100644 --- a/include/configs/colibri-imx6ull.h +++ b/include/configs/colibri-imx6ull.h @@ -40,8 +40,6 @@ #define MEM_LAYOUT_ENV_SETTINGS \ "bootm_size=0x10000000\0" \ "fdt_addr_r=0x82100000\0" \ - "fdt_high=0xffffffff\0" \ - "initrd_high=0xffffffff\0" \ "kernel_addr_r=0x81000000\0" \ "pxefile_addr_r=0x87100000\0" \ "ramdisk_addr_r=0x82200000\0" \
From: Igor Opaniuk igor.opaniuk@toradex.com Remove 'fdt_high' and 'initrd_high' environment variables (set to 0xFFFFFFFF) from default environment which prevents relocation of FDT and initrd. Rely on 'bootm_size' value instead to safely relocate kernel, device tree and initrd. Signed-off-by: Igor Opaniuk igor.opaniuk@toradex.com Reviewed-by: Oleksandr Suvorov oleksandr.suvorov@toradex.com
Applied to u-boot-imx, master, thanks !
Best regards, Stefano Babic
From: Igor Opaniuk igor.opaniuk@toradex.com
Remove 'fdt_high' and 'initrd_high' environment variables (set to 0xFFFFFFFF) from default environment which prevents relocation of FDT and initrd. Rely on 'bootm_size' value instead to safely relocate kernel, device tree and initrd.
Signed-off-by: Igor Opaniuk igor.opaniuk@toradex.com Reviewed-by: Oleksandr Suvorov oleksandr.suvorov@toradex.com ---
include/configs/colibri_imx6.h | 2 -- 1 file changed, 2 deletions(-)
diff --git a/include/configs/colibri_imx6.h b/include/configs/colibri_imx6.h index cbc7501bcc..4cdd3c53af 100644 --- a/include/configs/colibri_imx6.h +++ b/include/configs/colibri_imx6.h @@ -134,8 +134,6 @@ #define MEM_LAYOUT_ENV_SETTINGS \ "bootm_size=0x10000000\0" \ "fdt_addr_r=0x12100000\0" \ - "fdt_high=0xffffffff\0" \ - "initrd_high=0xffffffff\0" \ "kernel_addr_r=0x11000000\0" \ "pxefile_addr_r=0x17100000\0" \ "ramdisk_addr_r=0x12200000\0" \
From: Igor Opaniuk igor.opaniuk@toradex.com Remove 'fdt_high' and 'initrd_high' environment variables (set to 0xFFFFFFFF) from default environment which prevents relocation of FDT and initrd. Rely on 'bootm_size' value instead to safely relocate kernel, device tree and initrd. Signed-off-by: Igor Opaniuk igor.opaniuk@toradex.com Reviewed-by: Oleksandr Suvorov oleksandr.suvorov@toradex.com
Applied to u-boot-imx, master, thanks !
Best regards, Stefano Babic
participants (3)
-
Igor Opaniuk -
Oleksandr Suvorov -
sbabic＠denx.de