Re: [PATCH 1/2] serial: imx: Introduce timeout when waiting on transmitter empty

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

To: Esben Haabendal <esben@xxxxxxxxxx>
Subject: Re: [PATCH 1/2] serial: imx: Introduce timeout when waiting on transmitter empty
From: Fabio Estevam <festevam@xxxxxxxxx>
Date: Fri, 5 Apr 2024 14:38:48 -0300
Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>, Jiri Slaby <jirislaby@xxxxxxxxxx>, Shawn Guo <shawnguo@xxxxxxxxxx>, Sascha Hauer <s.hauer@xxxxxxxxxxxxxx>, Pengutronix Kernel Team <kernel@xxxxxxxxxxxxxx>, Marc Kleine-Budde <mkl@xxxxxxxxxxxxxx>, linux-kernel@xxxxxxxxxxxxxxx, linux-serial@xxxxxxxxxxxxxxx, imx@xxxxxxxxxxxxxxx, linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
In-reply-to: <76cf9ce9cbf9dcdf78bc00ce7a919db1776ebce1.1712309058.git.esben@geanix.com>
References: <76cf9ce9cbf9dcdf78bc00ce7a919db1776ebce1.1712309058.git.esben@geanix.com>

On Fri, Apr 5, 2024 at 6:25 AM Esben Haabendal <esben@xxxxxxxxxx> wrote:
>
> By waiting at most 1 second for USR2_TXDC to be set, we avoid a potentital

s/potentital/potential

Could you elaborate on this deadlock? Have you seen it in practice?

Should a Fixes tag be added?

Follow-Ups:
- Re: [PATCH 1/2] serial: imx: Introduce timeout when waiting on transmitter empty
  - From: Esben Haabendal
- Re: [PATCH 1/2] serial: imx: Introduce timeout when waiting on transmitter empty
  - From: Sergey Organov

References:
- [PATCH 1/2] serial: imx: Introduce timeout when waiting on transmitter empty
  - From: Esben Haabendal

Prev by Date: Re: [PATCH 1/2] serial: imx: Introduce timeout when waiting on transmitter empty
Next by Date: Re: [PATCH 1/2] serial: imx: Introduce timeout when waiting on transmitter empty
Previous by thread: Re: [PATCH 1/2] serial: imx: Introduce timeout when waiting on transmitter empty
Next by thread: Re: [PATCH 1/2] serial: imx: Introduce timeout when waiting on transmitter empty
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]