Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756580AbbGVMjG (ORCPT ); Wed, 22 Jul 2015 08:39:06 -0400 Received: from mail-qk0-f181.google.com ([209.85.220.181]:34175 "EHLO mail-qk0-f181.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752271AbbGVMjE (ORCPT ); Wed, 22 Jul 2015 08:39:04 -0400 Message-ID: <55AF8EE3.1040406@hurleysoftware.com> Date: Wed, 22 Jul 2015 08:38:59 -0400 From: Peter Hurley User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.8.0 MIME-Version: 1.0 To: Noam Camus CC: linux-kernel@vger.kernel.org, linux-serial@vger.kernel.org, Alexey.Brodkin@synopsys.com, vgupta@synopsys.com, gregkh@linuxfoundation.org, jslaby@suse.com Subject: Re: [PATCH 3/4] serial: 8250_dw: Add UPF_FIXED_TYPE to flags References: <1437557699-25722-1-git-send-email-noamc@ezchip.com> <1437557699-25722-2-git-send-email-noamc@ezchip.com> <1437557699-25722-3-git-send-email-noamc@ezchip.com> <1437557699-25722-4-git-send-email-noamc@ezchip.com> In-Reply-To: <1437557699-25722-4-git-send-email-noamc@ezchip.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1759 Lines: 50 Hi Noam, On 07/22/2015 05:34 AM, Noam Camus wrote: > From: Noam Camus > > Always add UPF_FIXED_TYPE to flags so autoconf() will be skipped. > We do that since autoconf() performs many writes to LCR that cause > BUSY interrupt. > The problem with such interrupt is that driver is not yet called to > request_irq() and generic IRQ subsystem will mask the UART line. > > Signed-off-by: Noam Camus > --- > drivers/tty/serial/8250/8250_dw.c | 8 ++++++++ > 1 files changed, 8 insertions(+), 0 deletions(-) > > diff --git a/drivers/tty/serial/8250/8250_dw.c b/drivers/tty/serial/8250/8250_dw.c > index 1a57105..620f983 100644 > --- a/drivers/tty/serial/8250/8250_dw.c > +++ b/drivers/tty/serial/8250/8250_dw.c > @@ -362,6 +362,14 @@ static int dw8250_probe_of(struct uart_port *p, > if (has_ucv) > dw8250_setup_port(up); > > + /* Writing to LCR may cause BUSY interrupt before we > + * register the IRQ line. > + * Currently autoconf() uses several writes to LCR. > + * In order to avoid calling to autoconf() always add > + * following flag. > + */ > + p->flags |= UPF_FIXED_TYPE; Why only for devicetree DW8250's? Don't all DW8250's have this LCR "feature"? And what port type does this id DW8250's as, PORT_8250? Except with fifos, autoflow control, dma, etc.? If the port type is being fixed, then please fix it to a new port type. > + > /* if we have a valid fifosize, try hooking up DMA here */ > if (p->fifosize) { > up->dma = &data->dma; > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/