On Mon, Oct 07, 2013 at 09:44:32AM -0300, Fabio Estevam wrote: > From: Sascha Hauer <s.hauer@xxxxxxxxxxxxxx> > > Commit f7d2c0bbdb (serial: i.MX: evaluate linux,stdout-path property) adds a > call to add_preferred_console() into drivers/tty/serial/imx.c. The file can be > built as module, yet add_preferred_console() is not exported. > > Fix the build failure by disabling module build for the i.MX driver. > > Reported-by: Guenter Roeck <linux@xxxxxxxxxxxx> > Signed-off-by: Sascha Hauer <s.hauer@xxxxxxxxxxxxxx> > Signed-off-by: Fabio Estevam <fabio.estevam@xxxxxxxxxxxxx> > --- > drivers/tty/serial/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/tty/serial/Kconfig b/drivers/tty/serial/Kconfig > index 701ca60..47a05fd 100644 > --- a/drivers/tty/serial/Kconfig > +++ b/drivers/tty/serial/Kconfig > @@ -551,7 +551,7 @@ config BFIN_UART3_CTSRTS > Enable hardware flow control in the driver. > > config SERIAL_IMX > - tristate "IMX serial port support" > + bool "IMX serial port support" No, please don't force this driver to be built into the kernel. Do the same as other drivers, and have a SERIAL_IMX_CONSOLE option, that splits the code for the console layer out of the driver to only be built when this option is selected. Lots of other serial drivers do this, and they depend on when the driver is built into the kernel. Otherwise I, and lots of other people (and distros), will NOT build this driver and test it out. sorry, I can't take this patch, and for now, I'm going to revert the commit that is causing the build error, as we can't live with that for now. greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-serial" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
