diff mbox series

ls1043a: Remove "fdt_high" environment variable

Message ID 20200203093227.37591-1-wen.he_1@nxp.com
State New
Headers show
Series ls1043a: Remove "fdt_high" environment variable | expand

Commit Message

Wen He Feb. 3, 2020, 9:32 a.m. UTC
Remove "fdt_high" environment variable to use the bootm_size(or set
CONFIG_SYS_BOOTMAPSZ) to amount of memory available to safely contain
a kernel, device tree and initrd for relocation.

Signed-off-by: Wen He <wen.he_1 at nxp.com>
---
 include/configs/ls1043a_common.h | 1 -
 1 file changed, 1 deletion(-)

Comments

Tom Rini Feb. 4, 2020, 4:13 a.m. UTC | #1
On Mon, Feb 03, 2020 at 05:32:27PM +0800, Wen He wrote:

> Remove "fdt_high" environment variable to use the bootm_size(or set
> CONFIG_SYS_BOOTMAPSZ) to amount of memory available to safely contain
> a kernel, device tree and initrd for relocation.
> 
> Signed-off-by: Wen He <wen.he_1 at nxp.com>
> ---
>  include/configs/ls1043a_common.h | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/include/configs/ls1043a_common.h b/include/configs/ls1043a_common.h
> index 1670476..751117b 100644
> --- a/include/configs/ls1043a_common.h
> +++ b/include/configs/ls1043a_common.h
> @@ -234,7 +234,6 @@
>  /* Initial environment variables */
>  #define CONFIG_EXTRA_ENV_SETTINGS		\
>  	"hwconfig=fsl_ddr:bank_intlv=auto\0"	\
> -	"fdt_high=0xffffffffffffffff\0"		\
>  	"initrd_high=0xffffffffffffffff\0"	\
>  	"fdt_addr=0x64f00000\0"		 	\
>  	"kernel_addr=0x61000000\0"		\

I don't see either bootm_size in context here nor CONFIG_SYS_BOOTMAPSZ,
does this depend on some other patch?  Thanks!
Priyanka Jain (OSS) Feb. 26, 2020, 4:36 a.m. UTC | #2
>-----Original Message-----
>From: U-Boot <u-boot-bounces at lists.denx.de> On Behalf Of Tom Rini
>Sent: Tuesday, February 4, 2020 9:44 AM
>To: Wen He <wen.he_1 at nxp.com>
>Cc: u-boot at lists.denx.de; u-boot at linux.freescale.net
>Subject: Re: [PATCH] ls1043a: Remove "fdt_high" environment variable
>
>On Mon, Feb 03, 2020 at 05:32:27PM +0800, Wen He wrote:
>
>> Remove "fdt_high" environment variable to use the bootm_size(or set
>> CONFIG_SYS_BOOTMAPSZ) to amount of memory available to safely contain
>> a kernel, device tree and initrd for relocation.
>>
>> Signed-off-by: Wen He <wen.he_1 at nxp.com>
>> ---
>>  include/configs/ls1043a_common.h | 1 -
>>  1 file changed, 1 deletion(-)
>>
>> diff --git a/include/configs/ls1043a_common.h
>> b/include/configs/ls1043a_common.h
>> index 1670476..751117b 100644
>> --- a/include/configs/ls1043a_common.h
>> +++ b/include/configs/ls1043a_common.h
>> @@ -234,7 +234,6 @@
>>  /* Initial environment variables */
>>  #define CONFIG_EXTRA_ENV_SETTINGS		\
>>  	"hwconfig=fsl_ddr:bank_intlv=auto\0"	\
>> -	"fdt_high=0xffffffffffffffff\0"		\
>>  	"initrd_high=0xffffffffffffffff\0"	\
>>  	"fdt_addr=0x64f00000\0"		 	\
>>  	"kernel_addr=0x61000000\0"		\
>
>I don't see either bootm_size in context here nor CONFIG_SYS_BOOTMAPSZ,
>does this depend on some other patch?  Thanks!
>
>--
>Tom
Wen,
Kindly help to respond to Tom's query
Thanks
Priyanka
diff mbox series

Patch

diff --git a/include/configs/ls1043a_common.h b/include/configs/ls1043a_common.h
index 1670476..751117b 100644
--- a/include/configs/ls1043a_common.h
+++ b/include/configs/ls1043a_common.h
@@ -234,7 +234,6 @@ 
 /* Initial environment variables */
 #define CONFIG_EXTRA_ENV_SETTINGS		\
 	"hwconfig=fsl_ddr:bank_intlv=auto\0"	\
-	"fdt_high=0xffffffffffffffff\0"		\
 	"initrd_high=0xffffffffffffffff\0"	\
 	"fdt_addr=0x64f00000\0"		 	\
 	"kernel_addr=0x61000000\0"		\