diff --git a/components/esp32/dport_access.c b/components/esp32/dport_access.c index c2c1294a16..ce2e99ccb1 100644 --- a/components/esp32/dport_access.c +++ b/components/esp32/dport_access.c @@ -22,7 +22,6 @@ #include #include -#include #include "esp_attr.h" #include "esp_err.h" #include "esp_intr_alloc.h" @@ -38,6 +37,8 @@ #include "xtensa/core-macros.h" +#include "sdkconfig.h" + #ifndef CONFIG_FREERTOS_UNICORE static portMUX_TYPE g_dport_mux = portMUX_INITIALIZER_UNLOCKED; diff --git a/components/esp32/hw_random.c b/components/esp32/hw_random.c index faf297503a..d4883982b7 100644 --- a/components/esp32/hw_random.c +++ b/components/esp32/hw_random.c @@ -20,7 +20,6 @@ #include "esp_attr.h" #include "esp32/clk.h" #include "soc/wdev_reg.h" -#include "freertos/FreeRTOSConfig.h" #include "xtensa/core-macros.h" uint32_t IRAM_ATTR esp_random(void) diff --git a/components/esp32/include/esp32/dport_access.h b/components/esp32/include/esp32/dport_access.h index 8ea60c4cdb..0833cae869 100644 --- a/components/esp32/include/esp32/dport_access.h +++ b/components/esp32/include/esp32/dport_access.h @@ -11,12 +11,15 @@ // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. // See the License for the specific language governing permissions and // limitations under the License. +#include #include #ifndef _ESP_DPORT_ACCESS_H_ #define _ESP_DPORT_ACCESS_H_ +#include "xtensa/xtruntime.h" + #ifdef __cplusplus extern "C" { #endif diff --git a/components/esp32/system_api_esp32.c b/components/esp32/system_api_esp32.c index 34dedf074c..781e2a8714 100644 --- a/components/esp32/system_api_esp32.c +++ b/components/esp32/system_api_esp32.c @@ -17,7 +17,6 @@ #include "esp_private/system_internal.h" #include "esp_attr.h" #include "esp_efuse.h" -#include "esp_wifi.h" #include "esp_log.h" #include "sdkconfig.h" #include "esp32/rom/cache.h" @@ -30,7 +29,9 @@ #include "soc/cpu.h" #include "soc/rtc.h" #include "hal/wdt_hal.h" +#include "hal/cpu_hal.h" #include "freertos/xtensa_api.h" +#include "soc/soc_memory_layout.h" #include "esp32/cache_err_int.h" @@ -57,7 +58,7 @@ void IRAM_ATTR esp_restart_noos(void) // CPU must be reset before stalling, in case it was running a s32c1i // instruction. This would cause memory pool to be locked by arbiter // to the stalled CPU, preventing current CPU from accessing this pool. - const uint32_t core_id = xPortGetCoreID(); + const uint32_t core_id = cpu_hal_get_core_id(); const uint32_t other_core_id = (core_id == 0) ? 1 : 0; esp_cpu_reset(other_core_id); esp_cpu_stall(other_core_id); diff --git a/components/esp32s2/hw_random.c b/components/esp32s2/hw_random.c index 3342950925..c8709e28f2 100644 --- a/components/esp32s2/hw_random.c +++ b/components/esp32s2/hw_random.c @@ -20,7 +20,6 @@ #include "esp_attr.h" #include "esp32s2/clk.h" #include "soc/wdev_reg.h" -#include "freertos/FreeRTOSConfig.h" #include "xtensa/core-macros.h" uint32_t IRAM_ATTR esp_random(void) diff --git a/components/esp32s2/system_api_esp32s2.c b/components/esp32s2/system_api_esp32s2.c index bad5c7bb61..e8a6c4794c 100644 --- a/components/esp32s2/system_api_esp32s2.c +++ b/components/esp32s2/system_api_esp32s2.c @@ -17,7 +17,6 @@ #include "esp_system.h" #include "esp_private/system_internal.h" #include "esp_attr.h" -#include "esp_wifi.h" #include "esp_log.h" #include "esp32s2/rom/cache.h" #include "esp_rom_uart.h" @@ -30,6 +29,7 @@ #include "soc/syscon_reg.h" #include "hal/wdt_hal.h" #include "freertos/xtensa_api.h" +#include "hal/cpu_hal.h" /* "inner" restart function for after RTOS, interrupts & anything else on this * core are already stopped. Stalls other core, resets hardware, @@ -55,7 +55,7 @@ void IRAM_ATTR esp_restart_noos(void) // CPU must be reset before stalling, in case it was running a s32c1i // instruction. This would cause memory pool to be locked by arbiter // to the stalled CPU, preventing current CPU from accessing this pool. - const uint32_t core_id = xPortGetCoreID(); + const uint32_t core_id = cpu_hal_get_core_id(); //Todo: Refactor to use Interrupt or Task Watchdog API, and a system level WDT context // Disable TG0/TG1 watchdogs diff --git a/components/esp32s3/hw_random.c b/components/esp32s3/hw_random.c index 251538b878..711dd0574f 100644 --- a/components/esp32s3/hw_random.c +++ b/components/esp32s3/hw_random.c @@ -20,7 +20,6 @@ #include "esp_attr.h" #include "esp32s3/clk.h" #include "soc/wdev_reg.h" -#include "freertos/FreeRTOSConfig.h" #include "xtensa/core-macros.h" uint32_t IRAM_ATTR esp_random(void) diff --git a/components/esp32s3/system_api_esp32s3.c b/components/esp32s3/system_api_esp32s3.c index 252a171bcf..5e787fdd90 100644 --- a/components/esp32s3/system_api_esp32s3.c +++ b/components/esp32s3/system_api_esp32s3.c @@ -17,7 +17,6 @@ #include "esp_system.h" #include "esp_private/system_internal.h" #include "esp_attr.h" -#include "esp_wifi.h" #include "esp_log.h" #include "esp32s3/rom/cache.h" #include "esp_rom_uart.h" @@ -55,7 +54,7 @@ void IRAM_ATTR esp_restart_noos(void) // CPU must be reset before stalling, in case it was running a s32c1i // instruction. This would cause memory pool to be locked by arbiter // to the stalled CPU, preventing current CPU from accessing this pool. - const uint32_t core_id = xPortGetCoreID(); + const uint32_t core_id = cpu_hal_get_core_id(); #if !CONFIG_FREERTOS_UNICORE const uint32_t other_core_id = (core_id == 0) ? 1 : 0; esp_cpu_reset(other_core_id); diff --git a/components/esp_common/src/brownout.c b/components/esp_common/src/brownout.c index cd83c35796..990f3938d0 100644 --- a/components/esp_common/src/brownout.c +++ b/components/esp_common/src/brownout.c @@ -16,16 +16,20 @@ #include #include #include -#include "sdkconfig.h" -#include "esp_log.h" + +#include "esp_private/system_internal.h" +#include "driver/rtc_cntl.h" + +#include "esp_rom_sys.h" + #include "soc/soc.h" #include "soc/cpu.h" #include "soc/rtc_periph.h" +#include "hal/cpu_hal.h" + #include "hal/brownout_hal.h" -#include "esp_private/system_internal.h" -#include "driver/rtc_cntl.h" -#include "freertos/FreeRTOS.h" -#include "esp_rom_sys.h" + +#include "sdkconfig.h" #if defined(CONFIG_ESP32_BROWNOUT_DET_LVL) #define BROWNOUT_DET_LVL CONFIG_ESP32_BROWNOUT_DET_LVL @@ -52,7 +56,7 @@ static void rtc_brownout_isr_handler(void *arg) /* Stall the other CPU to make sure the code running there doesn't use UART * at the same time as the following esp_rom_printf. */ - esp_cpu_stall(!xPortGetCoreID()); + esp_cpu_stall(!cpu_hal_get_core_id()); esp_reset_reason_set_hint(ESP_RST_BROWNOUT); esp_rom_printf("\r\nBrownout detector was triggered\r\n\r\n"); esp_restart_noos(); diff --git a/components/esp_system/panic.c b/components/esp_system/panic.c index 73dee777a1..2c3781d66f 100644 --- a/components/esp_system/panic.c +++ b/components/esp_system/panic.c @@ -16,7 +16,6 @@ #include "esp_err.h" #include "esp_attr.h" -#include "esp_spi_flash.h" #include "esp_private/system_internal.h" #include "esp_private/gdbstub.h" #include "esp_private/usb_console.h" diff --git a/components/esp_system/sleep_modes.c b/components/esp_system/sleep_modes.c index 4a22c2486f..e6fd9136c5 100644 --- a/components/esp_system/sleep_modes.c +++ b/components/esp_system/sleep_modes.c @@ -19,6 +19,7 @@ #include "esp_attr.h" #include "esp_sleep.h" #include "esp_private/esp_timer_private.h" +#include "esp_private/system_internal.h" #include "esp_log.h" #include "esp_newlib.h" #include "esp_timer.h" @@ -429,7 +430,7 @@ esp_err_t esp_light_sleep_start(void) */ esp_timer_private_lock(); s_config.rtc_ticks_at_sleep_start = rtc_time_get(); - uint64_t frc_time_at_start = esp_timer_get_time(); + uint64_t frc_time_at_start = esp_system_get_time(); DPORT_STALL_OTHER_CPU_START(); // Decide which power domains can be powered down @@ -475,7 +476,7 @@ esp_err_t esp_light_sleep_start(void) // FRC1 has been clock gated for the duration of the sleep, correct for that. uint64_t rtc_ticks_at_end = rtc_time_get(); - uint64_t frc_time_at_end = esp_timer_get_time(); + uint64_t frc_time_at_end = esp_system_get_time(); uint64_t rtc_time_diff = rtc_time_slowclk_to_us(rtc_ticks_at_end - s_config.rtc_ticks_at_sleep_start, esp_clk_slowclk_cal_get()); diff --git a/components/esp_wifi/CMakeLists.txt b/components/esp_wifi/CMakeLists.txt index cb3bfc75e4..ff2ea100d1 100644 --- a/components/esp_wifi/CMakeLists.txt +++ b/components/esp_wifi/CMakeLists.txt @@ -35,7 +35,7 @@ idf_component_register(SRCS "src/coexist.c" "${idf_target}/esp_adapter.c" INCLUDE_DIRS "include" "${idf_target}/include" REQUIRES esp_event - PRIV_REQUIRES esp_pm wpa_supplicant nvs_flash esp_netif ${extra_priv_requires} + PRIV_REQUIRES esp_timer esp_pm wpa_supplicant nvs_flash esp_netif ${extra_priv_requires} LDFRAGMENTS "${ldfragments}") idf_build_get_property(build_dir BUILD_DIR) diff --git a/components/esp_wifi/esp32/esp_adapter.c b/components/esp_wifi/esp32/esp_adapter.c index d6e17d013e..8505b5d792 100644 --- a/components/esp_wifi/esp32/esp_adapter.c +++ b/components/esp_wifi/esp32/esp_adapter.c @@ -48,6 +48,7 @@ #include "esp_coexist_internal.h" #include "esp_coexist_adapter.h" #include "esp32/dport_access.h" +#include "esp_timer.h" #define TAG "esp_adapter" diff --git a/components/fatfs/test_fatfs_host/Makefile.files b/components/fatfs/test_fatfs_host/Makefile.files index ae9b5edcb7..73b2774296 100644 --- a/components/fatfs/test_fatfs_host/Makefile.files +++ b/components/fatfs/test_fatfs_host/Makefile.files @@ -16,7 +16,6 @@ INCLUDE_DIRS := \ $(addprefix ../../spi_flash/sim/stubs/, \ app_update/include \ driver/include \ - esp32/include \ freertos/include \ log/include \ newlib/include \ @@ -25,6 +24,7 @@ INCLUDE_DIRS := \ ) \ $(addprefix ../../../components/, \ esp_rom/include \ + esp_system/include \ xtensa/include \ xtensa/esp32/include \ soc/soc/esp32/include \ diff --git a/components/hal/esp32/include/hal/clk_gate_ll.h b/components/hal/esp32/include/hal/clk_gate_ll.h index bedfc6e770..9c332dafe7 100644 --- a/components/hal/esp32/include/hal/clk_gate_ll.h +++ b/components/hal/esp32/include/hal/clk_gate_ll.h @@ -20,6 +20,7 @@ extern "C" { #include #include +#include "esp_attr.h" #include "soc/periph_defs.h" #include "soc/dport_reg.h" #include "soc/dport_access.h" diff --git a/components/hal/esp32s2/include/hal/clk_gate_ll.h b/components/hal/esp32s2/include/hal/clk_gate_ll.h index 9724171163..041081c113 100644 --- a/components/hal/esp32s2/include/hal/clk_gate_ll.h +++ b/components/hal/esp32s2/include/hal/clk_gate_ll.h @@ -20,6 +20,7 @@ extern "C" { #include #include +#include "esp_attr.h" #include "soc/periph_defs.h" #include "soc/system_reg.h" #include "soc/syscon_reg.h" diff --git a/components/hal/esp32s3/include/hal/clk_gate_ll.h b/components/hal/esp32s3/include/hal/clk_gate_ll.h index 7c073b8e69..5b37d008d9 100644 --- a/components/hal/esp32s3/include/hal/clk_gate_ll.h +++ b/components/hal/esp32s3/include/hal/clk_gate_ll.h @@ -20,6 +20,7 @@ extern "C" { #include #include +#include "esp_attr.h" #include "soc/periph_defs.h" #include "soc/system_reg.h" #include "soc/syscon_reg.h" diff --git a/components/newlib/syscall_table.c b/components/newlib/syscall_table.c index 5f8fea2d0a..16d2d6f5fa 100644 --- a/components/newlib/syscall_table.c +++ b/components/newlib/syscall_table.c @@ -22,7 +22,6 @@ #include #include #include -#include "esp_vfs.h" #include "esp_newlib.h" #include "sdkconfig.h" #include "soc/soc_caps.h" diff --git a/components/soc/soc/esp32/include/soc/dport_access.h b/components/soc/soc/esp32/include/soc/dport_access.h index fe7e70ebc0..7ce525dcda 100644 --- a/components/soc/soc/esp32/include/soc/dport_access.h +++ b/components/soc/soc/esp32/include/soc/dport_access.h @@ -16,11 +16,11 @@ #define _DPORT_ACCESS_H_ #include + #include "esp_attr.h" #include "esp32/dport_access.h" -#include "soc.h" -#include "uart_reg.h" -#include "xtensa/xtruntime.h" + +#include "sdkconfig.h" #ifdef __cplusplus extern "C" { diff --git a/components/soc/soc/esp32s2/include/soc/dport_access.h b/components/soc/soc/esp32s2/include/soc/dport_access.h index e5dfeda836..d6d9bfffb6 100644 --- a/components/soc/soc/esp32s2/include/soc/dport_access.h +++ b/components/soc/soc/esp32s2/include/soc/dport_access.h @@ -16,11 +16,8 @@ #define _DPORT_ACCESS_H_ #include -#include "esp_attr.h" + #include "esp32s2/dport_access.h" -#include "soc.h" -#include "uart_reg.h" -#include "xtensa/xtruntime.h" #ifdef __cplusplus extern "C" { diff --git a/components/soc/soc/esp32s3/include/soc/dport_access.h b/components/soc/soc/esp32s3/include/soc/dport_access.h index 2f8b1d48d9..1223fb8b3a 100644 --- a/components/soc/soc/esp32s3/include/soc/dport_access.h +++ b/components/soc/soc/esp32s3/include/soc/dport_access.h @@ -15,12 +15,8 @@ #pragma once #include -#include "esp_attr.h" -#include "esp_attr.h" + #include "esp32s3/dport_access.h" -#include "soc.h" -#include "uart_reg.h" -#include "xtensa/xtruntime.h" #ifdef __cplusplus extern "C" { diff --git a/components/spi_flash/esp_flash_api.c b/components/spi_flash/esp_flash_api.c index b05b530c2c..79e242e160 100644 --- a/components/spi_flash/esp_flash_api.c +++ b/components/spi_flash/esp_flash_api.c @@ -21,10 +21,9 @@ #include "memspi_host_driver.h" #include "esp_log.h" #include "sdkconfig.h" -#include "esp_heap_caps.h" #include "esp_flash_internal.h" -#include -#include "esp_timer.h" +#include "esp_private/system_internal.h" + static const char TAG[] = "spi_flash"; @@ -378,7 +377,7 @@ esp_err_t IRAM_ATTR esp_flash_erase_region(esp_flash_t *chip, uint32_t start, ui #endif while (err == ESP_OK && len >= sector_size) { #ifdef CONFIG_SPI_FLASH_YIELD_DURING_ERASE - int64_t start_time_us = esp_timer_get_time(); + int64_t start_time_us = esp_system_get_time(); #endif err = rom_spiflash_api_funcs->start(chip); if (err != ESP_OK) { @@ -403,7 +402,7 @@ esp_err_t IRAM_ATTR esp_flash_erase_region(esp_flash_t *chip, uint32_t start, ui err = rom_spiflash_api_funcs->end(chip, err); #ifdef CONFIG_SPI_FLASH_YIELD_DURING_ERASE - no_yield_time_us += (esp_timer_get_time() - start_time_us); + no_yield_time_us += (esp_system_get_time() - start_time_us); if (no_yield_time_us / 1000 >= CONFIG_SPI_FLASH_ERASE_YIELD_DURATION_MS) { no_yield_time_us = 0; if (chip->os_func->yield) { diff --git a/components/spi_flash/flash_ops.c b/components/spi_flash/flash_ops.c index 0068f4cd69..cd1f2ea115 100644 --- a/components/spi_flash/flash_ops.c +++ b/components/spi_flash/flash_ops.c @@ -28,6 +28,7 @@ #include "esp_attr.h" #include "esp_spi_flash.h" #include "esp_log.h" +#include "esp_private/system_internal.h" #if CONFIG_IDF_TARGET_ESP32 #include "esp32/rom/spi_flash.h" #include "esp32/rom/cache.h" @@ -45,7 +46,6 @@ #include "cache_utils.h" #include "esp_flash.h" #include "esp_attr.h" -#include "esp_timer.h" esp_rom_spiflash_result_t IRAM_ATTR spi_flash_write_encrypted_chip(size_t dest_addr, const void *src, size_t size); @@ -143,7 +143,6 @@ static __attribute__((unused)) bool is_safe_write_address(size_t addr, size_t si return true; } - void spi_flash_init(void) { spi_flash_init_lock(); @@ -260,7 +259,7 @@ esp_err_t IRAM_ATTR spi_flash_erase_range(size_t start_addr, size_t size) #endif for (size_t sector = start; sector != end && rc == ESP_ROM_SPIFLASH_RESULT_OK; ) { #ifdef CONFIG_SPI_FLASH_YIELD_DURING_ERASE - int64_t start_time_us = esp_timer_get_time(); + int64_t start_time_us = esp_system_get_time(); #endif spi_flash_guard_start(); #ifndef CONFIG_SPI_FLASH_BYPASS_BLOCK_ERASE @@ -277,7 +276,7 @@ esp_err_t IRAM_ATTR spi_flash_erase_range(size_t start_addr, size_t size) } spi_flash_guard_end(); #ifdef CONFIG_SPI_FLASH_YIELD_DURING_ERASE - no_yield_time_us += (esp_timer_get_time() - start_time_us); + no_yield_time_us += (esp_system_get_time() - start_time_us); if (no_yield_time_us / 1000 >= CONFIG_SPI_FLASH_ERASE_YIELD_DURATION_MS) { no_yield_time_us = 0; if (s_flash_guard_ops && s_flash_guard_ops->yield) { diff --git a/components/spi_flash/sim/Makefile.files b/components/spi_flash/sim/Makefile.files index 036cc627e4..5c05144322 100644 --- a/components/spi_flash/sim/Makefile.files +++ b/components/spi_flash/sim/Makefile.files @@ -16,7 +16,6 @@ INCLUDE_DIRS := \ $(addprefix stubs/, \ app_update/include \ driver/include \ - esp32/include \ esp_timer/include \ freertos/include \ log/include \ @@ -27,6 +26,7 @@ INCLUDE_DIRS := \ $(addprefix ../../../components/, \ esp_rom/include \ esp_common/include \ + esp_system/include \ xtensa/include \ xtensa/esp32/include \ soc/soc/esp32/include \ diff --git a/components/spi_flash/sim/stubs/Makefile.files b/components/spi_flash/sim/stubs/Makefile.files index 8fdf44c025..9b63d80c97 100644 --- a/components/spi_flash/sim/stubs/Makefile.files +++ b/components/spi_flash/sim/stubs/Makefile.files @@ -12,7 +12,6 @@ INCLUDE_DIRS := \ ../private_include \ app_update/include \ driver/include \ - esp32/include \ esp_timer/include \ freertos/include \ log/include \ @@ -21,6 +20,7 @@ INCLUDE_DIRS := \ vfs/include \ $(addprefix ../../../../components/, \ esp_common/include \ + esp_system/include \ soc/soc/esp32/include \ soc/include \ soc/soc/include \ diff --git a/components/spi_flash/sim/stubs/esp32/include/esp_system.h b/components/spi_flash/sim/stubs/esp32/include/esp_system.h deleted file mode 100644 index 74511d5ab9..0000000000 --- a/components/spi_flash/sim/stubs/esp32/include/esp_system.h +++ /dev/null @@ -1,15 +0,0 @@ -#pragma once - -#include -#include - -#ifdef __cplusplus -extern "C" { -#endif - -uint32_t esp_random(void); - -#ifdef __cplusplus -} -#endif - diff --git a/components/spiffs/test_spiffs_host/Makefile.files b/components/spiffs/test_spiffs_host/Makefile.files index c6f9b69b38..f5278d0002 100644 --- a/components/spiffs/test_spiffs_host/Makefile.files +++ b/components/spiffs/test_spiffs_host/Makefile.files @@ -16,7 +16,6 @@ INCLUDE_DIRS := \ $(addprefix ../../spi_flash/sim/stubs/, \ app_update/include \ driver/include \ - esp32/include \ freertos/include \ log/include \ newlib/include \ @@ -26,6 +25,7 @@ INCLUDE_DIRS := \ $(addprefix ../../../components/, \ esp_rom/include \ esp_common/include \ + esp_system/include \ xtensa/include \ xtensa/esp32/include \ soc/soc/esp32/include \ diff --git a/components/wear_levelling/test_wl_host/Makefile.files b/components/wear_levelling/test_wl_host/Makefile.files index 35428a419b..93cd2bc722 100644 --- a/components/wear_levelling/test_wl_host/Makefile.files +++ b/components/wear_levelling/test_wl_host/Makefile.files @@ -15,7 +15,6 @@ INCLUDE_DIRS := \ $(addprefix ../../spi_flash/sim/stubs/, \ app_update/include \ driver/include \ - esp32/include \ freertos/include \ log/include \ newlib/include \ @@ -24,6 +23,7 @@ INCLUDE_DIRS := \ ) \ $(addprefix ../../../components/, \ esp_rom/include \ + esp_system/include \ esp_common/include \ xtensa/include \ xtensa/esp32/include \ diff --git a/components/xtensa/include/esp_debug_helpers.h b/components/xtensa/include/esp_debug_helpers.h index d088ddeef1..5bdc028c61 100644 --- a/components/xtensa/include/esp_debug_helpers.h +++ b/components/xtensa/include/esp_debug_helpers.h @@ -22,7 +22,7 @@ extern "C" { #include #include "esp_err.h" -#include "soc/soc.h" +#include "soc/soc.h" // [refactor-todo] IDF-2297 #define ESP_WATCHPOINT_LOAD 0x40000000 #define ESP_WATCHPOINT_STORE 0x80000000 diff --git a/components/xtensa/include/xtensa-debug-module.h b/components/xtensa/include/xtensa-debug-module.h index 6e44d78261..8ac47a9ca8 100644 --- a/components/xtensa/include/xtensa-debug-module.h +++ b/components/xtensa/include/xtensa-debug-module.h @@ -1,6 +1,6 @@ #ifndef XTENSA_DEBUG_MODULE_H #define XTENSA_DEBUG_MODULE_H -#include "soc/cpu.h" + /* ERI registers / OCD offsets and field definitions */