GetLitFam/esp-hal
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Use cfg_if in favour of multiple cfgs in set_rx_fifo_full_threshold function (#2670)

Browse Source
This commit is contained in:
Jesse Braham 2024-12-03 01:27:08 -08:00 committed by GitHub
parent da59a4eb36
commit a189eff517
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 11 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
esp-hal/src/uart.rs
View File

@ -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)]
cfg_if::cfg_if! {
if #[cfg(esp32)] {
const MAX_THRHD: u16 = 0x7F;
#[cfg(any(esp32c6, esp32h2))]
} else if #[cfg(any(esp32c6, esp32h2))] {
const MAX_THRHD: u16 = 0xFF;
#[cfg(any(esp32c3, esp32c2, esp32s2))]
} else if #[cfg(any(esp32c3, esp32c2, esp32s2))] {
const MAX_THRHD: u16 = 0x1FF;
#[cfg(esp32s3)]
} else if #[cfg(esp32s3)] {
const MAX_THRHD: u16 = 0x3FF;
}
}
if threshold > MAX_THRHD {
return Err(ConfigError::UnsupportedFifoThreshold);