* [U-Boot] [PATCH] arm:ls1021a: Reserve secure code into RAM instead of OCRAM
@ 2015-03-24 9:27 Zhuoyu Zhang
2015-04-16 20:02 ` York Sun
2015-08-04 15:43 ` York Sun
0 siblings, 2 replies; 6+ messages in thread
From: Zhuoyu Zhang @ 2015-03-24 9:27 UTC (permalink / raw)
To: u-boot
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
--
2.1.0.27.g96db324
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] arm:ls1021a: Reserve secure code into RAM instead of OCRAM
2015-03-24 9:27 [U-Boot] [PATCH] arm:ls1021a: Reserve secure code into RAM instead of OCRAM Zhuoyu Zhang
@ 2015-04-16 20:02 ` York Sun
2015-07-29 2:25 ` Zhang Z.Y.
2015-07-31 6:02 ` Zhang Z.Y.
2015-08-04 15:43 ` York Sun
1 sibling, 2 replies; 6+ messages in thread
From: York Sun @ 2015-04-16 20:02 UTC (permalink / raw)
To: u-boot
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
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] arm:ls1021a: Reserve secure code into RAM instead of OCRAM
2015-04-16 20:02 ` York Sun
@ 2015-07-29 2:25 ` Zhang Z.Y.
2015-07-31 6:02 ` Zhang Z.Y.
1 sibling, 0 replies; 6+ messages in thread
From: Zhang Z.Y. @ 2015-07-29 2:25 UTC (permalink / raw)
To: u-boot
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 at 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
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] arm:ls1021a: Reserve secure code into RAM instead of OCRAM
2015-04-16 20:02 ` York Sun
2015-07-29 2:25 ` Zhang Z.Y.
@ 2015-07-31 6:02 ` Zhang Z.Y.
2015-08-03 2:29 ` Huan Wang
1 sibling, 1 reply; 6+ messages in thread
From: Zhang Z.Y. @ 2015-07-31 6:02 UTC (permalink / raw)
To: u-boot
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 at 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 at 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
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] arm:ls1021a: Reserve secure code into RAM instead of OCRAM
2015-07-31 6:02 ` Zhang Z.Y.
@ 2015-08-03 2:29 ` Huan Wang
0 siblings, 0 replies; 6+ messages in thread
From: Huan Wang @ 2015-08-03 2:29 UTC (permalink / raw)
To: u-boot
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 at 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 at 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 at 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
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] arm:ls1021a: Reserve secure code into RAM instead of OCRAM
2015-03-24 9:27 [U-Boot] [PATCH] arm:ls1021a: Reserve secure code into RAM instead of OCRAM Zhuoyu Zhang
2015-04-16 20:02 ` York Sun
@ 2015-08-04 15:43 ` York Sun
1 sibling, 0 replies; 6+ messages in thread
From: York Sun @ 2015-08-04 15:43 UTC (permalink / raw)
To: u-boot
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
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2015-08-04 15:43 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-03-24 9:27 [U-Boot] [PATCH] arm:ls1021a: Reserve secure code into RAM instead of OCRAM Zhuoyu Zhang
2015-04-16 20:02 ` York Sun
2015-07-29 2:25 ` Zhang Z.Y.
2015-07-31 6:02 ` Zhang Z.Y.
2015-08-03 2:29 ` Huan Wang
2015-08-04 15:43 ` York Sun
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.