Hi Andreas, On Fri, Apr 19, 2013 at 1:48 AM, Andreas Bießmann <andreas.de...@googlemail.com> wrote: > This series started with intention to make a hung board visualize the > (end-)user that it does hang(). The first approach was to make all the > different hang() functions weak and provide board specific versions to > implement the 'visualization' there. > After some discussion it turned out, that there is still an interface to > visualize such states to the user: the 'bootstage' stuff. > > BTW: > This is my first try sending a series with patman. All I can say is: > patman is really cool ;) > > The 'microblaze: fix style in board.c' is an leftover from the first approach. > > The 'nios2: fix style in board.c' is an leftover from the first approach. > > The 'lib: consolidate hang()' is build tested on arm and avr32. It may > produces errors on other arches like ppc cause the previous SPL inlined hang() > function was also removed. We need to discuss if we always pull in the > libgeneric unconditional into SPL or provide a second hang() for those boards > which do not link against libgeneric in SPL. > > Since v2 got a complete (compile-)test on powerpc by Wolfgang I think we can > stick with the current solution. v3 includes some minor changes mentioned in > the changelog of respective patches.
This series tested on snow with CONFIG_SYS_GENERIC_BOARD enabled. No warnings or problems noted and hang() still works. Tested-by: Simon Glass <s...@chromium.org> Acked-by: Simon Glass <s...@chromium.org> > > Changes in v3: > * shorten error string for 'no FDT' > * remove unnecessarily added linux/compiler.h > * fix typo in new hang() docu > * use simplified macro for SPL_LIBCOMMON_SUPPORT as suggested by Benoît > > Changes in v2: > * delete all hang() and provide generic lib/hang.c > > Andreas Bießmann (5): > microblaze: fix style in board.c > nios2: fix style in board.c. > mx31pdk: add CONFIG_SPL_LIBGENERIC_SUPPORT > tx25: add CONFIG_SPL_LIBGENERIC_SUPPORT > lib: consolidate hang() > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot