On Thu, Jan 19, 2023 at 09:38:17AM +0100, Sjoerd Simons wrote: > Since commit 06d514d77c37 ("lmb: consider EFI memory map") the EFI regions > are also pushed into the lmb if EFI_LOADER is enabled (which is by > default on most system). Which can cause the number of entries to go > over the maximum as it's default is only 8. > > Specifically i ran into this case on an TI am62 which has an fdt with > 4 reserved regions (in practice 3 lmb entries due to adjecent ranges). > > As this is likely to impact more devices bump the default max > regions to 16 so there is a bit more slack. > > Signed-off-by: Sjoerd Simons <sjo...@collabora.com>
As Philippe Schenker explained, I was mis-interpreting what my attempted solution would do, so I've reverted that and this patch here makes the most sense to use as a general fix. As such I've now applied this to u-boot/master. -- Tom
signature.asc
Description: PGP signature