Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753370AbbLJVlN (ORCPT ); Thu, 10 Dec 2015 16:41:13 -0500 Received: from mail-pf0-f169.google.com ([209.85.192.169]:34359 "EHLO mail-pf0-f169.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751223AbbLJVlL (ORCPT ); Thu, 10 Dec 2015 16:41:11 -0500 Subject: Re: [PATCH LINUX v4 06/13] tty: xuartps: Move request_irq to after setting up the HW To: Soren Brinkmann References: <1449376769-13369-1-git-send-email-soren.brinkmann@xilinx.com> <1449376769-13369-7-git-send-email-soren.brinkmann@xilinx.com> Cc: Greg Kroah-Hartman , Jiri Slaby , Michal Simek , linux-serial@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Moritz Fischer From: Peter Hurley Message-ID: <5669F172.6020503@hurleysoftware.com> Date: Thu, 10 Dec 2015 13:41:06 -0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.4.0 MIME-Version: 1.0 In-Reply-To: <1449376769-13369-7-git-send-email-soren.brinkmann@xilinx.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: 2281 Lines: 66 On 12/05/2015 08:39 PM, Soren Brinkmann wrote: > Request_irq() should be _after_ h/w programming, otherwise an > interrupt could be triggered and in-progress before the h/w has been > setup. Slight misunderstanding. My fault; I should have been more explicit. 1. Any setup necessary for the isr not to be confused and misdirect spurious interrupts (or hang) should be before installing the isr with request_irq() None of this code should trigger an interrupt. 2. Clear pending interrupts 3. Install the isr with request_irq() 4. Enable interrupts For extra safety, first disable interrupts before starting h/w programming. I would do the v5 series in the same order as the v3 series only up to what I reviewed. Then do another series with the remainder plus new changes, ok? Regards, Peter Hurley > Reported-by: Peter Hurley > Signed-off-by: Soren Brinkmann > --- > v4: > - this patch has been added. Thanks to Peter for pointing it out and providing > commit message > --- > drivers/tty/serial/xilinx_uartps.c | 9 ++------- > 1 file changed, 2 insertions(+), 7 deletions(-) > > diff --git a/drivers/tty/serial/xilinx_uartps.c b/drivers/tty/serial/xilinx_uartps.c > index 6ffd3bbe3e18..1e9053656610 100644 > --- a/drivers/tty/serial/xilinx_uartps.c > +++ b/drivers/tty/serial/xilinx_uartps.c > @@ -759,12 +759,7 @@ static void cdns_uart_set_termios(struct uart_port *port, > static int cdns_uart_startup(struct uart_port *port) > { > unsigned long flags; > - unsigned int retval = 0, status = 0; > - > - retval = request_irq(port->irq, cdns_uart_isr, 0, CDNS_UART_NAME, > - (void *)port); > - if (retval) > - return retval; > + unsigned int status = 0; > > spin_lock_irqsave(&port->lock, flags); > > @@ -818,7 +813,7 @@ static int cdns_uart_startup(struct uart_port *port) > > spin_unlock_irqrestore(&port->lock, flags); > > - return retval; > + return request_irq(port->irq, cdns_uart_isr, 0, CDNS_UART_NAME, port); > } > > /** > -- 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/