Page MenuHomeFreeBSD
Differential D37185

ns8250: Fix sense of LSR_TEMT FCR check
ClosedPublic
Actions

Authored by cperciva on Oct 28 2022, 4:45 AM.
Tags
None
Referenced Files
Unknown Object (File)
Nov 1 2024, 9:35 PM
Unknown Object (File)
Nov 1 2024, 9:35 PM
Unknown Object (File)
Nov 1 2024, 9:35 PM
Unknown Object (File)
Nov 1 2024, 9:17 PM
Unknown Object (File)
Sep 24 2024, 12:15 PM
Unknown Object (File)
Sep 18 2024, 10:43 PM
Unknown Object (File)
Sep 17 2024, 2:15 PM
Unknown Object (File)
Sep 16 2024, 10:53 PM
View All 38 Files
Subscribers
None

Details

Reviewers
emaste
imp
jrtc27
Commits
rG5ad8c32c722b: ns8250: Fix sense of LSR_TEMT FCR check
Summary

When flushing the UART, we need to drain manually if LSR_TEMT is
*not* asserted, aka. if the transmit FIFO is not empty.

Reported by: void <void@f-m.fm>
Fixes: c4b68e7e53bb "ns8250: Check if flush via FCR succeeded"

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

cperciva requested review of this revision.Oct 28 2022, 4:45 AM
cperciva created this revision.
Harbormaster completed remote builds in B48064: Diff 112316.Oct 28 2022, 4:45 AM
cperciva added a comment.Oct 28 2022, 7:20 PM

Tested by "void", confirmed that this eliminates the warning from his bhyve VMs.

This revision was not accepted when it landed; it landed in state Needs Review.Oct 28 2022, 7:21 PM
Closed by commit rG5ad8c32c722b: ns8250: Fix sense of LSR_TEMT FCR check (authored by cperciva). · Explain Why
This revision was automatically updated to reflect the committed changes.
cperciva added a commit: rG5ad8c32c722b: ns8250: Fix sense of LSR_TEMT FCR check.

Revision Contents
Changeset List

PathSize
sys/
dev/
uart/
2 lines

Diff 112345

View Options

sys/dev/uart/uart_dev_ns8250.c

Loading...