Re: [PATCH] serial: i.MX: do not allow to build as module

[Greg KH <gregkh@linuxfoundation.org>]

On Mon, Oct 07, 2013 at 09:44:32AM -0300, Fabio Estevam wrote:
> From: Sascha Hauer <s.hauer@pengutronix.de>
> 
> 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@roeck-us.net>
> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
>  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