Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Sherry Sun <sherry.sun@nxp.com>
To:     gregkh@linuxfoundation.org, jirislaby@kernel.org,
        tomonori.sakita@sord.co.jp, atsushi.nemoto@sord.co.jp
Cc:     linux-serial@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux-imx@nxp.com
Subject: [PATCH] tty: serial: fsl_lpuart: Clear the error flags by writing 1 for lpuart32 platforms
Date:   Mon, 31 Jul 2023 09:50:53 +0800
Message-Id: <20230731015053.15150-1-sherry.sun@nxp.com>
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?IUjNX4rqQlg6MXRJkKEpz61iGDySRbh5uirwpV4VcrGv+nuXureGr2gcLH46?=
 =?us-ascii?Q?OWR267vmaTv0xUVwkAS4JiO++5vIgIiPrZh8lK/j0V/ffKoY4T7cY7JqsW1V?=
 =?us-ascii?Q?+og8Tilsp00oYpq6EtXtbsT0FpDwQg1qbUDombmbaq+IsNqGwF7csejaNYto?=
 =?us-ascii?Q?eJGG0TiRiD+iz8FBfsi4jqHPKVyeNoTtM0eNiRSrvu8qrgzRu9y8sYbegjQ4?=
 =?us-ascii?Q?ljeu0xDb0NQH9WIfPU/Siym2iW/tsVDBuILh9FnOzutU+GRuxbiBs7vf6BT+?=
 =?us-ascii?Q?UNUT/FgVG1YEGSIju2Ld6VY24b7TbBcMab1W57UQGO949/c4V8IrRS+l0SCU?=
 =?us-ascii?Q?0iyLKqcF54sCcbl1KLiKTo2J+uyVEvpf0CEnzsosrRDbV7WGaWSc1DLfu8i9?=
 =?us-ascii?Q?LXqMDtZBfmF2+MZUcB1KqDa+R8hluHZQPfVxTBrCeVx0LMGM/5LRO7RrKLPD?=
 =?us-ascii?Q?OZOVk0/TkADaAW7lvMjjtMlBActdTXv9K1IV4RAQBlN/sU4e0ztjgaXsTvzh?=
 =?us-ascii?Q?1EUEvv37bAJWOz77oywwyoGVtCdt8OxhPQq+VyTZfMBravG/dz43c9CP2d2B?=
 =?us-ascii?Q?oBgqoU9sAjCu0xSc1hS3NfjDvniuUZUoDETcZBZIQVSCyonJd5ufv1ERp3Bc?=
 =?us-ascii?Q?pgQUm0rF8Bo5HyLwDv7b+Il0xJuHu+fYQgj6hp6fBGhZuK28J9yzkQB2NNDw?=
 =?us-ascii?Q?OzCriDir98MmJpvIxg+8OCjjnDZO4YSoFQr06k4i+AV1Xq0tU+iN/77xFKft?=
 =?us-ascii?Q?fnv9tqWEKzIDJbtnRKINdtYnQUtLNjgxygCZHwbZYNyck3RKgn7+/+GmyA/5?=
 =?us-ascii?Q?Hb9qmEk1hRgBix5Ij96QixiBKFUaTDdiK2riz0MpK4KeaWEZnCxPulz4cu9a?=
 =?us-ascii?Q?Lap1OET5e5QnPcBQ+cxHhppvPPr14DOMx/tajpu1Q/tucQmIvRlidCCIE+nQ?=
 =?us-ascii?Q?aRyjSP/3NfFtKm6Xf2Fcf+wINnPwlKBGGgmLEOe3S2QCzYsVd2mTFvxfkiWh?=
 =?us-ascii?Q?saAGi8UKO+A07C6DXkWv73nFEDVj/UwmoZmIu8UjtQfp28WkwR5//5p7Bobz?=
 =?us-ascii?Q?I37X4m3C9gwwNiSmYbX2aM/ZBsGKBQYGwoSXclK/tfgRgpjXSUxQENmM2wUo?=
 =?us-ascii?Q?XgV48iSr+HZUiJeNAJxhRoDdkdehpOSgacCsZaPkyPqMxjKdlsGAPqYIOaK5?=
 =?us-ascii?Q?mh42MvpwSiIhjnb0HFapnGJ9X2zIQL4ArA/Hr2WaCXtNwvowHx24cTVyqJQC?=
 =?us-ascii?Q?Tt8SNosbMDs4wozTAA8/yuWtREfueIsPluoV2l+p+TgTLXtjOXsJb741zlFj?=
 =?us-ascii?Q?jeVUsjvgJZQCh0u9BOrNhfwGP3IjfO1e4piQkRczn8skOQHh1Msy72HbdqQ3?=
 =?us-ascii?Q?DsG2mLRSfIQ4nZN0UioKoOFVKcAtckjNvVahWUd1gEQCKqoE9QWl1eawZ7kn?=
 =?us-ascii?Q?hIycEiPLCGH6cuKqE4w1Bh7VL/IBr9t828yfbZZdrRsxrhywa1ecKzzz17r9?=
 =?us-ascii?Q?27XiKIAb8Chu0y/giOYvRRTcg9jqqNDoQAvAk/gbpEZZFFwX0A5OZj6ubB8l?=
 =?us-ascii?Q?J8hFP7cO8Pw4wDVQXMisBOw0z3wOkA5QF/uOt7jp?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 5112a0af-a7bd-46c8-921f-08db91697f70
X-MS-Exchange-CrossTenant-AuthSource: AS8PR04MB8404.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 31 Jul 2023 01:57:30.2332
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: aj5Z+rr2KUC0vThFPGSfkZC30M3qlNK60DY1LSgdcPeCKY1kPSCF+ZUFEPJ2z2it3pDb6BrlitE/vMuX5SwxYA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB9PR04MB8393
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_BLOCKED,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Do not read the data register to clear the error flags for lpuart32
platforms, the additional read may cause the receive FIFO underflow
since the DMA has already read the data register.
Now all lpuart32 platforms support write 1 to clear those error bits,
let's use this method to better clear the error flags.

Fixes: 42b68768e51b ("serial: fsl_lpuart: DMA support for 32-bit variant")
Signed-off-by: Sherry Sun <sherry.sun@nxp.com>
---
 drivers/tty/serial/fsl_lpuart.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/tty/serial/fsl_lpuart.c b/drivers/tty/serial/fsl_lpuart.c
index f6644c5989d3..f72e1340b47d 100644
--- a/drivers/tty/serial/fsl_lpuart.c
+++ b/drivers/tty/serial/fsl_lpuart.c
@@ -1120,8 +1120,8 @@ static void lpuart_copy_rx_to_tty(struct lpuart_port *sport)
 		unsigned long sr = lpuart32_read(&sport->port, UARTSTAT);
 
 		if (sr & (UARTSTAT_PE | UARTSTAT_FE)) {
-			/* Read DR to clear the error flags */
-			lpuart32_read(&sport->port, UARTDATA);
+			/* Clear the error flags */
+			lpuart32_write(&sport->port, sr, UARTSTAT);
 
 			if (sr & UARTSTAT_PE)
 				sport->port.icount.parity++;
-- 
2.17.1