Dear "Jason Hobbs", In message <1308853655-12407-4-git-send-email-jason.ho...@calxeda.com> you wrote: > Signed-off-by: Jason Hobbs <jason.ho...@calxeda.com> > --- > changes in v2: > - expose abortboot externally instead of using a wrapper > - expose abortboot externally when CONFIG_MENU is set > > common/main.c | 12 ++++++++---- > include/common.h | 3 +++ > 2 files changed, 11 insertions(+), 4 deletions(-) ... > +#ifdef CONFIG_MENU
This being a new CONFIG_ variable, it must be documented in the README. But as is this makes little sense, so I suggest you change the order of your patches and add the abortboot support to the menu system in a second step, after adding (and documenting) the menu stuff. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de You have the capacity to learn from mistakes. You'll learn a lot today. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot