mirror of
https://github.com/espressif/esp-idf.git
synced 2026-04-27 19:13:21 +00:00
fix: clear modem RF flag before PMU sleep to prevent open rf failed on next wake-up
This commit is contained in:
@@ -39,7 +39,7 @@ static _lock_t s_modem_prepare_lock;
|
||||
#endif // SOC_PM_RETENTION_HAS_CLOCK_BUG && CONFIG_MAC_BB_PD
|
||||
|
||||
#if CONFIG_MAC_BB_PD
|
||||
#define MAC_BB_POWER_DOWN_CB_NO (3)
|
||||
#define MAC_BB_POWER_DOWN_CB_NO (4)
|
||||
#define MAC_BB_POWER_UP_CB_NO (3)
|
||||
|
||||
static DRAM_ATTR mac_bb_power_down_cb_t s_mac_bb_power_down_cb[MAC_BB_POWER_DOWN_CB_NO];
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
/*
|
||||
* SPDX-FileCopyrightText: 2015-2024 Espressif Systems (Shanghai) CO LTD
|
||||
* SPDX-FileCopyrightText: 2015-2025 Espressif Systems (Shanghai) CO LTD
|
||||
*
|
||||
* SPDX-License-Identifier: Apache-2.0
|
||||
*/
|
||||
@@ -15,6 +15,7 @@ extern "C" {
|
||||
#endif
|
||||
|
||||
#define ESP_CAL_DATA_CHECK_FAIL 1
|
||||
#define ESP_MODEM_RF_FLAG_UPDATE_CB_REQUIRED (SOC_PM_MODEM_RF_FLAG_UPDATE_WORKAROUND || CONFIG_ESP_WIFI_MODEM_RF_FLAG_UPDATE_DEBUG)
|
||||
|
||||
typedef struct {
|
||||
uint8_t cmd_type; /* the command type of the current phy i2c master command memory config */
|
||||
@@ -244,6 +245,16 @@ uint32_t phy_ana_i2c_master_burst_rf_onoff(bool on);
|
||||
void phy_wakeup_from_modem_state_extra_init(void);
|
||||
#endif
|
||||
|
||||
#if SOC_PM_SUPPORT_PMU_MODEM_STATE && CONFIG_ESP_WIFI_ENHANCED_LIGHT_SLEEP && ESP_MODEM_RF_FLAG_UPDATE_CB_REQUIRED
|
||||
/**
|
||||
* @brief Update modem RF flag
|
||||
*
|
||||
* This function is called as a callback during MAC/BB power down operations.
|
||||
* It checks if modem RF is already enabled and clears the RF power state accordingly.
|
||||
*/
|
||||
void esp_phy_modem_rf_flag_update(void);
|
||||
#endif
|
||||
|
||||
#if SOC_PM_MODEM_RETENTION_BY_REGDMA && CONFIG_MAC_BB_PD
|
||||
/**
|
||||
* @brief PHY module sleep data (includes AGC, TX, NRX, BB, FE, etc..) initialize.
|
||||
|
||||
@@ -64,6 +64,11 @@
|
||||
extern wifi_mac_time_update_cb_t s_wifi_mac_time_update_cb;
|
||||
#endif
|
||||
|
||||
#if SOC_PM_SUPPORT_PMU_MODEM_STATE && CONFIG_ESP_WIFI_ENHANCED_LIGHT_SLEEP
|
||||
extern void pm_mac_modem_clear_rf_power_state(void);
|
||||
extern bool pm_mac_modem_rf_already_enabled(void);
|
||||
#endif
|
||||
|
||||
static const char* TAG = "phy_init";
|
||||
|
||||
static _lock_t s_phy_access_lock;
|
||||
@@ -295,6 +300,18 @@ static inline void phy_digital_regs_load(void)
|
||||
}
|
||||
#endif // SOC_PM_MODEM_RETENTION_BY_BACKUPDMA
|
||||
|
||||
#if SOC_PM_SUPPORT_PMU_MODEM_STATE && CONFIG_ESP_WIFI_ENHANCED_LIGHT_SLEEP && ESP_MODEM_RF_FLAG_UPDATE_CB_REQUIRED
|
||||
void IRAM_ATTR esp_phy_modem_rf_flag_update(void)
|
||||
{
|
||||
if (pm_mac_modem_rf_already_enabled()) {
|
||||
#if CONFIG_ESP_WIFI_MODEM_RF_FLAG_UPDATE_DEBUG
|
||||
assert(0);
|
||||
#endif
|
||||
pm_mac_modem_clear_rf_power_state();
|
||||
}
|
||||
}
|
||||
#endif
|
||||
|
||||
void esp_phy_enable(esp_phy_modem_t modem)
|
||||
{
|
||||
_lock_acquire(&s_phy_access_lock);
|
||||
@@ -313,7 +330,6 @@ void esp_phy_enable(esp_phy_modem_t modem)
|
||||
s_is_phy_calibrated = true;
|
||||
} else {
|
||||
#if SOC_PM_SUPPORT_PMU_MODEM_STATE && CONFIG_ESP_WIFI_ENHANCED_LIGHT_SLEEP
|
||||
extern bool pm_mac_modem_rf_already_enabled(void);
|
||||
if (!pm_mac_modem_rf_already_enabled()) {
|
||||
if (sleep_modem_wifi_modem_state_enabled() && sleep_modem_wifi_modem_link_done()) {
|
||||
sleep_modem_wifi_do_phy_retention(true);
|
||||
@@ -380,7 +396,6 @@ void esp_phy_disable(esp_phy_modem_t modem)
|
||||
phy_digital_regs_store();
|
||||
#endif
|
||||
#if SOC_PM_SUPPORT_PMU_MODEM_STATE && CONFIG_ESP_WIFI_ENHANCED_LIGHT_SLEEP
|
||||
extern void pm_mac_modem_clear_rf_power_state(void);
|
||||
pm_mac_modem_clear_rf_power_state();
|
||||
if (sleep_modem_wifi_modem_state_enabled()) {
|
||||
sleep_modem_wifi_do_phy_retention(false);
|
||||
|
||||
@@ -872,6 +872,15 @@ menu "Wi-Fi"
|
||||
Select this configuration to free dynamic buffers during WiFi enterprise connection.
|
||||
This will enable chip to reduce heap consumption during WiFi enterprise connection.
|
||||
|
||||
config ESP_WIFI_MODEM_RF_FLAG_UPDATE_DEBUG
|
||||
bool "Enable debug assertions for modem RF flag update"
|
||||
depends on ESP_WIFI_ENHANCED_LIGHT_SLEEP
|
||||
default n
|
||||
help
|
||||
Enable debug assertions to verify modem RF flag update operations.
|
||||
This option enables assert checks to verify that modem RF power state
|
||||
is correctly cleared before pmu sleep.
|
||||
|
||||
endif # wifi enabled
|
||||
|
||||
endmenu # Wi-Fi
|
||||
|
||||
@@ -253,7 +253,10 @@ static esp_err_t wifi_deinit_internal(void)
|
||||
#if CONFIG_ESP_WIFI_ENHANCED_LIGHT_SLEEP
|
||||
esp_wifi_internal_modem_state_configure(false);
|
||||
esp_pm_unregister_skip_light_sleep_callback(sleep_modem_wifi_modem_state_skip_light_sleep);
|
||||
#if ESP_MODEM_RF_FLAG_UPDATE_CB_REQUIRED
|
||||
esp_unregister_mac_bb_pd_callback(esp_phy_modem_rf_flag_update);
|
||||
#endif
|
||||
#endif /* CONFIG_ESP_WIFI_ENHANCED_LIGHT_SLEEP */
|
||||
#ifdef CONFIG_ESP_PHY_ENABLED
|
||||
esp_phy_modem_deinit();
|
||||
#endif
|
||||
@@ -453,6 +456,12 @@ esp_err_t esp_wifi_init(const wifi_init_config_t *config)
|
||||
if (sleep_modem_wifi_modem_state_enabled()) {
|
||||
esp_pm_register_skip_light_sleep_callback(sleep_modem_wifi_modem_state_skip_light_sleep);
|
||||
esp_wifi_internal_modem_state_configure(true); /* require WiFi to enable automatically receives the beacon */
|
||||
#if ESP_MODEM_RF_FLAG_UPDATE_CB_REQUIRED
|
||||
if (esp_register_mac_bb_pd_callback(esp_phy_modem_rf_flag_update) != ESP_OK) {
|
||||
ESP_LOGE(TAG, "Failed to register modem RF flag update callback");
|
||||
goto _deinit;
|
||||
}
|
||||
#endif
|
||||
}
|
||||
#endif
|
||||
#if CONFIG_IDF_TARGET_ESP32
|
||||
|
||||
@@ -1191,6 +1191,10 @@ config SOC_PM_RETENTION_HAS_CLOCK_BUG
|
||||
bool
|
||||
default y
|
||||
|
||||
config SOC_PM_MODEM_RF_FLAG_UPDATE_WORKAROUND
|
||||
bool
|
||||
default y
|
||||
|
||||
config SOC_PM_PAU_LINK_NUM
|
||||
int
|
||||
default 4
|
||||
|
||||
@@ -491,6 +491,8 @@
|
||||
#define SOC_PM_MODEM_RETENTION_BY_REGDMA (1)
|
||||
#define SOC_PM_RETENTION_HAS_CLOCK_BUG (1)
|
||||
|
||||
#define SOC_PM_MODEM_RF_FLAG_UPDATE_WORKAROUND (1)
|
||||
|
||||
#define SOC_PM_PAU_LINK_NUM (4)
|
||||
#define SOC_PM_PAU_REGDMA_LINK_MULTI_ADDR (1)
|
||||
#define SOC_PM_PAU_REGDMA_LINK_WIFIMAC (1)
|
||||
|
||||
Reference in New Issue
Block a user