* [U-Boot] [PATCH] ARM: HYP/non-sec: Make a variable as a local one
@ 2014-09-18 9:12 Tang Yuantian
2014-09-18 9:43 ` Albert ARIBAUD
0 siblings, 1 reply; 3+ messages in thread
From: Tang Yuantian @ 2014-09-18 9:12 UTC (permalink / raw)
To: u-boot
Defining variable gic_dist_addr as a globe one prevents some
functions, which use this variable, from being used before relocation
which happened in the deep sleep resume process on Freescale SoC
platforms.
Signed-off-by: Tang Yuantian <Yuantian.Tang@freescale.com>
---
arch/arm/cpu/armv7/virt-v7.c | 9 +++++++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/arch/arm/cpu/armv7/virt-v7.c b/arch/arm/cpu/armv7/virt-v7.c
index 651ca40..b69fd37 100644
--- a/arch/arm/cpu/armv7/virt-v7.c
+++ b/arch/arm/cpu/armv7/virt-v7.c
@@ -15,8 +15,6 @@
#include <asm/io.h>
#include <asm/secure.h>
-unsigned long gic_dist_addr;
-
static unsigned int read_id_pfr1(void)
{
unsigned int reg;
@@ -68,6 +66,12 @@ static void kick_secondary_cpus_gic(unsigned long gicdaddr)
void __weak smp_kick_all_cpus(void)
{
+ unsigned long gic_dist_addr;
+
+ gic_dist_addr = get_gicd_base_address();
+ if (gic_dist_addr == -1)
+ return;
+
kick_secondary_cpus_gic(gic_dist_addr);
}
@@ -75,6 +79,7 @@ int armv7_init_nonsec(void)
{
unsigned int reg;
unsigned itlinesnr, i;
+ unsigned long gic_dist_addr;
/* check whether the CPU supports the security extensions */
reg = read_id_pfr1();
--
2.1.0.27.g96db324
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [U-Boot] [PATCH] ARM: HYP/non-sec: Make a variable as a local one
2014-09-18 9:12 [U-Boot] [PATCH] ARM: HYP/non-sec: Make a variable as a local one Tang Yuantian
@ 2014-09-18 9:43 ` Albert ARIBAUD
2014-09-19 1:58 ` Yuantian Tang
0 siblings, 1 reply; 3+ messages in thread
From: Albert ARIBAUD @ 2014-09-18 9:43 UTC (permalink / raw)
To: u-boot
Hi Tang,
Please fix subject / commit summary to indicate which variable is to be
made local.
On Thu, 18 Sep 2014 17:12:34 +0800, Tang Yuantian
<Yuantian.Tang@freescale.com> wrote:
> Defining variable gic_dist_addr as a globe one prevents some
> functions, which use this variable, from being used before relocation
> which happened in the deep sleep resume process on Freescale SoC
> platforms.
>
> Signed-off-by: Tang Yuantian <Yuantian.Tang@freescale.com>
> ---
> arch/arm/cpu/armv7/virt-v7.c | 9 +++++++--
> 1 file changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/cpu/armv7/virt-v7.c b/arch/arm/cpu/armv7/virt-v7.c
> index 651ca40..b69fd37 100644
> --- a/arch/arm/cpu/armv7/virt-v7.c
> +++ b/arch/arm/cpu/armv7/virt-v7.c
> @@ -15,8 +15,6 @@
> #include <asm/io.h>
> #include <asm/secure.h>
>
> -unsigned long gic_dist_addr;
> -
> static unsigned int read_id_pfr1(void)
> {
> unsigned int reg;
> @@ -68,6 +66,12 @@ static void kick_secondary_cpus_gic(unsigned long gicdaddr)
>
> void __weak smp_kick_all_cpus(void)
> {
> + unsigned long gic_dist_addr;
> +
> + gic_dist_addr = get_gicd_base_address();
> + if (gic_dist_addr == -1)
> + return;
> +
> kick_secondary_cpus_gic(gic_dist_addr);
> }
>
> @@ -75,6 +79,7 @@ int armv7_init_nonsec(void)
> {
> unsigned int reg;
> unsigned itlinesnr, i;
> + unsigned long gic_dist_addr;
>
> /* check whether the CPU supports the security extensions */
> reg = read_id_pfr1();
Did you check that the global was not used as a common variable between
the functions?
Amicalement,
--
Albert.
^ permalink raw reply [flat|nested] 3+ messages in thread
* [U-Boot] [PATCH] ARM: HYP/non-sec: Make a variable as a local one
2014-09-18 9:43 ` Albert ARIBAUD
@ 2014-09-19 1:58 ` Yuantian Tang
0 siblings, 0 replies; 3+ messages in thread
From: Yuantian Tang @ 2014-09-19 1:58 UTC (permalink / raw)
To: u-boot
> -----Original Message-----
> From: Albert ARIBAUD [mailto:albert.u.boot at aribaud.net]
> Sent: Thursday, September 18, 2014 5:44 PM
> To: Tang Yuantian-B29983
> Cc: andre.przywara at linaro.org; ijc at hellion.org.uk; marc.zyngier at arm.com;
> u-boot at lists.denx.de; Jin Zhengxiong-R64188
> Subject: Re: [PATCH] ARM: HYP/non-sec: Make a variable as a local one
>
> Hi Tang,
>
> Please fix subject / commit summary to indicate which variable is to be made
> local.
>
> On Thu, 18 Sep 2014 17:12:34 +0800, Tang Yuantian
> <Yuantian.Tang@freescale.com> wrote:
>
> > Defining variable gic_dist_addr as a globe one prevents some
> > functions, which use this variable, from being used before relocation
> > which happened in the deep sleep resume process on Freescale SoC
> > platforms.
> >
> > Signed-off-by: Tang Yuantian <Yuantian.Tang@freescale.com>
> > ---
> > arch/arm/cpu/armv7/virt-v7.c | 9 +++++++--
> > 1 file changed, 7 insertions(+), 2 deletions(-)
> >
> > diff --git a/arch/arm/cpu/armv7/virt-v7.c
> > b/arch/arm/cpu/armv7/virt-v7.c index 651ca40..b69fd37 100644
> > --- a/arch/arm/cpu/armv7/virt-v7.c
> > +++ b/arch/arm/cpu/armv7/virt-v7.c
> > @@ -15,8 +15,6 @@
> > #include <asm/io.h>
> > #include <asm/secure.h>
> >
> > -unsigned long gic_dist_addr;
> > -
> > static unsigned int read_id_pfr1(void) {
> > unsigned int reg;
> > @@ -68,6 +66,12 @@ static void kick_secondary_cpus_gic(unsigned long
> > gicdaddr)
> >
> > void __weak smp_kick_all_cpus(void)
> > {
> > + unsigned long gic_dist_addr;
> > +
> > + gic_dist_addr = get_gicd_base_address();
> > + if (gic_dist_addr == -1)
> > + return;
> > +
> > kick_secondary_cpus_gic(gic_dist_addr);
> > }
> >
> > @@ -75,6 +79,7 @@ int armv7_init_nonsec(void) {
> > unsigned int reg;
> > unsigned itlinesnr, i;
> > + unsigned long gic_dist_addr;
> >
> > /* check whether the CPU supports the security extensions */
> > reg = read_id_pfr1();
>
> Did you check that the global was not used as a common variable between the
> functions?
>
Yes, I grep it globally. It was referred by only 2 places which are both in virt-v7.c file.
I will fix the subject.
Thanks,
Yuantian
> Amicalement,
> --
> Albert.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-09-19 1:58 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-09-18 9:12 [U-Boot] [PATCH] ARM: HYP/non-sec: Make a variable as a local one Tang Yuantian
2014-09-18 9:43 ` Albert ARIBAUD
2014-09-19 1:58 ` Yuantian Tang
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.