From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andy Shevchenko Date: Tue, 15 Dec 2020 12:05:30 +0200 Subject: [PATCH v1] x86: tangier: Find proper memory region for relocation In-Reply-To: References: <20201127124048.11843-1-andriy.shevchenko@linux.intel.com> Message-ID: <20201215100530.GP4077@smile.fi.intel.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 Mon, Nov 30, 2020 at 01:12:17PM -0700, Simon Glass wrote: > On Fri, 27 Nov 2020 at 05:40, Andy Shevchenko > wrote: > > > > It appears that U-Boot works by luck on Intel Edison board because the amount > > of RAM is less than 1 GB and standard way of calculating the top of it work > > for this configuration. However, this won't work if the amount of RAM is > > different and split differently in address space. We have to fine the suitable > > window correctly. > > > > Find proper memory region for relocation by scanning MMAP SFI table in > > board_get_usable_ram_top() callback. > > > > According to the address map documentation the Main Memory is guaranteed to lie > > in the 0..2 GB range, that's why we limit search by this range. > > > > Fixes: e71de54a4943 ("x86: Add Intel Tangier support") > > Signed-off-by: Andy Shevchenko > > --- > > arch/x86/cpu/tangier/sdram.c | 43 ++++++++++++++++++++++++++++++++++++ > > 1 file changed, 43 insertions(+) > > Reviewed-by: Simon Glass Thanks! Bin, can we get this, somehow important fix, into v2021.01 release? And any comments about the rest tangier/edison patches? -- With Best Regards, Andy Shevchenko