[U-Boot] Simple Malloc Breaks omap3_logic

[Adam Ford]

On Feb 14, 2017 3:10 PM, "Tom Rini" <trini at 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?

Thanks

Adam


--
Tom