Re: [U-Boot] [PATCH 5/8] armv8: fsl-layerscape: add define CONFIG_STANDALONE_LOAD_ADDR for standalone app

[Shaohui Xie]

> On 08/26/2016 04:40 AM, Gong Qianyu wrote:
> > From: Shaohui Xie 
> >
> > The CONFIG_STANDALONE_LOAD_ADDR is set to 0x8030 by default.
> >
> > Signed-off-by: Shaohui Xie 
> > Signed-off-by: Gong Qianyu 
> > ---
> >  arch/arm/include/asm/arch-fsl-layerscape/config.h | 2 ++
> >  1 file changed, 2 insertions(+)
> >
> > diff --git a/arch/arm/include/asm/arch-fsl-layerscape/config.h
> b/arch/arm/include/asm/arch-fsl-layerscape/config.h
> > index b0ad4b4..c7e374c 100644
> > --- a/arch/arm/include/asm/arch-fsl-layerscape/config.h
> > +++ b/arch/arm/include/asm/arch-fsl-layerscape/config.h
> > @@ -9,6 +9,8 @@
> >
> >  #include 
> >
> > +#define CONFIG_STANDALONE_LOAD_ADDR0x8030
> > +
> 
> Do you really need this? It is default to 0x8030 in arch/arm/config.mk.
[S.H] It's needed, in the arch/arm/config.mk, it's defined as:

ifndef CONFIG_STANDALONE_LOAD_ADDR
ifneq ($(CONFIG_OMAP_COMMON),)
CONFIG_STANDALONE_LOAD_ADDR = 0x8030
else
CONFIG_STANDALONE_LOAD_ADDR = 0xc10
endif
endif

we need the CONFIG_STANDALONE_LOAD_ADDR defined as a valid address for ARM.
The proposed patch is to clearly define the CONFIG_STANDALONE_LOAD_ADDR for our 
SOCs.

Thanks,
Shaohui



___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 5/8] armv8: fsl-layerscape: add define CONFIG_STANDALONE_LOAD_ADDR for standalone app

[york sun]

On 08/28/2016 09:58 PM, Shaohui Xie wrote:
>> On 08/26/2016 04:40 AM, Gong Qianyu wrote:
>>> From: Shaohui Xie 
>>>
>>> The CONFIG_STANDALONE_LOAD_ADDR is set to 0x8030 by default.
>>>
>>> Signed-off-by: Shaohui Xie 
>>> Signed-off-by: Gong Qianyu 
>>> ---
>>>  arch/arm/include/asm/arch-fsl-layerscape/config.h | 2 ++
>>>  1 file changed, 2 insertions(+)
>>>
>>> diff --git a/arch/arm/include/asm/arch-fsl-layerscape/config.h
>> b/arch/arm/include/asm/arch-fsl-layerscape/config.h
>>> index b0ad4b4..c7e374c 100644
>>> --- a/arch/arm/include/asm/arch-fsl-layerscape/config.h
>>> +++ b/arch/arm/include/asm/arch-fsl-layerscape/config.h
>>> @@ -9,6 +9,8 @@
>>>
>>>  #include 
>>>
>>> +#define CONFIG_STANDALONE_LOAD_ADDR0x8030
>>> +
>>
>> Do you really need this? It is default to 0x8030 in arch/arm/config.mk.
> [S.H] It's needed, in the arch/arm/config.mk, it's defined as:
>
> ifndef CONFIG_STANDALONE_LOAD_ADDR
> ifneq ($(CONFIG_OMAP_COMMON),)
> CONFIG_STANDALONE_LOAD_ADDR = 0x8030
> else
> CONFIG_STANDALONE_LOAD_ADDR = 0xc10
> endif
> endif
>
> we need the CONFIG_STANDALONE_LOAD_ADDR defined as a valid address for ARM.
> The proposed patch is to clearly define the CONFIG_STANDALONE_LOAD_ADDR for 
> our SOCs.
>

All right. Somehow I remember using 0x8030 before, without this 
change. I must have it set somewhere else.

York

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 5/8] armv8: fsl-layerscape: add define CONFIG_STANDALONE_LOAD_ADDR for standalone app

[york sun]

On 08/26/2016 04:40 AM, Gong Qianyu wrote:
> From: Shaohui Xie 
>
> The CONFIG_STANDALONE_LOAD_ADDR is set to 0x8030 by default.
>
> Signed-off-by: Shaohui Xie 
> Signed-off-by: Gong Qianyu 
> ---
>  arch/arm/include/asm/arch-fsl-layerscape/config.h | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/arch/arm/include/asm/arch-fsl-layerscape/config.h 
> b/arch/arm/include/asm/arch-fsl-layerscape/config.h
> index b0ad4b4..c7e374c 100644
> --- a/arch/arm/include/asm/arch-fsl-layerscape/config.h
> +++ b/arch/arm/include/asm/arch-fsl-layerscape/config.h
> @@ -9,6 +9,8 @@
>
>  #include 
>
> +#define CONFIG_STANDALONE_LOAD_ADDR  0x8030
> +

Do you really need this? It is default to 0x8030 in arch/arm/config.mk.

York

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH 5/8] armv8: fsl-layerscape: add define CONFIG_STANDALONE_LOAD_ADDR for standalone app

[Gong Qianyu]

From: Shaohui Xie 

The CONFIG_STANDALONE_LOAD_ADDR is set to 0x8030 by default.

Signed-off-by: Shaohui Xie 
Signed-off-by: Gong Qianyu 
---
 arch/arm/include/asm/arch-fsl-layerscape/config.h | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/arm/include/asm/arch-fsl-layerscape/config.h 
b/arch/arm/include/asm/arch-fsl-layerscape/config.h
index b0ad4b4..c7e374c 100644
--- a/arch/arm/include/asm/arch-fsl-layerscape/config.h
+++ b/arch/arm/include/asm/arch-fsl-layerscape/config.h
@@ -9,6 +9,8 @@
 
 #include 
 
+#define CONFIG_STANDALONE_LOAD_ADDR0x8030
+
 #ifdef CONFIG_SYS_FSL_DDR4
 #define CONFIG_SYS_FSL_DDRC_GEN4
 #else
-- 
2.1.0.27.g96db324

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot