[U-Boot] [PATCH v2] Kconfig: Enlarge default SYS_MALLOC_F_LEN for AM33XX

[U-Boot] [PATCH v2] Kconfig: Enlarge default SYS_MALLOC_F_LEN for AM33XX

[Sjoerd Simons]

Since commit 8e14ba7bd524 ("gpio: omap_gpio: Add DM_FLAG_PRE_RELOC
flag") omap GPIO gets bound before relocation.  Unfortunately due to
this, on at least the beaglebone black, the pre-relocation memory pool
gets exhausted before probing the serial port. This then causes u-boot
to panic as CONFIG_REQUIRE_SERIAL_CONSOLE is set...

Resolve this by resizing the default size of the pre-relocation malloc
pool for AM335X platforms.

Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>

---

Changes in v2:
- Set the default in the toplevel kConfig file rather then the omap2plus one

 Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Kconfig b/Kconfig
index 6670913799..55dddc844d 100644
--- a/Kconfig
+++ b/Kconfig
@@ -131,6 +131,7 @@ config SYS_MALLOC_F
 config SYS_MALLOC_F_LEN
 	hex "Size of malloc() pool before relocation"
 	depends on SYS_MALLOC_F
+	default 0x1000 if AM33XX
 	default 0x400
 	help
 	  Before relocation, memory is very limited on many platforms. Still,
-- 
2.17.0

[U-Boot] [U-Boot, v2] Kconfig: Enlarge default SYS_MALLOC_F_LEN for AM33XX

[Tom Rini]

On Thu, Apr 12, 2018 at 06:09:58PM +0200, Sjoerd Simons wrote:

> Since commit 8e14ba7bd524 ("gpio: omap_gpio: Add DM_FLAG_PRE_RELOC
> flag") omap GPIO gets bound before relocation.  Unfortunately due to
> this, on at least the beaglebone black, the pre-relocation memory pool
> gets exhausted before probing the serial port. This then causes u-boot
> to panic as CONFIG_REQUIRE_SERIAL_CONSOLE is set...
> 
> Resolve this by resizing the default size of the pre-relocation malloc
> pool for AM335X platforms.
> 
> Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180416/99098fee/attachment.sig>

[U-Boot] [U-Boot, v2] Kconfig: Enlarge default SYS_MALLOC_F_LEN for AM33XX

[Tom Rini]

On Mon, Apr 16, 2018 at 09:22:51PM -0400, Tom Rini wrote:
> On Thu, Apr 12, 2018 at 06:09:58PM +0200, Sjoerd Simons wrote:
> 
> > Since commit 8e14ba7bd524 ("gpio: omap_gpio: Add DM_FLAG_PRE_RELOC
> > flag") omap GPIO gets bound before relocation.  Unfortunately due to
> > this, on at least the beaglebone black, the pre-relocation memory pool
> > gets exhausted before probing the serial port. This then causes u-boot
> > to panic as CONFIG_REQUIRE_SERIAL_CONSOLE is set...
> > 
> > Resolve this by resizing the default size of the pre-relocation malloc
> > pool for AM335X platforms.
> > 
> > Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>
> 
> Applied to u-boot/master, thanks!

That done, a patch on top of this to move ARCH_OMAP2PLUS to 0x2000
instead would be greatly appreciated for post v2018.05, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180417/22aa807d/attachment.sig>