Compile error in analog_reader.h with S2 and S3 variants

[CaptIgmu]

Getting compile error in platformio with SensESP v3 in file analog_reader.h when using ESP32-S2 and S3 variants:

pio/libdeps/adafruit_qtpy_esp32s3_n4r2/SensESP/src/sensesp/sensors/analog_reader.h:28:30: error: 'ADC_ATTEN_DB_12' was not declared in this scope
adc_atten_t attenuation = ADC_ATTEN_DB_12;
^~~~~~~~~~~~~~~
.pio/libdeps/adafruit_qtpy_esp32s3_n4r2/SensESP/src/sensesp/sensors/analog_reader.h:28:30: note: suggested alternative: 'ADC_ATTEN_DB_11'_

I can get it to compile by changing to ADC_ATTEN_DB_11 but noticed in the ESP-IDF docs:
enumerator ADC_ATTEN_DB_11
This is deprecated, it behaves the same as ADC_ATTEN_DB_12

So not sure if a Pull request will work for all ESP32s !

Update 8Sept2024: Same error with my new Halmet board with regular ESP32!

[imoldovavan]

Try ADC_ATTEN_DB_0

[mairas]

Could you provide your platformio.ini file? I tried with setting the board to esp32-s3-devkitc-1 and used a simple program with the following:

  auto* analog_input =
      new AnalogInput(pin, read_delay, analog_in_config_path, output_scale);

  auto analog_sk_output = new SKOutput<float>(sk_path);
  analog_input->connect_to(analog_sk_output);

With the above setup, I couldn't reproduce the error.

[CaptIgmu]

Here is my platformio.ini file:

[env]
lib_ldf_mode = deep
monitor_speed = 115200
upload_speed = 460800

[espressif32_base]
build_unflags = -Werror=reorder
board_build.partitions = min_spiffs.csv
monitor_filters = esp32_exception_decoder

[env:adafruit_qtpy_esp32s3_n4r2]
extends = espressif32_base
platform = espressif32
board = adafruit_qtpy_esp32s3_n4r2
framework = arduino
lib_deps =
signalk/SensESP @ >=3.0.0-beta.1,<4
# https://github.com/SignalK/SensESP.git # Use this line to use the latest git version
# symlink:///Users/mairas/src/SignalK/SensESP # Use this line to use a local copy
mairas/ReactESP @ >=2.1.0,<3
sensesp/OneWire @ >=3.0.0,<4

build_flags =
; Max (and default) debugging level in Arduino ESP32 Core
-D CORE_DEBUG_LEVEL=ARDUHAL_LOG_LEVEL_VERBOSE
; Arduino Core bug workaround: define the log tag for the Arduino
; logging macros.
-D TAG='"Arduino"'
; Use the ESP-IDF logging library - required by SensESP.
-D USE_ESP_IDF_LOG

[janifr]

I have the exact same problem compiling unmodified project template.

In file included from .pio/libdeps/esp32dev/SensESP/src/sensesp/sensors/analog_input.h:4, from src/main.cpp:11: .pio/libdeps/esp32dev/SensESP/src/sensesp/sensors/analog_reader.h:28:30: error: 'ADC_ATTEN_DB_12' was not declared in this scope adc_atten_t attenuation_ = ADC_ATTEN_DB_12; ^~~~~~~~~~~~~~~ .pio/libdeps/esp32dev/SensESP/src/sensesp/sensors/analog_reader.h:28:30: note: suggested alternative: 'ADC_ATTEN_DB_11' adc_atten_t attenuation_ = ADC_ATTEN_DB_12; ^~~~~~~~~~~~~~~ ADC_ATTEN_DB_11

[CaptIgmu]

Same error with my new Halmet board with regular ESP32!

[GHBLoos]

Same issue. I didn't change anything of the downloaded code.
After changing it to ADC_ATTEN_DB_11;, the build succeeds

[smeisner]

I am seeing this as well with an ESP32-DevKitC board. I will eventually be using the ESP32-S3, so it won't be resolved with the target HW.

If I change it to ADC_ATTEN_DB_11 or ADC_ATTEN_DB_0, it works fine.

Is there a problem changing this to ADC_ATTEN_DB_0 (as you commented above) in the source file?

[JohnySeven]

Hit the same roadblock.

Steps to reproduce:

1. Created SensESP repo from template https://github.com/SensESP/SensESP-project-template
2. Git clone to my computer and opened VS Code
3. Hit that build button

Same issue.

[mairas]

I am unable to repeat this issue, even with a fresh install of PlatformIO framework. Both esp32 and esp32s3 builds go through just fine.

Could you try deleting the .pio directory under the project directory and the .platformio directory under your home directory. When you rebuild the project, fresh platform, framework and library versions will be retrieved.

[smeisner]

I am unable to repeat this issue, even with a fresh install of PlatformIO framework. Both esp32 and esp32s3 builds go through just fine.

Could you try deleting the .pio directory under the project directory and the .platformio directory under your home directory. When you rebuild the project, fresh platform, framework and library versions will be retrieved.

Is it possible you have a custom sdkconfig file for your ESP controller? I'm wondering if a setting in there can cause your defaults to change.

[JohnySeven]

@mairas as stated in my comment I've created completely new repo using your template (no .pio existed prior building new empty firmware). Will check version of ESP32 SDK version and try to update to latest.

[mairas]

Is it possible you have a custom sdkconfig file for your ESP controller? I'm wondering if a setting in there can cause your defaults to change.

No, not really. When using the default Arduino framework, sdkconfig.h is always provided by the framework (unless you very specifically configure Arduino as an ESP-IDF component, but that's a whole different kettle of fish).

[mairas]

Some more observations. Most of the examples are being built in CI, and many of them use AnalogInput (and AnalogReader by extension). Similarly, I just created a new Ubuntu 22.04 virtual machine, installed platformio and cloned the project-template repo. There were no issues in compiling.

Now, don't get me wrong. I'm not suggesting this isn't a real problem for many people - I would very much like to find a fix for it, even if it's just pinning some platformio framework versions in platformio.ini. But since I can't reproduce the issue, I can't do much about it.

If/when someone tries to clean their .platformio directory, it might be a good idea to just rename it. If that fixes the issue, the exact version changes could be investigated.

[GHBLoos]

I just build a previous project -> Build succeeded
I removed .pio folder -> Build succeeded
I updated the SensESP library from 3.0b6 to 3.0. Now I get the following error when building

In file included from .pio/libdeps/esp32dev/SensESP/src/sensesp/sensors/analog_input.h:5,
                 from src/main.cpp:11:
.pio/libdeps/esp32dev/SensESP/src/sensesp/sensors/analog_reader.h:28:30: error: 'ADC_ATTEN_DB_12' was not declared in this scope
   adc_atten_t attenuation_ = ADC_ATTEN_DB_12;
                              ^~~~~~~~~~~~~~~
.pio/libdeps/esp32dev/SensESP/src/sensesp/sensors/analog_reader.h:28:30: note: suggested alternative: 'ADC_ATTEN_DB_11'
   adc_atten_t attenuation_ = ADC_ATTEN_DB_12;
                              ^~~~~~~~~~~~~~~
                              ADC_ATTEN_DB_11

Platformio.ini

;
;   Build options: build flags, source filter
;   Upload options: custom upload port, speed and extra flags
;   Library options: dependencies, extra library storages
;   Advanced options: extra scripting
;
; Please visit documentation for the other options and examples
; https://docs.platformio.org/page/projectconf.html

[env:esp32dev]
platform = espressif32
board = esp32dev
framework = arduino
lib_ldf_mode = deep
upload_speed = 460800
monitor_speed = 115200
lib_deps = 
	sensesp/OneWire@^3.0.1
	signalk/SensESP@^3.0.0
build_unflags = 
	-Werror=reorder
board_build.partitions = min_spiffs.csv
monitor_filters = esp32_exception_decoder
extends = espressif32_base
build_flags = 
	-D LED_BUILTIN=2
	-D CORE_DEBUG_LEVEL=ARDUHAL_LOG_LEVEL_VERBOSE
	-D TAG='"Arduino"'
	-D USE_ESP_IDF_LOG

I will try to rename my .platformio directory and build again tomorrow

[JohnySeven]

If/when someone tries to clean their .platformio directory, it might be a good idea to just rename it. If that fixes the issue, the exact version changes could be investigated.

Will try to clean my platformio and see how that goes. Little busy with life right now, sorry!