Hi Heiko, >> +#define GPIO_BASE (u_char *)0x30400000 >> > > should be > > #define GPIO_BASE (u_char *)CONFIG_SYS_CS3_START
Fixed, thanks. >> +#define SERIAL_PORT_BASE (u_char *)0x80000000 >> > > this should be > > #define SERIAL_PORT_BASE (u_char *)CONFIG_SYS_CS2_START Fixed, thanks. [...] >> +#define UART_MSR_DCD 0x80 /* Data Carrier Detect */ >> +#define UART_MSR_DSR 0x20 /* Data Set Ready */ >> +#define UART_MSR_CTS 0x10 /* Clear to Send */ >> > > Are this not common 16xxx UART registers? If so, I think, they > should go in include/ns16550.h ... Ok, done. I'm glad I posted this series of patches again with my name added. On the previous posting, the code unfortunately did not get this quality review ;) Cheers Detlev -- Wenn ein Kopf und ein Buch zusammenstossen und es klingt hohl; ist denn das allemal im Buche? - Lichtenberg -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: d...@denx.de _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot