Fix ESP32 deep_sleep hold

Supercedes: atomvm#2063

CI: Only test release-v5.4 and 5.5.2 for ESP32P4, due to their support for both P4 revisions.

Changelog fixes and additions.

Other minor fixes related to CI.

Signed-off-by: Peter M <petermm@gmail.com>

Author: petermm

.github/workflows/esp32-build.yaml

@@ -45,8 +45,7 @@ jobs:
       fail-fast: false
 
       matrix:
-        esp-idf-target: ["esp32", "esp32c3", "esp32p4"]
-        language: ['cpp']
+        esp-idf-target: ["esp32", "esp32c3"]
         idf-version:
          - 'v5.1.6'
          - 'v5.2.6'
@@ -57,10 +56,11 @@ jobs:
         exclude:
         - esp-idf-target: "esp32c3"
           idf-version: 'v5.1.6'
+        include:
         - esp-idf-target: "esp32p4"
-          idf-version: 'v5.1.6'
+          idf-version: 'release-v5.4'
         - esp-idf-target: "esp32p4"
-          idf-version: 'v5.2.6'
+          idf-version: 'v5.5.2'
 
     steps:
     - name: Checkout repo
@@ -70,9 +70,9 @@ jobs:
       run: git config --global --add safe.directory /__w/AtomVM/AtomVM
 
     - name: "Initialize CodeQL"
-      uses: github/codeql-action/init@v3
+      uses: github/codeql-action/init@v4
       with:
-        languages: ${{matrix.language}}
+        languages: "cpp"
         build-mode: manual
         queries: +./code-queries/term-to-non-term-func.ql,./code-queries/non-term-to-term-func.ql
 
@@ -94,7 +94,7 @@ jobs:
         idf.py size-components
 
     - name: "Perform CodeQL Analysis"
-      uses: github/codeql-action/analyze@v3
+      uses: github/codeql-action/analyze@v4
 
     - name: Install dependencies to build host AtomVM and run qemu
       run: |

.github/workflows/esp32-simtest.yaml

@@ -96,10 +96,10 @@ jobs:
             idf-version: "v5.2.6"
           - esp-idf-target: "esp32c5"
             idf-version: "v5.3.4"
-        # CI now uses chip revision 3 that is currently only available in release-v5.5 branch
+        # CI now uses chip revision 3 that is currently only available in 5.5.2
         include:
           - esp-idf-target: "esp32p4"
-            idf-version: "release-v5.5"
+            idf-version: "v5.5.2"
 
     steps:
       - name: Checkout repo

CHANGELOG.md

@@ -73,7 +73,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Changed
 
 - Removed `externalterm_to_term_copy` added in [0.6.5] and introduced flags to `externalterm_to_term` to perform copy.
-- Release images for ESP32 chips are built with ESP-IDF v5.4
+- Release images for ESP32 chips are built with ESP-IDF v5.5
+- Only support ESP32P4 on ESP-IDF v5.5.2, v5.4.4 and later, release images are for the mass market rev3 version.
 - ESP32: SPI peripheral defaults to `"spi2"` instead of deprecated `hspi`
 - Added `zlib:compress/1`
 - Entry point now is `init:boot/1` if it exists. It starts the kernel application and calls `start/0` from the

src/platforms/esp32/components/avm_builtins/gpio_driver.c

@@ -723,7 +723,7 @@ static term nif_gpio_hold_dis(Context *ctx, int argc, term argv[])
     return hold_dis(argv[0]);
 }
 
-#if (ESP_IDF_VERSION >= ESP_IDF_VERSION_VAL(5, 3, 2) && SOC_GPIO_SUPPORT_HOLD_IO_IN_DSLP && !SOC_GPIO_SUPPORT_HOLD_SINGLE_IO_IN_DSLP) || (ESP_IDF_VERSION < ESP_IDF_VERSION_VAL(5, 3, 2) && !SOC_GPIO_SUPPORT_HOLD_SINGLE_IO_IN_DSLP)
+#if !SOC_GPIO_SUPPORT_HOLD_SINGLE_IO_IN_DSLP
 static term nif_gpio_deep_sleep_hold_en(Context *ctx, int argc, term argv[])
 {
     UNUSED(ctx);
@@ -785,7 +785,7 @@ static const struct Nif gpio_hold_dis_nif =
     .nif_ptr = nif_gpio_hold_dis
 };
 
-#if (ESP_IDF_VERSION >= ESP_IDF_VERSION_VAL(5, 3, 2) && SOC_GPIO_SUPPORT_HOLD_IO_IN_DSLP && !SOC_GPIO_SUPPORT_HOLD_SINGLE_IO_IN_DSLP) || (ESP_IDF_VERSION < ESP_IDF_VERSION_VAL(5, 3, 2) && !SOC_GPIO_SUPPORT_HOLD_SINGLE_IO_IN_DSLP)
+#if !SOC_GPIO_SUPPORT_HOLD_SINGLE_IO_IN_DSLP
 static const struct Nif gpio_deep_sleep_hold_en_nif = {
     .base.type = NIFFunctionType,
     .nif_ptr = nif_gpio_deep_sleep_hold_en
@@ -844,7 +844,7 @@ const struct Nif *gpio_nif_get_nif(const char *nifname)
         return &gpio_hold_dis_nif;
     }
 
-#if (ESP_IDF_VERSION >= ESP_IDF_VERSION_VAL(5, 3, 2) && SOC_GPIO_SUPPORT_HOLD_IO_IN_DSLP && !SOC_GPIO_SUPPORT_HOLD_SINGLE_IO_IN_DSLP) || (ESP_IDF_VERSION < ESP_IDF_VERSION_VAL(5, 3, 2) && !SOC_GPIO_SUPPORT_HOLD_SINGLE_IO_IN_DSLP)
+#if !SOC_GPIO_SUPPORT_HOLD_SINGLE_IO_IN_DSLP
     if (strcmp("gpio:deep_sleep_hold_en/0", nifname) == 0 || strcmp("Elixir.GPIO:deep_sleep_hold_en/0", nifname) == 0) {
         TRACE("Resolved platform nif %s ...\n", nifname);
         return &gpio_deep_sleep_hold_en_nif;

src/platforms/esp32/test/main/test_erl_sources/CMakeLists.txt

@@ -76,6 +76,7 @@ compile_erlang(test_ssl)
 compile_erlang(test_time_and_processes)
 compile_erlang(test_twdt)
 compile_erlang(test_tz)
+compile_erlang(test_deep_sleep_hold)
 
 set(erlang_test_beams
     test_esp_partition.beam
@@ -96,6 +97,7 @@ set(erlang_test_beams
     test_time_and_processes.beam
     test_twdt.beam
     test_tz.beam
+    test_deep_sleep_hold.beam
 )
 
 if(NOT AVM_DISABLE_JIT)

src/platforms/esp32/test/main/test_erl_sources/test_deep_sleep_hold.erl

@@ -0,0 +1,41 @@
+%
+% This file is part of AtomVM.
+%
+% Copyright 2026 Peter M <petermm@gmail.com>
+%
+% Licensed under the Apache License, Version 2.0 (the "License");
+% you may not use this file except in compliance with the License.
+% You may obtain a copy of the License at
+%
+%    http://www.apache.org/licenses/LICENSE-2.0
+%
+% Unless required by applicable law or agreed to in writing, software
+% distributed under the License is distributed on an "AS IS" BASIS,
+% 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.
+%
+% SPDX-License-Identifier: Apache-2.0 OR LGPL-2.1-or-later
+%
+
+-module(test_deep_sleep_hold).
+-export([start/0]).
+
+start() ->
+    Sysinfo = erlang:system_info(esp32_chip_info),
+    Model =
+        if
+            is_map(Sysinfo) -> maps:get(model, Sysinfo);
+            true -> undefined
+        end,
+    case Model of
+        % esp32_p4 has no support on earlier revisions, and thus left out here.
+        M when M =:= esp32; M =:= esp32_s2; M =:= esp32_c3; M =:= esp32_s3; M =:= esp32_c2 ->
+            io:format("Testing: deep_sleep_hold_en/dis\n"),
+            ok = gpio:deep_sleep_hold_en(),
+            ok = gpio:deep_sleep_hold_dis();
+        _ ->
+            io:format("Not testing: deep_sleep_hold_en/dis\n"),
+            ok
+    end,
+    ok.

src/platforms/esp32/test/main/test_main.c

@@ -515,6 +515,12 @@ TEST_CASE("test_tz", "[test_run]")
     TEST_ASSERT(ret_value == OK_ATOM);
 }
 
+TEST_CASE("test_deep_sleep_hold", "[test_run]")
+{
+    term ret_value = avm_test_case("test_deep_sleep_hold.beam");
+    TEST_ASSERT(ret_value == OK_ATOM);
+}
+
 #ifndef AVM_NO_SMP
 TEST_CASE("atomvm_smp_0", "[smp]")
 {
@@ -613,7 +619,7 @@ TEST_CASE("test_wifi_example", "[test_run]")
 #endif
 
 // Works C3 on local runs, but fails GH actions
-#if (ESP_IDF_VERSION_MAJOR >= 5 && !CONFIG_IDF_TARGET_ESP32C3) || (ESP_IDF_VERSION_MAJOR >= 5 && !CONFIG_ETH_USE_OPENETH)
+#if (ESP_IDF_VERSION_MAJOR >= 5 && !CONFIG_IDF_TARGET_ESP32C3 && !CONFIG_IDF_TARGET_ESP32P4) || (ESP_IDF_VERSION_MAJOR >= 5 && !CONFIG_IDF_TARGET_ESP32P4 && !CONFIG_ETH_USE_OPENETH)
 TEST_CASE("test_twdt", "[test_run]")
 {
     term ret_value = avm_test_case("test_twdt.beam");