Re: [PATCH] uart:8250: change lock order in serial8250_do_startup()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Sergey Senozhatsky <sergey.senozhatsky@xxxxxxxxx>
Subject: Re: [PATCH] uart:8250: change lock order in serial8250_do_startup()
From: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
Date: Fri, 14 Aug 2020 18:40:37 +0300
Cc: Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx>, Guenter Roeck <linux@xxxxxxxxxxxx>, Tony Lindgren <tony@xxxxxxxxxxx>, Kurt Kanzenbach <kurt@xxxxxxxxxxxxx>, Raul Rangel <rrangel@xxxxxxxxxx>, Petr Mladek <pmladek@xxxxxxxx>, Steven Rostedt <rostedt@xxxxxxxxxxx>, John Ogness <john.ogness@xxxxxxxxxxxxx>, linux-kernel <linux-kernel@xxxxxxxxxxxxxxx>, linux-serial@xxxxxxxxxxxxxxx
In-reply-to: <20200814112940.GB582@jagdpanzerIV.localdomain>
Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo
References: <20200814013802.357412-1-sergey.senozhatsky@gmail.com> <20200814095928.GK1891694@smile.fi.intel.com> <20200814112940.GB582@jagdpanzerIV.localdomain>

On Fri, Aug 14, 2020 at 08:29:40PM +0900, Sergey Senozhatsky wrote:
> On (20/08/14 12:59), Andy Shevchenko wrote:

...

> > I think we can stick with newer:
> >
> > 		if (port->irqflags & IRQF_SHARED)
> 
> I'll take a look.

Thanks!

One more thing, perhaps update prefix to be 'serial: 8250:'.

-- 
With Best Regards,
Andy Shevchenko

References:
- [PATCH] uart:8250: change lock order in serial8250_do_startup()
  - From: Sergey Senozhatsky
- Re: [PATCH] uart:8250: change lock order in serial8250_do_startup()
  - From: Andy Shevchenko
- Re: [PATCH] uart:8250: change lock order in serial8250_do_startup()
  - From: Sergey Senozhatsky

Prev by Date: Re: [PATCH] serial: sh-sci: Make sure status register SCxSR is read in correct sequence
Next by Date: Re: [PATCH] dt-bindings: Whitespace clean-ups in schema files
Previous by thread: Re: [PATCH] uart:8250: change lock order in serial8250_do_startup()
Next by thread: Re: [PATCH] uart:8250: change lock order in serial8250_do_startup()
Index(es):
- Date
- Thread

[Index of Archives] [Kernel Newbies] [Security] [Netfilter] [Bugtraq] [Linux PPP] [Linux FS] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Samba] [Video 4 Linux] [Linmodem] [Device Mapper] [Linux Kernel for ARM]