From 02bb85a791bc2c27cdfbf9db5492c8d65b591608 Mon Sep 17 00:00:00 2001
From: Michael de Silva <michael@mwdesilva.com>
Date: Wed, 24 Apr 2024 01:59:00 +0530
Subject: [PATCH] Add trace logging for USART

---
 embassy-stm32/src/usart/mod.rs | 32 ++++++++++++++++++++++++++------
 1 file changed, 26 insertions(+), 6 deletions(-)

diff --git a/embassy-stm32/src/usart/mod.rs b/embassy-stm32/src/usart/mod.rs
index d21e5c47c..6d6e678c8 100644
--- a/embassy-stm32/src/usart/mod.rs
+++ b/embassy-stm32/src/usart/mod.rs
@@ -1270,8 +1270,14 @@ fn configure(
 
     let (mul, brr_min, brr_max) = match kind {
         #[cfg(any(usart_v3, usart_v4))]
-        Kind::Lpuart => (256, 0x300, 0x10_0000),
-        Kind::Uart => (1, 0x10, 0x1_0000),
+        Kind::Lpuart => {
+            trace!("USART: Kind::Lpuart");
+            (256, 0x300, 0x10_0000)
+        }
+        Kind::Uart => {
+            trace!("USART: Kind::Uart");
+            (1, 0x10, 0x1_0000)
+        }
     };
 
     fn calculate_brr(baud: u32, pclk: u32, presc: u32, mul: u32) -> u32 {
@@ -1374,21 +1380,35 @@ fn configure(
         // configure word size
         // if using odd or even parity it must be configured to 9bits
         w.set_m0(if config.parity != Parity::ParityNone {
+            trace!("USART: m0: vals::M0::BIT9");
             vals::M0::BIT9
         } else {
+            trace!("USART: m0: vals::M0::BIT8");
             vals::M0::BIT8
         });
         // configure parity
         w.set_pce(config.parity != Parity::ParityNone);
         w.set_ps(match config.parity {
-            Parity::ParityOdd => vals::Ps::ODD,
-            Parity::ParityEven => vals::Ps::EVEN,
-            _ => vals::Ps::EVEN,
+            Parity::ParityOdd => {
+                trace!("USART: set_ps: vals::Ps::ODD");
+                vals::Ps::ODD
+            }
+            Parity::ParityEven => {
+                trace!("USART: set_ps: vals::Ps::EVEN");
+                vals::Ps::EVEN
+            }
+            _ => {
+                trace!("USART: set_ps: vals::Ps::EVEN");
+                vals::Ps::EVEN
+            }
         });
         #[cfg(not(usart_v1))]
         w.set_over8(vals::Over8::from_bits(over8 as _));
         #[cfg(usart_v4)]
-        w.set_fifoen(true);
+        {
+            trace!("USART: set_fifoen: true (usart_v4)");
+            w.set_fifoen(true);
+        }
     });
 
     Ok(())