Re: [U-Boot] [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 > --- Applied to u-boot-fsl-qoriq master. Thanks. York ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] arm:ls1021a: Reserve secure code into RAM instead of OCRAM
Acked-by: Alison Wang Tested-by: Alison Wang 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 > > > > --- > > > > 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_ADDR0x01ee0200 > > > > #define CONFIG_TIMER_CLK_FREQ 1250 > > > > -#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_ADDR0x01ee0200 > > > > #define CONFIG_TIMER_CLK_FREQ 1250 > > > > -#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 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [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 > > > --- > > > 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_FREQ1250 > > > -#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_FREQ1250 > > > -#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 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [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 > > --- > > 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_ADDR0x01ee0200 > > #define CONFIG_TIMER_CLK_FREQ 1250 > > -#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_ADDR0x01ee0200 > > #define CONFIG_TIMER_CLK_FREQ 1250 > > -#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 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [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 > --- > 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_FREQ1250 > -#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_FREQ1250 > -#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 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[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 --- 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_ADDR0x01ee0200 #define CONFIG_TIMER_CLK_FREQ 1250 -#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_ADDR0x01ee0200 #define CONFIG_TIMER_CLK_FREQ 1250 -#define CONFIG_ARMV7_SECURE_BASE OCRAM_BASE_S_ADDR #define CONFIG_HWCONFIG #define HWCONFIG_BUFFER_SIZE 128 -- 2.1.0.27.g96db324 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
