From mboxrd@z Thu Jan 1 00:00:00 1970 From: Simon Glass Date: Wed, 20 Jun 2018 20:01:45 -0600 Subject: [U-Boot] [PATCH v8 19/30] efi: Relocate FDT to 127MB instead of 128MB In-Reply-To: <51cecd97-ff01-056b-f722-3cf777b7821c@suse.de> References: <20180618140835.195901-1-sjg@chromium.org> <20180618140835.195901-20-sjg@chromium.org> <51cecd97-ff01-056b-f722-3cf777b7821c@suse.de> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Alex, On 18 June 2018 at 08:52, Alexander Graf wrote: > On 06/18/2018 04:08 PM, Simon Glass wrote: >> >> Sandbox only has 128MB of memory so we cannot relocate the device tree up >> to start at 128MB. Use 127MB instead, which should be safe. >> >> Signed-off-by: Simon Glass > > > We should just drop into the fallback case if allocation fails, no? I cannot see how the allocation could ever fail, now that we are definitely in sandbox RAM. So I'm OK with doing that if you are. Regards, Simon