[U-Boot] [PATCH] arm:ls1021a: Reserve secure code into RAM instead of OCRAM

[Huan Wang]

Acked-by: Alison Wang <alison.wang at freescale.com>
Tested-by: Alison Wang <alison.wang at 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 at 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