Hi Stefan, > your patch 7b5611cdd12ca0cc33f994f0d4a4454788fc3124 [inka4x0: Add hardware > diagnosis functions for inka4x0] introduced all kind of new defines in the > ns16550.h header resulting in a warning on some 4xx platforms: > > [ste...@kubuntu u-boot (master)]$ ./MAKEALL ml300 > Configuring for ml300 board... > In file included from serial.c:53: > /home/stefan/git/u-boot/u-boot/include/ns16550.h:151:1: warning: "MSR_RI" > redefined > In file included from serial.c:41: > /home/stefan/git/u-boot/u-boot/include/asm/processor.h:48:1: warning: this is > the location of the previous definition > > I suggest to either remove this MSI_RI from the header again (it doesn't seem > to be used here) or rename it.
Ok, so much for the plan to add the defines not bit by bit as needed, but register-wise, while we're there. I actually would prefer to use UART_MSR_RI, as this is a UART thingy rather than to remove the define as we surely use the register, so I feel it worthwhile to have the whole definition on board. Let me cook up a rename for the UART stuff.. Cheers Detlev -- We have a live-manual. It's called emacs-de...@gnu.org. You can stick to just reading it, but you can skip to a specific chapter by simply sending an email asking for it ;-) -- Stefan Monnier -- 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