Re: [PATCH] serial: 8250_dw: Avoid pslverr on reading empty receiver fifo

From: Andy Shevchenko
Date: Wed Jul 13 2022 - 06:19:46 EST


On Wed, Jul 13, 2022 at 10:13 AM VAMSHI GAJJELA
<vamshigajjela@xxxxxxxxxx> wrote:
> On Tue, Jul 12, 2022 at 6:56 PM Andy Shevchenko
> <andy.shevchenko@xxxxxxxxx> wrote:
> > On Tue, Jul 12, 2022 at 3:16 PM Vamshi Gajjela <vamshigajjela@xxxxxxxxxx> wrote:

...

> > > + lsr = p->serial_in(p, UART_LSR);
> >
> > The only caller of this function already has the lsr value, why you
> > can't (re)use it?
> lsr is not read before, caller function (dw8250_check_lcr) reads lcr.

I see, thanks for elaboration.

--
With Best Regards,
Andy Shevchenko