Re: [PATCH 1/3] Bump LMB_MAX_REGIONS default to 16

2023-02-08 Thread Tom Rini
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

Re: [PATCH 1/3] Bump LMB_MAX_REGIONS default to 16

2023-01-27 Thread Tom Rini
On Fri, Jan 27, 2023 at 08:55:42AM +0100, Sjoerd Simons wrote: > On Thu, 2023-01-26 at 13:28 -0500, Tom Rini wrote: > > On Thu, Jan 19, 2023 at 09:38:17AM +0100, Sjoerd Simons wrote: > > > > > > As this is likely to impact more devices bump the default max > > > regions to 16 so there is a bit mor

Re: [PATCH 1/3] Bump LMB_MAX_REGIONS default to 16

2023-01-26 Thread Sjoerd Simons
On Thu, 2023-01-26 at 13:28 -0500, Tom Rini wrote: > On Thu, Jan 19, 2023 at 09:38:17AM +0100, Sjoerd Simons wrote: > > > > 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 > > I prefer to use: > h

Re: [PATCH 1/3] Bump LMB_MAX_REGIONS default to 16

2023-01-26 Thread Tom Rini
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

[PATCH 1/3] Bump LMB_MAX_REGIONS default to 16

2023-01-19 Thread Sjoerd Simons
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