diff mbox series

[v2] Kconfig: Enlarge default SYS_MALLOC_F_LEN for AM33XX

Message ID 20180412160959.28996-1-sjoerd.simons@collabora.co.uk
State Accepted
Commit 90e407ae01838692bacfbcbcb83d0fb33f3ba7e7
Headers show
Series [v2] Kconfig: Enlarge default SYS_MALLOC_F_LEN for AM33XX | expand

Commit Message

Sjoerd Simons April 12, 2018, 4:09 p.m. UTC
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(+)

Comments

Tom Rini April 17, 2018, 1:22 a.m. UTC | #1
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
Tom Rini April 17, 2018, 4:49 p.m. UTC | #2
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
diff mbox series

Patch

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,