From 11c0daef62d3d19a0dd97f5dafbd592addd6f3bb Mon Sep 17 00:00:00 2001 From: Jack Date: Thu, 4 Jan 2018 11:48:24 +0800 Subject: [PATCH] Fix bug that when we do full calibration init data bin is not used --- components/esp32/Kconfig | 9 +++++++-- components/esp32/phy_init.c | 15 ++++++++------- 2 files changed, 15 insertions(+), 9 deletions(-) diff --git a/components/esp32/Kconfig b/components/esp32/Kconfig index 888bba17eb..57b0599d9d 100644 --- a/components/esp32/Kconfig +++ b/components/esp32/Kconfig @@ -940,13 +940,18 @@ endmenu # Wi-Fi menu PHY config ESP32_PHY_CALIBRATION_AND_DATA_STORAGE - bool "Do phy calibration and store calibration data in NVS" + bool "Store phy calibration data in NVS" default y help If this option is enabled, NVS will be initialized and calibration data will be loaded from there. PHY calibration will be skipped on deep sleep wakeup. If calibration data is not found, full calibration - will be performed and stored in NVS. In all other cases, only partial calibration will be performed. + will be performed and stored in NVS. Normally, only partial calibration will be performed. + If this option is disabled, full calibration will be performed. + If it's easy that your board calibrate bad data, choose 'n'. + Two cases for example, you should choose 'n': + 1.If your board is easy to be booted up with antenna disconnected. + 2.Because of your board design, each time when you do calibration, the result are too unstable. If unsure, choose 'y'. config ESP32_PHY_INIT_DATA_IN_PARTITION diff --git a/components/esp32/phy_init.c b/components/esp32/phy_init.c index 94a89afd67..280a4f0942 100644 --- a/components/esp32/phy_init.c +++ b/components/esp32/phy_init.c @@ -266,17 +266,17 @@ void esp_phy_load_cal_and_init(void) abort(); } -#ifdef CONFIG_ESP32_PHY_CALIBRATION_AND_DATA_STORAGE - esp_phy_calibration_mode_t calibration_mode = PHY_RF_CAL_PARTIAL; - if (rtc_get_reset_reason(0) == DEEPSLEEP_RESET) { - calibration_mode = PHY_RF_CAL_NONE; - } const esp_phy_init_data_t* init_data = esp_phy_get_init_data(); if (init_data == NULL) { ESP_LOGE(TAG, "failed to obtain PHY init data"); abort(); } +#ifdef CONFIG_ESP32_PHY_CALIBRATION_AND_DATA_STORAGE + esp_phy_calibration_mode_t calibration_mode = PHY_RF_CAL_PARTIAL; + if (rtc_get_reset_reason(0) == DEEPSLEEP_RESET) { + calibration_mode = PHY_RF_CAL_NONE; + } esp_err_t err = esp_phy_load_cal_data_from_nvs(cal_data); if (err != ESP_OK) { ESP_LOGW(TAG, "failed to load RF calibration data (0x%x), falling back to full calibration", err); @@ -290,11 +290,12 @@ void esp_phy_load_cal_and_init(void) } else { err = ESP_OK; } - esp_phy_release_init_data(init_data); #else - esp_phy_rf_init(NULL, PHY_RF_CAL_FULL, cal_data); + esp_phy_rf_init(init_data, PHY_RF_CAL_FULL, cal_data); #endif + esp_phy_release_init_data(init_data); + free(cal_data); // PHY maintains a copy of calibration data, so we can free this }