sw: bug fix for the pinmux_check

Check only waited for an empty fifo and not for the transmission to be finished. New pinmux driver allowed optimisations caused the check to break because of this. Co-authored-by: Alex Jones <[email protected]>
lowRISC · Nov 14, 2024 · 453f0b2 · 453f0b2
1 parent 386b24c
commit 453f0b2
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/sw/cheri/checks/pinmux_check.cc b/sw/cheri/checks/pinmux_check.cc
@@ -22,9 +22,7 @@ using namespace CHERI;
  * Blocks until the UART transmit FIFO is empty.
  */
 void block_until_uart_tx_done(Capability<volatile OpenTitanUart> uart) {
-  while (uart->transmit_fifo_level() > 0) {
-    asm volatile("");
-  }
+  while (!(uart->status & uart->StatusTransmitIdle));
 }
 
 /**