Matthias Weisser wrote: > Signed-off-by: Matthias Weisser <matthias.weis...@graf-syteco.de> > --- > common/serial.c | 3 +- > cpu/arm926ejs/jade/Makefile | 47 +++++++++ > cpu/arm926ejs/jade/reset.c | 37 ++++++++ > cpu/arm926ejs/jade/timer.c | 127 +++++++++++++++++++++++++ > include/asm-arm/arch-jade/hardware.h | 31 ++++++ > include/asm-arm/arch-jade/jade.h | 170 > ++++++++++++++++++++++++++++++++++ > include/serial.h | 3 +- > 7 files changed, 416 insertions(+), 2 deletions(-) > create mode 100644 cpu/arm926ejs/jade/Makefile > create mode 100644 cpu/arm926ejs/jade/reset.c > create mode 100644 cpu/arm926ejs/jade/timer.c > create mode 100644 include/asm-arm/arch-jade/hardware.h > create mode 100644 include/asm-arm/arch-jade/jade.h > > diff --git a/common/serial.c b/common/serial.c > index 5f9ffd7..7397608 100644 > --- a/common/serial.c > +++ b/common/serial.c > @@ -41,7 +41,8 @@ struct serial_device *__default_serial_console (void) > #elif defined(CONFIG_405GP) || defined(CONFIG_405CR) || defined(CONFIG_440) \ > || defined(CONFIG_405EP) || defined(CONFIG_405EZ) || > defined(CONFIG_405EX) \ > || defined(CONFIG_MPC5xxx) || defined(CONFIG_MPC83xx) \ > - || defined(CONFIG_MPC85xx) || defined(CONFIG_MPC86xx) > + || defined(CONFIG_MPC85xx) || defined(CONFIG_MPC86xx) \ > + || defined(CONFIG_JADE) > #if defined(CONFIG_CONS_INDEX) && defined(CONFIG_SYS_NS16550_SERIAL) > #if (CONFIG_CONS_INDEX==1) > return &eserial1_device;
Including serial here is premature. I do not see where you set up there serial devices. Please save this for a later serial-only patch. Otherwise this patch looks fine. Tom _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot