Skip to content

Conversation

wmrsouza
Copy link
Contributor

@wmrsouza wmrsouza commented Sep 11, 2025

This PR enables adc_api tests for the following ESP32 platforms:

  • esp32_devkitc/esp32/procpu
  • esp32c3_devkitm
  • esp32c6_devkitc/esp32c6/hpcore
  • esp32h2_devkitm
  • esp32s2_devkitc
  • esp32s3_devkitc/esp32s3/procpu

@wmrsouza wmrsouza changed the title Enable adc_api tests for ESP32 platforms Enable adc api_test for ESP32 platforms Sep 11, 2025
@wmrsouza wmrsouza changed the title Enable adc api_test for ESP32 platforms Enable adc_api test for ESP32 platforms Sep 11, 2025
@wmrsouza wmrsouza force-pushed the tests_adc_dma branch 2 times, most recently from 819453f to 14099eb Compare September 11, 2025 18:27
Copy link

github-actions bot commented Sep 11, 2025

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff

All manifest checks OK

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@github-actions github-actions bot added manifest manifest-hal_espressif DNM (manifest) This PR should not be merged (controlled by action-manifest) labels Sep 11, 2025
@wmrsouza wmrsouza force-pushed the tests_adc_dma branch 11 times, most recently from 9b2553c to c4a3a10 Compare September 18, 2025 15:08
@github-actions github-actions bot removed manifest manifest-hal_espressif DNM (manifest) This PR should not be merged (controlled by action-manifest) labels Sep 23, 2025
@wmrsouza
Copy link
Contributor Author

Depends on #95075

@wmrsouza wmrsouza added the DNM This PR should not be merged (Do Not Merge) label Sep 23, 2025
@wmrsouza wmrsouza marked this pull request as ready for review September 23, 2025 14:13
@zephyrbot zephyrbot added the area: ADC Analog-to-Digital Converter (ADC) label Sep 23, 2025
@zephyrbot zephyrbot requested a review from anangl September 23, 2025 14:14
@wmrsouza wmrsouza force-pushed the tests_adc_dma branch 2 times, most recently from 81d62f0 to 259066b Compare September 26, 2025 19:41
@zephyrbot zephyrbot added the area: Tests Issues related to a particular existing or missing test label Sep 30, 2025
@zephyrbot zephyrbot requested a review from nashif September 30, 2025 17:58
nashif
nashif previously approved these changes Oct 4, 2025
@nashif nashif assigned sylvioalves and unassigned nashif Oct 7, 2025
@nashif nashif requested a review from sylvioalves October 7, 2025 12:00
@wmrsouza wmrsouza force-pushed the tests_adc_dma branch 4 times, most recently from a46cb86 to e41ad7c Compare October 8, 2025 16:15
Copy link
Contributor

@sylvioalves sylvioalves left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please, remove esp_wrover_kit and esp32s3_devkitm/esp32s3/procpu from boards folder. Keep devkitc only. There are also other esp32s3-based overlay files in the boards folder, check if any additional change is required.

sylvioalves
sylvioalves previously approved these changes Oct 8, 2025
kartben
kartben previously approved these changes Oct 9, 2025
Enables adc_api tests for these platforms:
- esp32_devkitc/esp32/procpu
- esp32c3_devkitm
- esp32c6_devkitc/esp32c6/hpcore
- esp32h2_devkitm
- esp32s2_devkitc
- esp32s3_devkitc/esp32s3/procpu

Signed-off-by: Marcio Ribeiro <[email protected]>
@wmrsouza wmrsouza dismissed stale reviews from kartben and sylvioalves via c1a12ca October 9, 2025 14:18
Copy link

sonarqubecloud bot commented Oct 9, 2025

@cfriedt cfriedt merged commit ac7799f into zephyrproject-rtos:main Oct 9, 2025
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: ADC Analog-to-Digital Converter (ADC) area: Tests Issues related to a particular existing or missing test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants