From a189eff51751e7cfc162b6cdd5887fb373b9679f Mon Sep 17 00:00:00 2001
From: Jesse Braham <jessebraham@users.noreply.github.com>
Date: Tue, 3 Dec 2024 01:27:08 -0800
Subject: [PATCH] Use `cfg_if` in favour of multiple `cfg`s in
 `set_rx_fifo_full_threshold` function (#2670)

---
 esp-hal/src/uart.rs | 19 +++++++++++--------
 1 file changed, 11 insertions(+), 8 deletions(-)

diff --git a/esp-hal/src/uart.rs b/esp-hal/src/uart.rs
index be07a6dea..4b5f8f79b 100644
--- a/esp-hal/src/uart.rs
+++ b/esp-hal/src/uart.rs
@@ -2324,14 +2324,17 @@ impl Info {
     /// - `esp32c3`, `esp32c2`, `esp32s2` **0x1FF**
     /// - `esp32s3` **0x3FF**
     fn set_rx_fifo_full_threshold(&self, threshold: u16) -> Result<(), ConfigError> {
-        #[cfg(esp32)]
-        const MAX_THRHD: u16 = 0x7F;
-        #[cfg(any(esp32c6, esp32h2))]
-        const MAX_THRHD: u16 = 0xFF;
-        #[cfg(any(esp32c3, esp32c2, esp32s2))]
-        const MAX_THRHD: u16 = 0x1FF;
-        #[cfg(esp32s3)]
-        const MAX_THRHD: u16 = 0x3FF;
+        cfg_if::cfg_if! {
+            if #[cfg(esp32)] {
+                const MAX_THRHD: u16 = 0x7F;
+            } else if #[cfg(any(esp32c6, esp32h2))] {
+                const MAX_THRHD: u16 = 0xFF;
+            } else if #[cfg(any(esp32c3, esp32c2, esp32s2))] {
+                const MAX_THRHD: u16 = 0x1FF;
+            } else if #[cfg(esp32s3)] {
+                const MAX_THRHD: u16 = 0x3FF;
+            }
+        }
 
         if threshold > MAX_THRHD {
             return Err(ConfigError::UnsupportedFifoThreshold);