All of lore.kernel.org
 help / color / mirror / Atom feed
From: Serge Semin <Sergey.Semin@baikalelectronics.ru>
To: Johan Hovold <johan@kernel.org>
Cc: Serge Semin <fancer.lancer@gmail.com>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Jiri Slaby <jirislaby@kernel.org>,
	Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
	<linux-serial@vger.kernel.org>, <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 0/3] serial: 8250: fix racy uartclk update
Date: Sat, 16 Oct 2021 18:25:21 +0300	[thread overview]
Message-ID: <20211016152521.zmzhtkswhphos4q7@mobilestation> (raw)
In-Reply-To: <20211015191000.hspyxgkwwd47w4nl@mobilestation>

Hello Johan

On Fri, Oct 15, 2021 at 10:10:00PM +0300, Serge Semin wrote:
> On Fri, Oct 15, 2021 at 01:14:19PM +0200, Johan Hovold wrote:
> > Here's a fix for the asynchronous uartclk-update hack which was added
> > last year and a couple of related clean ups.
> 
> Yeah, that was very unfortunate patchset. Thank you very much for the fix.
> I'll test it out tomorrow.
> 
> -Sergey

The series has been tested on Baikal-T1 SoC with two UART ports
feed with a common reference clock. It's working well. Thanks again
for fixing the problem. For the whole series
Reviewed-by: Serge Semin <fancer.lancer@gmail.com>
Tested-by: Serge Semin <fancer.lancer@gmail.com>

-Sergey

> 
> > 
> > Johan
> > 
> > 
> > Johan Hovold (3):
> >   serial: 8250: fix racy uartclk update
> >   serial: 8250: rename unlock labels
> >   serial: 8250_dw: drop bogus uartclk optimisation
> > 
> >  drivers/tty/serial/8250/8250_dw.c   | 11 ++++------
> >  drivers/tty/serial/8250/8250_port.c | 31 ++++++++++++++++++++---------
> >  2 files changed, 26 insertions(+), 16 deletions(-)
> > 
> > -- 
> > 2.32.0
> > 

  reply	other threads:[~2021-10-16 15:25 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-10-15 11:14 [PATCH 0/3] serial: 8250: fix racy uartclk update Johan Hovold
2021-10-15 11:14 ` [PATCH 1/3] " Johan Hovold
2021-10-15 14:23   ` Andy Shevchenko
2021-10-15 11:14 ` [PATCH 2/3] serial: 8250: rename unlock labels Johan Hovold
2021-10-15 14:24   ` Andy Shevchenko
2021-10-16 15:40   ` Serge Semin
2021-10-15 11:14 ` [PATCH 3/3] serial: 8250_dw: drop bogus uartclk optimisation Johan Hovold
2021-10-15 14:25   ` Andy Shevchenko
2021-10-15 19:10 ` [PATCH 0/3] serial: 8250: fix racy uartclk update Serge Semin
2021-10-16 15:25   ` Serge Semin [this message]
2021-10-18  6:29     ` Johan Hovold

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20211016152521.zmzhtkswhphos4q7@mobilestation \
    --to=sergey.semin@baikalelectronics.ru \
    --cc=andriy.shevchenko@linux.intel.com \
    --cc=fancer.lancer@gmail.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=jirislaby@kernel.org \
    --cc=johan@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-serial@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.