cfg away ps API

esp-wifi/CHANGELOG.md

@@ -18,6 +18,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - No need to add `rom_functions.x` manually anymore (#2374)
 - esp-now: Data is now private in `ReceivedData` - use `data()`(#2396)
 - Changed the async APIs to have a `_async` postfix to avoid name collisions (#2446)
+- `phy_enable_usb` is enabled by default (#2446)
 
 ### Fixed
 

esp-wifi/src/config.rs

@@ -27,6 +27,7 @@ pub(crate) struct EspWifiConfig {
     pub(crate) scan_method: u32,
 }
 
+#[cfg(not(coex))]
 #[non_exhaustive]
 #[derive(Default)]
 pub enum PowerSaveMode {
@@ -36,6 +37,7 @@ pub enum PowerSaveMode {
     Maximum,
 }
 
+#[cfg(not(coex))]
 impl From<PowerSaveMode> for include::wifi_ps_type_t {
     fn from(s: PowerSaveMode) -> Self {
         match s {

esp-wifi/src/esp_now/mod.rs

@@ -16,9 +16,10 @@ use critical_section::Mutex;
 use enumset::EnumSet;
 use portable_atomic::{AtomicBool, AtomicU8, Ordering};
 
+#[cfg(coex)]
+use crate::config::PowerSaveMode;
 use crate::{
     binary::include::*,
-    config::PowerSaveMode,
     hal::peripheral::{Peripheral, PeripheralRef},
     wifi::{Protocol, RxControlInfo, WifiError},
     EspWifiController,
@@ -336,10 +337,8 @@ impl EspNowManager<'_> {
         Ok(())
     }
 
+    #[cfg(not(coex))]
     /// Configures modem power saving
-    ///
-    /// This is ignored, and set to `PowerSaveMode::Minimum` when `coex` is
-    /// enabled.
     pub fn set_power_saving(&self, ps: PowerSaveMode) -> Result<(), WifiError> {
         crate::wifi::apply_power_saving(ps)
     }

esp-wifi/src/wifi/mod.rs

@@ -61,9 +61,10 @@ use serde::{Deserialize, Serialize};
 use smoltcp::phy::{Device, DeviceCapabilities, RxToken, TxToken};
 pub use state::*;
 
+#[cfg(not(coex))]
+use crate::config::PowerSaveMode;
 use crate::{
     common_adapter::*,
-    config::PowerSaveMode,
     esp_wifi_result,
     hal::{
         macros::ram,
@@ -2591,10 +2592,8 @@ impl<'d> WifiController<'d> {
         Ok(())
     }
 
+    #[cfg(not(coex))]
     /// Configures modem power saving
-    ///
-    /// This is ignored, and set to `PowerSaveMode::Minimum` when `coex` is
-    /// enabled.
     pub fn set_power_saving(&mut self, ps: PowerSaveMode) -> Result<(), WifiError> {
         apply_power_saving(ps)
     }
@@ -3266,10 +3265,8 @@ pub(crate) mod embassy {
     }
 }
 
-pub(crate) fn apply_power_saving(mut ps: PowerSaveMode) -> Result<(), WifiError> {
-    if cfg!(coex) {
-        ps = PowerSaveMode::Minimum;
-    }
+#[cfg(not(coex))]
+pub(crate) fn apply_power_saving(ps: PowerSaveMode) -> Result<(), WifiError> {
     esp_wifi_result!(unsafe { esp_wifi_sys::include::esp_wifi_set_ps(ps.into()) })?;
     Ok(())
 }