Re: [U-Boot] [PATCH 2/4] ARM: HYP/non-sec: Make armv7_init_nonsec() usable before relocation

2014-10-28 Thread Albert ARIBAUD
Hello Yuantian, On Tue, 28 Oct 2014 02:24:10 +, Yuantian Tang yuantian.t...@freescale.com wrote: -Original Message- From: Albert ARIBAUD [mailto:albert.u.b...@aribaud.net] Sent: Monday, October 27, 2014 5:42 PM To: Tang Yuantian-B29983 Cc: Sun York-R58495;

Re: [U-Boot] [PATCH 2/4] ARM: HYP/non-sec: Make armv7_init_nonsec() usable before relocation

2014-10-27 Thread Albert ARIBAUD
Hello Yuantian, On Thu, 16 Oct 2014 04:42:06 +, Yuantian Tang yuantian.t...@freescale.com wrote: Wouldn't it be better to declare gic_dist_base as a local variable? It is only used once outside function armv7_switch_nonsec(). It could be replaced with get_gicd_base_address() call.

Re: [U-Boot] [PATCH 2/4] ARM: HYP/non-sec: Make armv7_init_nonsec() usable before relocation

2014-10-27 Thread Yuantian Tang
-Original Message- From: Albert ARIBAUD [mailto:albert.u.b...@aribaud.net] Sent: Monday, October 27, 2014 5:42 PM To: Tang Yuantian-B29983 Cc: Sun York-R58495; u-boot@lists.denx.de; Jin Zhengxiong-R64188 Subject: Re: [PATCH 2/4] ARM: HYP/non-sec: Make armv7_init_nonsec() usable

Re: [U-Boot] [PATCH 2/4] ARM: HYP/non-sec: Make armv7_init_nonsec() usable before relocation

2014-10-15 Thread York Sun
On 10/09/2014 01:11 AM, yuantian.t...@freescale.com wrote: From: Tang Yuantian yuantian.t...@freescale.com Defining variable gic_dist_addr as a globe one prevents function armv7_init_nonsec() from being used before relocation which is the case in the deep sleep resume process on Freescale

Re: [U-Boot] [PATCH 2/4] ARM: HYP/non-sec: Make armv7_init_nonsec() usable before relocation

2014-10-15 Thread Yuantian Tang
-Original Message- From: Sun York-R58495 Sent: Wednesday, October 15, 2014 11:44 PM To: Tang Yuantian-B29983; albert.u.b...@aribaud.net Cc: u-boot@lists.denx.de; Jin Zhengxiong-R64188 Subject: Re: [PATCH 2/4] ARM: HYP/non-sec: Make armv7_init_nonsec() usable before relocation On

Re: [U-Boot] [PATCH 2/4] ARM: HYP/non-sec: Make armv7_init_nonsec() usable before relocation

2014-10-15 Thread York Sun
On 10/15/14 8:02 PM, Tang Yuantian-B29983 yuantian.t...@freescale.com wrote: -Original Message- From: Sun York-R58495 Sent: Wednesday, October 15, 2014 11:44 PM To: Tang Yuantian-B29983; albert.u.b...@aribaud.net Cc: u-boot@lists.denx.de; Jin Zhengxiong-R64188 Subject: Re: [PATCH

Re: [U-Boot] [PATCH 2/4] ARM: HYP/non-sec: Make armv7_init_nonsec() usable before relocation

2014-10-15 Thread Yuantian Tang
Wouldn't it be better to declare gic_dist_base as a local variable? It is only used once outside function armv7_switch_nonsec(). It could be replaced with get_gicd_base_address() call. I am with you. That's what I did in the first version of this patch. Patch links is at:

[U-Boot] [PATCH 2/4] ARM: HYP/non-sec: Make armv7_init_nonsec() usable before relocation

2014-10-09 Thread Yuantian.Tang
From: Tang Yuantian yuantian.t...@freescale.com Defining variable gic_dist_addr as a globe one prevents function armv7_init_nonsec() from being used before relocation which is the case in the deep sleep resume process on Freescale QorIQ SoC platforms. This patch removes this limitation by adding

[U-Boot] [PATCH 2/4] ARM: HYP/non-sec: Make armv7_init_nonsec() usable before relocation

2014-09-28 Thread Yuantian.Tang
From: Tang Yuantian yuantian.t...@freescale.com Defining variable gic_dist_addr as a globe one prevents function armv7_init_nonsec() from being used before relocation which is the case in the deep sleep resume process on Freescale QorIQ SoC platforms. This patch removes this limitation by adding