Re: [PATCH 1/5] x86: fsp: Allow skipping init code when chain loading

2020-02-05 Thread Simon Glass
Hi Bin, On Mon, 3 Feb 2020 at 21:52, Bin Meng wrote: > > Hi Simon, > > On Tue, Feb 4, 2020 at 1:15 AM Simon Glass wrote: > > > > Hi Bin, > > > > On Mon, 3 Feb 2020 at 10:10, Bin Meng wrote: > > > > > > Hi Simon, > > > > > > On Tue, Feb 4, 2020 at 12:20 AM Simon Glass wrote: > > > > > > > > Hi

Re: [PATCH 1/5] x86: fsp: Allow skipping init code when chain loading

2020-02-03 Thread Bin Meng
Hi Simon, On Tue, Feb 4, 2020 at 1:15 AM Simon Glass wrote: > > Hi Bin, > > On Mon, 3 Feb 2020 at 10:10, Bin Meng wrote: > > > > Hi Simon, > > > > On Tue, Feb 4, 2020 at 12:20 AM Simon Glass wrote: > > > > > > Hi Bin, > > > > > > On Mon, 3 Feb 2020 at 04:05, Bin Meng wrote: > > > > > > > > Hi

Re: [PATCH 1/5] x86: fsp: Allow skipping init code when chain loading

2020-02-03 Thread Simon Glass
Hi Bin, On Mon, 3 Feb 2020 at 10:10, Bin Meng wrote: > > Hi Simon, > > On Tue, Feb 4, 2020 at 12:20 AM Simon Glass wrote: > > > > Hi Bin, > > > > On Mon, 3 Feb 2020 at 04:05, Bin Meng wrote: > > > > > > Hi Simon, > > > > > > On Sun, Dec 22, 2019 at 12:13 AM Simon Glass wrote: > > > > > > > > W

Re: [PATCH 1/5] x86: fsp: Allow skipping init code when chain loading

2020-02-03 Thread Bin Meng
Hi Simon, On Tue, Feb 4, 2020 at 12:20 AM Simon Glass wrote: > > Hi Bin, > > On Mon, 3 Feb 2020 at 04:05, Bin Meng wrote: > > > > Hi Simon, > > > > On Sun, Dec 22, 2019 at 12:13 AM Simon Glass wrote: > > > > > > When U-Boot is no the first-stage bootloader much of this code is not > > > needed

Re: [PATCH 1/5] x86: fsp: Allow skipping init code when chain loading

2020-02-03 Thread Simon Glass
Hi Bin, On Mon, 3 Feb 2020 at 04:05, Bin Meng wrote: > > Hi Simon, > > On Sun, Dec 22, 2019 at 12:13 AM Simon Glass wrote: > > > > When U-Boot is no the first-stage bootloader much of this code is not > > needed and can break booting. Add checks for this to the FSP code. > > > > Rather than chec

Re: [PATCH 1/5] x86: fsp: Allow skipping init code when chain loading

2020-02-03 Thread Bin Meng
Hi Simon, On Sun, Dec 22, 2019 at 12:13 AM Simon Glass wrote: > > When U-Boot is no the first-stage bootloader much of this code is not > needed and can break booting. Add checks for this to the FSP code. > > Rather than checking for the amount of available SDRAM, just use 1GB in > this situation

[PATCH 1/5] x86: fsp: Allow skipping init code when chain loading

2019-12-21 Thread Simon Glass
When U-Boot is no the first-stage bootloader much of this code is not needed and can break booting. Add checks for this to the FSP code. Rather than checking for the amount of available SDRAM, just use 1GB in this situation, which should be safe. Using 2GB may run into a memory hole on some SoCs.