On Sat, Dec 29, 2007 at 12:54:08AM -0800, David Miller wrote: > From: Adrian Bunk <bunk@xxxxxxxxxx> > Date: Sat, 29 Dec 2007 10:48:46 +0200 > > > On Sat, Dec 29, 2007 at 12:14:11AM -0800, David Miller wrote: > > > That's why I'm not worried about this issue and it's not critical at > > > all. > > > > If a module calls sunserial_console_match() that's an Oops. > > That's true. > > I'm trying to figure out a way to fix this. #ifdef FOO_CONSOLE around the sunserial_console_match() calls in the drivers should work. If you consider this too many #ifdef's, an alternative solution would be doing the following in drivers/serial/suncore.h: #ifndef MODULE extern int sunserial_console_match(struct console *, struct device_node *, struct uart_driver *, int); #else static inline int sunserial_console_match(struct console *, struct device_node *, struct uart_driver *, int); { return 0; } #endif cu Adrian -- "Is there not promise of rain?" Ling Tan asked suddenly out of the darkness. There had been need of rain for many days. "Only a promise," Lao Er said. Pearl S. Buck - Dragon Seed - To unsubscribe from this list: send the line "unsubscribe sparclinux" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html