Dear "Jason Hobbs", In message <[email protected]> you wrote: > Signed-off-by: Jason Hobbs <[email protected]> > --- > 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: [email protected] You have the capacity to learn from mistakes. You'll learn a lot today. _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
