Re: [Xen-devel] [PATCH] ns16550: setup default lsr_mask for DT systems too (Was: Re: [PATCH V8 RESEND] ns16550: Add support for UART present in Broadcom TruManage capable NetXtreme chips)

[Chen Baozi <baozich@xxxxxxxxx>]

On Mar 21, 2014, at 1:04, Ian Campbell <Ian.Campbell@xxxxxxxxxx> wrote:

> On Thu, 2014-03-20 at 16:47 +0000, Ian Campbell wrote:
>> Unfortunately this patch breaks the console on the arm64 x-gene
>> platform, it drops a fairly large subset of the characters.
>> 
>> I'm not sure why yet.
> 
> The simple fix is below.
> 
> I'm going to take a look at making ns16550_init and ns16550_uart_dt_init
> share a common function for setting up the defaults, but for now I'd
> appreciate getting this bandaid in.
> 
> Ian.
> 
> ------8<----------------
> 
> From 4fc6d9c71ae6b40940040910e27ba0f5272a2f27 Mon Sep 17 00:00:00 2001
> From: Ian Campbell <ian.campbell@xxxxxxxxxx>
> Date: Thu, 20 Mar 2014 17:02:52 +0000
> Subject: [PATCH] ns16550: setup default lsr_mask for DT systems too
> 
> Signed-off-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
Tested-By: Chen Baozi <baozich@xxxxxxxxx>

> ---
> xen/drivers/char/ns16550.c |    2 ++
> 1 file changed, 2 insertions(+)
> 
> diff --git a/xen/drivers/char/ns16550.c b/xen/drivers/char/ns16550.c
> index 429d786..2dd32b2 100644
> --- a/xen/drivers/char/ns16550.c
> +++ b/xen/drivers/char/ns16550.c
> @@ -1129,6 +1129,8 @@ static int __init ns16550_uart_dt_init(struct 
> dt_device_node *dev,
>     uart->stop_bits = 1;
>     /* Default is no transmit FIFO. */
>     uart->fifo_size = 1;
> +    /* Default lsr_mask = UART_LSR_THRE */
> +    uart->lsr_mask = UART_LSR_THRE;
> 
>     res = dt_device_get_address(dev, 0, &uart->io_base, &io_size);
>     if ( res )
> -- 
> 1.7.10.4
> 
> 
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxx
> http://lists.xen.org/xen-devel


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel