diff mbox

[U-Boot,09/22,v2] armv8/ls2085a: Increase the supported kernel size

Message ID 1432805055-1244-9-git-send-email-prabhakar@freescale.com
State Accepted
Delegated to: York Sun
Headers show

Commit Message

Prabhakar Kushwaha May 28, 2015, 9:24 a.m. UTC 
From: Bhupesh Sharma <bhupesh.sharma at freescale.com>

Increases the kernel size supported for LS2085A platforms:-
 - Update environment variables
 - Add ramdisk_size in bootargs env variable
 - Define  CONFIG_SYS_BOOTM_LEN to 64MB

Signed-off-by: Bhupesh Sharma <bhupesh.sharma at freescale.com>
Signed-off-by: Prabhakar Kushwaha <prabhakar at freescale.com>
---
Changes for v2: sending as it is for patch set

 include/configs/ls2085a_common.h | 9 ++++++---
 include/configs/ls2085aqds.h     | 2 +-
 include/configs/ls2085ardb.h     | 2 +-
 3 files changed, 8 insertions(+), 5 deletions(-)

Comments

York Sun May 28, 2015, 4 p.m. UTC | #1 
On 05/28/2015 02:24 AM, Prabhakar Kushwaha wrote:
> From: Bhupesh Sharma <bhupesh.sharma at freescale.com>
> 
> Increases the kernel size supported for LS2085A platforms:-
>  - Update environment variables
>  - Add ramdisk_size in bootargs env variable
>  - Define  CONFIG_SYS_BOOTM_LEN to 64MB
> 
> Signed-off-by: Bhupesh Sharma <bhupesh.sharma at freescale.com>
> Signed-off-by: Prabhakar Kushwaha <prabhakar at freescale.com>
> ---
> Changes for v2: sending as it is for patch set
> 
>  include/configs/ls2085a_common.h | 9 ++++++---
>  include/configs/ls2085aqds.h     | 2 +-
>  include/configs/ls2085ardb.h     | 2 +-
>  3 files changed, 8 insertions(+), 5 deletions(-)
> 
> diff --git a/include/configs/ls2085a_common.h b/include/configs/ls2085a_common.h
> index 749c58d..a33b8a9 100644
> --- a/include/configs/ls2085a_common.h
> +++ b/include/configs/ls2085a_common.h
> @@ -248,13 +248,13 @@ unsigned long long get_qixis_addr(void);
>  	"initrd_high=0xffffffffffffffff\0"	\
>  	"kernel_start=0x581200000\0"		\
>  	"kernel_load=0xa0000000\0"		\
> -	"kernel_size=0x1000000\0"		\
> +	"kernel_size=0x2000000\0"		\
>  	"console=ttyAMA0,38400n8\0"

I didn't get a respond to my question for v1. How is kernel_size variable used?

York
Sharma Bhupesh May 28, 2015, 6:58 p.m. UTC | #2 
Hi York,

Sorry for missing this comment on v1.

> From: Sun York-R58495
> Sent: Thursday, May 28, 2015 9:31 PM
> 
> On 05/28/2015 02:24 AM, Prabhakar Kushwaha wrote:
> > From: Bhupesh Sharma <bhupesh.sharma at freescale.com>
> >
> > Increases the kernel size supported for LS2085A platforms:-
> >  - Update environment variables
> >  - Add ramdisk_size in bootargs env variable
> >  - Define  CONFIG_SYS_BOOTM_LEN to 64MB
> >
> > Signed-off-by: Bhupesh Sharma <bhupesh.sharma at freescale.com>
> > Signed-off-by: Prabhakar Kushwaha <prabhakar at freescale.com>
> > ---
> > Changes for v2: sending as it is for patch set
> >
> >  include/configs/ls2085a_common.h | 9 ++++++---
> >  include/configs/ls2085aqds.h     | 2 +-
> >  include/configs/ls2085ardb.h     | 2 +-
> >  3 files changed, 8 insertions(+), 5 deletions(-)
> >
> > diff --git a/include/configs/ls2085a_common.h
> b/include/configs/ls2085a_common.h
> > index 749c58d..a33b8a9 100644
> > --- a/include/configs/ls2085a_common.h
> > +++ b/include/configs/ls2085a_common.h
> > @@ -248,13 +248,13 @@ unsigned long long get_qixis_addr(void);
> >  	"initrd_high=0xffffffffffffffff\0"	\
> >  	"kernel_start=0x581200000\0"		\
> >  	"kernel_load=0xa0000000\0"		\
> > -	"kernel_size=0x1000000\0"		\
> > +	"kernel_size=0x2000000\0"		\
> >  	"console=ttyAMA0,38400n8\0"
> 
> I didn't get a respond to my question for v1. How is kernel_size variable
> used?

This variable is used for our default bootcmd to define the kernel size which needs to be copied
from the address pointed by the 'kernel_start' variable to the address pointed to by
the 'kernel_load' variable:

#define CONFIG_BOOTCOMMAND		"cp.b $kernel_start $kernel_load "     \
					"$kernel_size && bootm $kernel_load"

Regards,
Bhupesh
York Sun May 28, 2015, 7:02 p.m. UTC | #3 
On 05/28/2015 11:58 AM, Sharma Bhupesh-B45370 wrote:
> Hi York,
> 
> Sorry for missing this comment on v1.
> 
>> From: Sun York-R58495
>> Sent: Thursday, May 28, 2015 9:31 PM
>>
>> On 05/28/2015 02:24 AM, Prabhakar Kushwaha wrote:
>>> From: Bhupesh Sharma <bhupesh.sharma at freescale.com>
>>>
>>> Increases the kernel size supported for LS2085A platforms:-
>>>  - Update environment variables
>>>  - Add ramdisk_size in bootargs env variable
>>>  - Define  CONFIG_SYS_BOOTM_LEN to 64MB
>>>
>>> Signed-off-by: Bhupesh Sharma <bhupesh.sharma at freescale.com>
>>> Signed-off-by: Prabhakar Kushwaha <prabhakar at freescale.com>
>>> ---
>>> Changes for v2: sending as it is for patch set
>>>
>>>  include/configs/ls2085a_common.h | 9 ++++++---
>>>  include/configs/ls2085aqds.h     | 2 +-
>>>  include/configs/ls2085ardb.h     | 2 +-
>>>  3 files changed, 8 insertions(+), 5 deletions(-)
>>>
>>> diff --git a/include/configs/ls2085a_common.h
>> b/include/configs/ls2085a_common.h
>>> index 749c58d..a33b8a9 100644
>>> --- a/include/configs/ls2085a_common.h
>>> +++ b/include/configs/ls2085a_common.h
>>> @@ -248,13 +248,13 @@ unsigned long long get_qixis_addr(void);
>>>  	"initrd_high=0xffffffffffffffff\0"	\
>>>  	"kernel_start=0x581200000\0"		\
>>>  	"kernel_load=0xa0000000\0"		\
>>> -	"kernel_size=0x1000000\0"		\
>>> +	"kernel_size=0x2000000\0"		\
>>>  	"console=ttyAMA0,38400n8\0"
>>
>> I didn't get a respond to my question for v1. How is kernel_size variable
>> used?
> 
> This variable is used for our default bootcmd to define the kernel size which needs to be copied
> from the address pointed by the 'kernel_start' variable to the address pointed to by
> the 'kernel_load' variable:
> 
> #define CONFIG_BOOTCOMMAND		"cp.b $kernel_start $kernel_load "     \
> 					"$kernel_size && bootm $kernel_load"
> 

Got it. It is just a short-hand for copying images from NOR flash to RAM.

York
diff mbox

Patch

diff --git a/include/configs/ls2085a_common.h b/include/configs/ls2085a_common.h
index 749c58d..a33b8a9 100644
--- a/include/configs/ls2085a_common.h
+++ b/include/configs/ls2085a_common.h
@@ -248,13 +248,13 @@  unsigned long long get_qixis_addr(void);
 	"initrd_high=0xffffffffffffffff\0"	\
 	"kernel_start=0x581200000\0"		\
 	"kernel_load=0xa0000000\0"		\
-	"kernel_size=0x1000000\0"		\
+	"kernel_size=0x2000000\0"		\
 	"console=ttyAMA0,38400n8\0"
 
 #define CONFIG_BOOTARGS		"console=ttyS1,115200 root=/dev/ram0 " \
 				"earlycon=uart8250,mmio,0x21c0600,115200 " \
-				"default_hugepagesz=2m hugepagesz=2m " \
-				"hugepages=16"
+				"ramdisk_size=0x2000000 default_hugepagesz=2m" \
+				" hugepagesz=2m hugepages=16"
 #define CONFIG_BOOTCOMMAND		"cp.b $kernel_start $kernel_load "     \
 					"$kernel_size && bootm $kernel_load"
 #define CONFIG_BOOTDELAY		10
@@ -301,4 +301,7 @@  unsigned long get_dram_size_to_hide(void);
 #define CONFIG_SYS_SPL_MALLOC_START	0x80200000
 #define CONFIG_SYS_MONITOR_LEN		(512 * 1024)
 
+#define CONFIG_SYS_BOOTM_LEN   (64 << 20)      /* Increase max gunzip size */
+
+
 #endif /* __LS2_COMMON_H */
diff --git a/include/configs/ls2085aqds.h b/include/configs/ls2085aqds.h
index 87e9333..8cb8c78 100644
--- a/include/configs/ls2085aqds.h
+++ b/include/configs/ls2085aqds.h
@@ -336,7 +336,7 @@  unsigned long get_board_ddr_clk(void);
 	"initrd_high=0xffffffffffffffff\0"	\
 	"kernel_start=0x581100000\0"		\
 	"kernel_load=0xa0000000\0"		\
-	"kernel_size=0x1000000\0"
+	"kernel_size=0x2000000\0"
 
 #ifdef CONFIG_FSL_MC_ENET
 #define CONFIG_FSL_MEMAC
diff --git a/include/configs/ls2085ardb.h b/include/configs/ls2085ardb.h
index dab03bc..ba671dd 100644
--- a/include/configs/ls2085ardb.h
+++ b/include/configs/ls2085ardb.h
@@ -303,7 +303,7 @@  unsigned long get_board_sys_clk(void);
 	"initrd_high=0xffffffffffffffff\0"	\
 	"kernel_start=0x581100000\0"		\
 	"kernel_load=0xa0000000\0"		\
-	"kernel_size=0x1000000\0"
+	"kernel_size=0x2000000\0"
 
 /* MAC/PHY configuration */
 #ifdef CONFIG_FSL_MC_ENET