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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot