[U-Boot] Simple Malloc Breaks omap3_logic

[Tom Rini <trini@konsulko.com>]

On Tue, Feb 14, 2017 at 03:56:43PM -0600, Adam Ford wrote:
> On Feb 14, 2017 3:10 PM, "Tom Rini" <trini@konsulko.com> wrote:
> 
> On Tue, Feb 14, 2017 at 03:03:44PM -0600, Adam Ford wrote:
> 
> > Tom,
> >
> > I noticed there was an update to the omap3_logic_defconfig to use Simple
> Malloc
> >
> > http://git.denx.de/?p=u-boot.git;a=commit;h=0959649dc6d9e6a371617abd3b0363
> 0c5d4d5a72
> >
> >
> > I didn't see anything in my inbox indicating the patch, I only noticed
> > it because I pulled the latest from the trunk.  Unfortunately, this
> > patch breaks the board.
> >
> > (ie. MLO doesn't load U-Boot)
> >
> > See Log:
> >
> > U-Boot SPL 2017.03-rc2-00008-g2ebb842-dirty (Feb 14 2017 - 14:54:59)
> > Trying to boot from MMC1
> > reading args
> > spl_load_image_fat_os: error reading image args, err - -1
> > reading u-boot.img
> > reading u-boot.img
> >
> > (then it just hangs)
> >
> > Removing this line from the defconfig returns the board to a functional
> state.
> >
> > Is there an alternative that we can explore to give you what you need?
> >
> > I am able to build with gcc version 5.4.0 , but I can help you do some
> > testing if you need.
> 
> Oh, sorry, I thought I had made sure to copy all of the required logic
> to have simple malloc work.  I think it's highly likely I forgot to make
> sure that CONFIG_SYS_MALLOC_F_LEN is also set and that's why it all went
> south.  Sorry again!
> 
> 
> That's OK, I just hope I can be included in changes so I can test them. I
> do appreciate the help in making the code and user experience better.
> 
> Can you send me an updated patch so I can test your change?

Try this please:
diff --git a/configs/omap3_logic_defconfig b/configs/omap3_logic_defconfig
index fe762c0a6b04..9d1304f08e49 100644
--- a/configs/omap3_logic_defconfig
+++ b/configs/omap3_logic_defconfig
@@ -1,5 +1,6 @@
 CONFIG_ARM=y
 CONFIG_OMAP34XX=y
+CONFIG_SYS_MALLOC_F_LEN=0x2000
 CONFIG_TARGET_OMAP3_LOGIC=y
 CONFIG_SYS_EXTRA_OPTIONS="NAND"
 CONFIG_SYS_CONSOLE_INFO_QUIET=y

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170214/43ed7a69/attachment.sig>