On Tue, Feb 14, 2017 at 03:56:43PM -0600, Adam Ford wrote: > On Feb 14, 2017 3:10 PM, "Tom Rini" <tr...@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
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot