From mboxrd@z Thu Jan 1 00:00:00 1970 From: York Sun Date: Thu, 12 Nov 2015 08:42:46 -0800 Subject: [U-Boot] [RFC PATCH v2 1/2] Reserve secure memory In-Reply-To: <1447309701.2262.245.camel@freescale.com> References: <1447282212-1818-1-git-send-email-yorksun@freescale.com> <1447282212-1818-2-git-send-email-yorksun@freescale.com> <5643F6CD.40007@wytron.com.tw> <564408C8.9040006@freescale.com> <1447309701.2262.245.camel@freescale.com> Message-ID: <5644C186.6020300@freescale.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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