Hi Albert, On 04/16/2013 01:36 PM, Albert ARIBAUD wrote: > Hi Andreas, > > On Tue, 16 Apr 2013 12:14:09 +0200, Andreas Bießmann > <andreas.de...@googlemail.com> wrote: > >> This patch also does some minor coding style cleanups. >> >> Signed-off-by: Andreas Bießmann <andreas.de...@googlemail.com> >> >> --- >> arch/arm/lib/board.c | 3 ++- >> arch/avr32/lib/board.c | 3 ++- >> arch/blackfin/lib/board.c | 3 ++- >> arch/m68k/lib/board.c | 3 ++- >> arch/microblaze/lib/board.c | 5 +++-- >> arch/mips/lib/board.c | 3 ++- >> arch/nds32/lib/board.c | 3 ++- >> arch/nios2/lib/board.c | 5 +++-- >> arch/openrisc/lib/board.c | 3 ++- >> arch/powerpc/lib/board.c | 4 ++-- >> arch/sandbox/lib/board.c | 3 ++- >> arch/sh/lib/board.c | 3 ++- >> arch/sparc/lib/board.c | 3 ++- >> arch/x86/lib/board.c | 3 ++- >> common/board_f.c | 3 ++- >> common/spl/spl.c | 3 ++- >> drivers/mtd/nand/mxc_nand_spl.c | 3 ++- >> 17 files changed, 36 insertions(+), 20 deletions(-)
<snip> > > Apart from Stefan's comments: considered standalone, doesn't this patch > have zero effect? I suspect you intend to use this in a later patch, > and if so, I would suggest grouping these in a series so that the > subsequent patch(es) explain this one. I would like to use a specialized hang() for a not mainlined board, thus there will no more patches using this currently. Best regards Andreas Bießmann _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
