linux-kernel - Re: mxs-auart broken in v6.2 and onwards

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <040064c5-c285-4e02-b545-47c5085f0068@kernel.org>
Date: Tue, 30 Jan 2024 10:38:15 +0100
From: Jiri Slaby <jirislaby@...nel.org>
To: Emil Kronborg <emil.kronborg@...tonmail.com>, linux-serial@...r.kernel.org
Cc: linux-kernel@...r.kernel.org
Subject: Re: mxs-auart broken in v6.2 and onwards

Hi,

On 27. 01. 24, 22:35, Emil Kronborg wrote:
> After updating Linux on an i.MX28 board, serial communication over AUART
> broke. When I TX from the board and measure on the TX pin, it seems like
> the HW fifo is not emptied before the transmission is stopped. I
> bisected the bad commit to be 2d141e683e9a ("tty: serial: use
> uart_port_tx() helper"). Since it concerns multiple drivers, simply
> reverting it is not feasible. One solution would be to effectively
> revert the commit for just mxs-auart.c, but maybe you have a better
> idea? Any pointers is appreciated.

Hm, the tx stop handling is weird throughout mxs. What about the 
attached patch?

thanks,
-- 
js
suse labs

View attachment "0001-mxs-fix.patch" of type "text/x-patch" (2848 bytes)