[U-Boot] [PATCH] Create configuration option for restricted ns16550 functions

[Ron Madrid]

--- On Fri, 1/30/09, Ron Madrid <ron_madrid at sbcglobal.net> wrote:

> From: Ron Madrid <ron_madrid at sbcglobal.net>
> Subject: [PATCH] Create configuration option for restricted ns16550 functions
> To: u-boot at lists.denx.de
> Cc: "Ron Madrid" <ron_madrid at sbcglobal.net>
> Date: Friday, January 30, 2009, 3:45 PM
> This patch will create a configuration option for a minimum
> configuration for
> the ns16550 serial driver at drivers/serial/ns16550.c which
> will exclude all
> functions with exception of NS16550_putc and NS16550_init. 
> This will be used
> primarily to save space and remove unused code from builds
> in which space is
> limited.
> 
> Signed-off-by: Ron Madrid <ron_madrid at sbcglobal.net>
> ---
>  drivers/serial/ns16550.c |    5 ++++-
>  1 files changed, 4 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/serial/ns16550.c
> b/drivers/serial/ns16550.c
> index 93c2243..3e2a3c2 100644
> --- a/drivers/serial/ns16550.c
> +++ b/drivers/serial/ns16550.c
> @@ -39,6 +39,7 @@ void NS16550_init (NS16550_t com_port,
> int baud_divisor)
>  #endif
>  }
>  
> +#ifndef CONFIG_NS16550_MIN_FUNCTIONS
>  void NS16550_reinit (NS16550_t com_port, int baud_divisor)
>  {
>  	com_port->ier = 0x00;
> @@ -53,6 +54,7 @@ void NS16550_reinit (NS16550_t com_port,
> int baud_divisor)
>  	com_port->dlm = (baud_divisor >> 8) & 0xff;
>  	com_port->lcr = LCRVAL;
>  }
> +#endif /* CONFIG_NS16550_MIN_FUNCTIONS */
>  
>  void NS16550_putc (NS16550_t com_port, char c)
>  {
> @@ -60,6 +62,7 @@ void NS16550_putc (NS16550_t com_port,
> char c)
>  	com_port->thr = c;
>  }
>  
> +#ifndef CONFIG_NS16550_MIN_FUNCTIONS
>  char NS16550_getc (NS16550_t com_port)
>  {
>  	while ((com_port->lsr & LSR_DR) == 0) {
> @@ -75,5 +78,5 @@ int NS16550_tstc (NS16550_t com_port)
>  {
>  	return ((com_port->lsr & LSR_DR) != 0);
>  }
> -
> +#endif /* CONFIG_NS16550_MIN_FUNCTIONS */
>  #endif
> -- 
> 1.5.5.1

Did this get looked at?  I don't remember seeing anything about it.  If it
is still being looked at, just let me know.

Thanks,

Ron