[U-Boot] [PATCH] arm:ls1021a: Reserve secure code into RAM instead of OCRAM
For ls1021a, Reserve secure code in to memory in case OCRAM is needed by other usage.
Signed-off-by: Zhuoyu Zhang Zhuoyu.Zhang@freescale.com --- include/configs/ls1021aqds.h | 1 - include/configs/ls1021atwr.h | 1 - 2 files changed, 2 deletions(-)
diff --git a/include/configs/ls1021aqds.h b/include/configs/ls1021aqds.h index 3dc4da3..42439a4 100644 --- a/include/configs/ls1021aqds.h +++ b/include/configs/ls1021aqds.h @@ -554,7 +554,6 @@ unsigned long get_board_ddr_clk(void); #define CONFIG_LS102XA_NS_ACCESS #define CONFIG_SMP_PEN_ADDR 0x01ee0200 #define CONFIG_TIMER_CLK_FREQ 12500000 -#define CONFIG_ARMV7_SECURE_BASE OCRAM_BASE_S_ADDR
#define CONFIG_HWCONFIG #define HWCONFIG_BUFFER_SIZE 128 diff --git a/include/configs/ls1021atwr.h b/include/configs/ls1021atwr.h index a13876b..dccc661 100644 --- a/include/configs/ls1021atwr.h +++ b/include/configs/ls1021atwr.h @@ -347,7 +347,6 @@ #define CONFIG_LS102XA_NS_ACCESS #define CONFIG_SMP_PEN_ADDR 0x01ee0200 #define CONFIG_TIMER_CLK_FREQ 12500000 -#define CONFIG_ARMV7_SECURE_BASE OCRAM_BASE_S_ADDR
#define CONFIG_HWCONFIG #define HWCONFIG_BUFFER_SIZE 128
On 03/24/2015 02:27 AM, Zhuoyu Zhang wrote:
For ls1021a, Reserve secure code in to memory in case OCRAM is needed by other usage.
Signed-off-by: Zhuoyu Zhang Zhuoyu.Zhang@freescale.com
include/configs/ls1021aqds.h | 1 - include/configs/ls1021atwr.h | 1 - 2 files changed, 2 deletions(-)
diff --git a/include/configs/ls1021aqds.h b/include/configs/ls1021aqds.h index 3dc4da3..42439a4 100644 --- a/include/configs/ls1021aqds.h +++ b/include/configs/ls1021aqds.h @@ -554,7 +554,6 @@ unsigned long get_board_ddr_clk(void); #define CONFIG_LS102XA_NS_ACCESS #define CONFIG_SMP_PEN_ADDR 0x01ee0200 #define CONFIG_TIMER_CLK_FREQ 12500000 -#define CONFIG_ARMV7_SECURE_BASE OCRAM_BASE_S_ADDR
#define CONFIG_HWCONFIG #define HWCONFIG_BUFFER_SIZE 128 diff --git a/include/configs/ls1021atwr.h b/include/configs/ls1021atwr.h index a13876b..dccc661 100644 --- a/include/configs/ls1021atwr.h +++ b/include/configs/ls1021atwr.h @@ -347,7 +347,6 @@ #define CONFIG_LS102XA_NS_ACCESS #define CONFIG_SMP_PEN_ADDR 0x01ee0200 #define CONFIG_TIMER_CLK_FREQ 12500000 -#define CONFIG_ARMV7_SECURE_BASE OCRAM_BASE_S_ADDR
#define CONFIG_HWCONFIG #define HWCONFIG_BUFFER_SIZE 128
Zhuoyu,
I don't see how the secure code is reserved in RAM. You only remove CONFIG_ARMV7_SECURE_BASE. Did I miss something?
York
Hi, York
Sorry for the delay, I wrongly filter this patch and missed it. As for your question, if platform does not define CONFIG_ARMV7_SECURE_BASE, U-boot will help define a default secure text section in linker script to reserve secure code. CONFIG_ARMV7_SECURE_BASE for LS1 is previously defined as OCRAM_BASE, This patch just remove it and use DDR by default.
Zhuoyu
-----Original Message----- From: Sun York-R58495 Sent: Friday, April 17, 2015 4:03 AM To: Zhang Zhuoyu-B46552 Cc: u-boot@lists.denx.de; Jin Zhengxiong-R64188; Wang Huan-B18965 Subject: Re: [PATCH] arm:ls1021a: Reserve secure code into RAM instead of OCRAM
On 03/24/2015 02:27 AM, Zhuoyu Zhang wrote:
For ls1021a, Reserve secure code in to memory in case OCRAM is needed by other usage.
Signed-off-by: Zhuoyu Zhang Zhuoyu.Zhang@freescale.com
include/configs/ls1021aqds.h | 1 - include/configs/ls1021atwr.h | 1 - 2 files changed, 2 deletions(-)
diff --git a/include/configs/ls1021aqds.h b/include/configs/ls1021aqds.h index 3dc4da3..42439a4 100644 --- a/include/configs/ls1021aqds.h +++ b/include/configs/ls1021aqds.h @@ -554,7 +554,6 @@ unsigned long get_board_ddr_clk(void); #define CONFIG_LS102XA_NS_ACCESS #define CONFIG_SMP_PEN_ADDR 0x01ee0200 #define CONFIG_TIMER_CLK_FREQ 12500000 -#define CONFIG_ARMV7_SECURE_BASE OCRAM_BASE_S_ADDR
#define CONFIG_HWCONFIG #define HWCONFIG_BUFFER_SIZE 128 diff --git a/include/configs/ls1021atwr.h b/include/configs/ls1021atwr.h index a13876b..dccc661 100644 --- a/include/configs/ls1021atwr.h +++ b/include/configs/ls1021atwr.h @@ -347,7 +347,6 @@ #define CONFIG_LS102XA_NS_ACCESS #define CONFIG_SMP_PEN_ADDR 0x01ee0200 #define CONFIG_TIMER_CLK_FREQ 12500000 -#define CONFIG_ARMV7_SECURE_BASE OCRAM_BASE_S_ADDR
#define CONFIG_HWCONFIG #define HWCONFIG_BUFFER_SIZE 128
Zhuoyu,
I don't see how the secure code is reserved in RAM. You only remove CONFIG_ARMV7_SECURE_BASE. Did I miss something?
York
Hi, York
Could you help ACK this patch?
Best Regards Zhuoyu
-----Original Message----- From: Zhang Zhuoyu-B46552 Sent: Wednesday, July 29, 2015 10:25 AM To: Sun York-R58495 Cc: u-boot@lists.denx.de; Jin Zhengxiong-R64188; Wang Huan-B18965 Subject: RE: [PATCH] arm:ls1021a: Reserve secure code into RAM instead of OCRAM
Hi, York
Sorry for the delay, I wrongly filter this patch and missed it. As for your question, if platform does not define CONFIG_ARMV7_SECURE_BASE, U-boot will help define a default secure text section in linker script to reserve secure code. CONFIG_ARMV7_SECURE_BASE for LS1 is previously defined as OCRAM_BASE, This patch just remove it and use DDR by default.
Zhuoyu
-----Original Message----- From: Sun York-R58495 Sent: Friday, April 17, 2015 4:03 AM To: Zhang Zhuoyu-B46552 Cc: u-boot@lists.denx.de; Jin Zhengxiong-R64188; Wang Huan-B18965 Subject: Re: [PATCH] arm:ls1021a: Reserve secure code into RAM instead of OCRAM
On 03/24/2015 02:27 AM, Zhuoyu Zhang wrote:
For ls1021a, Reserve secure code in to memory in case OCRAM is needed by other usage.
Signed-off-by: Zhuoyu Zhang Zhuoyu.Zhang@freescale.com
include/configs/ls1021aqds.h | 1 - include/configs/ls1021atwr.h | 1 - 2 files changed, 2 deletions(-)
diff --git a/include/configs/ls1021aqds.h b/include/configs/ls1021aqds.h index 3dc4da3..42439a4 100644 --- a/include/configs/ls1021aqds.h +++ b/include/configs/ls1021aqds.h @@ -554,7 +554,6 @@ unsigned long get_board_ddr_clk(void); #define CONFIG_LS102XA_NS_ACCESS #define CONFIG_SMP_PEN_ADDR 0x01ee0200 #define CONFIG_TIMER_CLK_FREQ 12500000 -#define CONFIG_ARMV7_SECURE_BASE OCRAM_BASE_S_ADDR
#define CONFIG_HWCONFIG #define HWCONFIG_BUFFER_SIZE 128 diff --git a/include/configs/ls1021atwr.h b/include/configs/ls1021atwr.h index a13876b..dccc661 100644 --- a/include/configs/ls1021atwr.h +++ b/include/configs/ls1021atwr.h @@ -347,7 +347,6 @@ #define CONFIG_LS102XA_NS_ACCESS #define CONFIG_SMP_PEN_ADDR 0x01ee0200 #define CONFIG_TIMER_CLK_FREQ 12500000 -#define CONFIG_ARMV7_SECURE_BASE OCRAM_BASE_S_ADDR
#define CONFIG_HWCONFIG #define HWCONFIG_BUFFER_SIZE 128
Zhuoyu,
I don't see how the secure code is reserved in RAM. You only remove CONFIG_ARMV7_SECURE_BASE. Did I miss something?
York
Acked-by: Alison Wang alison.wang@freescale.com Tested-by: Alison Wang alison.wang@freescale.com
Best Regards, Alison Wang
-----Original Message----- From: Zhang Zhuoyu-B46552 Sent: Friday, July 31, 2015 2:02 PM To: Sun York-R58495 Cc: u-boot@lists.denx.de; Jin Zhengxiong-R64188; Wang Huan-B18965; Wang Dongsheng-B40534 Subject: RE: [PATCH] arm:ls1021a: Reserve secure code into RAM instead of OCRAM
Hi, York
Could you help ACK this patch?
Best Regards Zhuoyu
-----Original Message----- From: Zhang Zhuoyu-B46552 Sent: Wednesday, July 29, 2015 10:25 AM To: Sun York-R58495 Cc: u-boot@lists.denx.de; Jin Zhengxiong-R64188; Wang Huan-B18965 Subject: RE: [PATCH] arm:ls1021a: Reserve secure code into RAM
instead
of OCRAM
Hi, York
Sorry for the delay, I wrongly filter this patch and missed it. As for your question, if platform does not define CONFIG_ARMV7_SECURE_BASE, U-boot will help define a default secure text section in linker script to reserve secure code. CONFIG_ARMV7_SECURE_BASE for LS1 is previously defined as OCRAM_BASE, This patch just remove it and use DDR by default.
Zhuoyu
-----Original Message----- From: Sun York-R58495 Sent: Friday, April 17, 2015 4:03 AM To: Zhang Zhuoyu-B46552 Cc: u-boot@lists.denx.de; Jin Zhengxiong-R64188; Wang Huan-B18965 Subject: Re: [PATCH] arm:ls1021a: Reserve secure code into RAM instead of OCRAM
On 03/24/2015 02:27 AM, Zhuoyu Zhang wrote:
For ls1021a, Reserve secure code in to memory in case OCRAM is needed by other usage.
Signed-off-by: Zhuoyu Zhang Zhuoyu.Zhang@freescale.com
include/configs/ls1021aqds.h | 1 - include/configs/ls1021atwr.h | 1 - 2 files changed, 2 deletions(-)
diff --git a/include/configs/ls1021aqds.h b/include/configs/ls1021aqds.h index 3dc4da3..42439a4 100644 --- a/include/configs/ls1021aqds.h +++ b/include/configs/ls1021aqds.h @@ -554,7 +554,6 @@ unsigned long get_board_ddr_clk(void); #define CONFIG_LS102XA_NS_ACCESS #define CONFIG_SMP_PEN_ADDR 0x01ee0200 #define CONFIG_TIMER_CLK_FREQ 12500000 -#define CONFIG_ARMV7_SECURE_BASE OCRAM_BASE_S_ADDR
#define CONFIG_HWCONFIG #define HWCONFIG_BUFFER_SIZE 128 diff --git a/include/configs/ls1021atwr.h b/include/configs/ls1021atwr.h index a13876b..dccc661 100644 --- a/include/configs/ls1021atwr.h +++ b/include/configs/ls1021atwr.h @@ -347,7 +347,6 @@ #define CONFIG_LS102XA_NS_ACCESS #define CONFIG_SMP_PEN_ADDR 0x01ee0200 #define CONFIG_TIMER_CLK_FREQ 12500000 -#define CONFIG_ARMV7_SECURE_BASE OCRAM_BASE_S_ADDR
#define CONFIG_HWCONFIG #define HWCONFIG_BUFFER_SIZE 128
Zhuoyu,
I don't see how the secure code is reserved in RAM. You only remove CONFIG_ARMV7_SECURE_BASE. Did I miss something?
York
On 03/24/2015 02:27 AM, Zhuoyu Zhang wrote:
For ls1021a, Reserve secure code in to memory in case OCRAM is needed by other usage.
Signed-off-by: Zhuoyu Zhang Zhuoyu.Zhang@freescale.com
Applied to u-boot-fsl-qoriq master. Thanks.
York
participants (4)
-
Huan Wang -
York Sun -
Zhang Z.Y. -
Zhuoyu Zhang