[U-Boot] [RFC PATCH v2 1/2] Reserve secure memory

York Sun yorksun at freescale.com
Thu Nov 12 17:42:46 CET 2015



On 11/11/2015 10:28 PM, Scott Wood wrote:
> On Wed, 2015-11-11 at 19:34 -0800, York Sun wrote:
>>
>> On 11/11/2015 06:17 PM, Thomas Chou wrote:
>>> Hi York,
>>>
>>> On 2015年11月12日 06:50, York Sun wrote:
>>>> diff --git a/include/asm-generic/global_data.h b/include/asm
>>>> -generic/global_data.h
>>>> index d0383f3..336f3a0 100644
>>>> --- a/include/asm-generic/global_data.h
>>>> +++ b/include/asm-generic/global_data.h
>>>> @@ -58,6 +58,7 @@ typedef struct global_data {
>>>>
>>>>   	unsigned long relocaddr;	/* Start address of U-Boot in
>>>> RAM */
>>>>   	phys_size_t ram_size;	/* RAM size */
>>>> +	phys_addr_t secure_ram;	/* Secure memory addr */
>>>
>>> Shouldn't this be included only if CONFIG_SYS_MEM_RESERVE_SECURE ?
>>
>> It can be. It will require checking CONFIG_SYS_MEM_RESERVE_SECURE every time
>> this variable is used. I will add it in next version.
> 
> Why would that be better than leaving it unifdeffed?

Save a few bytes? I prefer not to use ifdef if possible, but I understand some
platforms don't want to spare a few bytes.

York



More information about the U-Boot mailing list