[riot-notifications] [RIOT-OS/RIOT] boards/esp32: changes the approach for peripheral configurations of ADC channels in ESP32 board definitions (#11289)

Gunar Schorcht notifications at github.com
Wed Mar 27 09:16:24 CET 2019

### Contribution description

This PR changes the approach of peripheral configurations for ADC in board definitions to the usual RIOT approach. With these changes, peripheral configurations use static const arrays in the `boards/esp32*/periph_conf.h` files and define the `*_NUMOF` macros using the size of these static array.

The static configuration arrays contain only definitions that can be changed by the board definition or the application. They do not contain any MCU implementation detail. The board definitions use preprocessor defines as before to fill these static configuration arrays. This makes it possible to override all configurations either with the make command or application specific configuration files.

### Testing procedure

Compilation and test with the most common ESP32 board should be executed
make BOARD=esp32-wroom-32 -C tests/periph_adc flash test

### Issues/PRs references

You can view, comment on, or merge this pull request online at:


-- Commit Summary --

  * cpu/esp32: GPIO defs required for periph conf
  * cpu/esp32: ADC config approach changed
  * boards/esp32: ADC config approach changed

-- File Changes --

    M boards/common/esp32/include/periph_conf_common.h (7)
    M boards/esp32-olimex-evb/Makefile.features (1)
    M cpu/esp32/include/periph_cpu.h (114)
    M cpu/esp32/periph/adc.c (26)

-- Patch Links --


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190327/f1e34518/attachment.html>

More information about the notifications mailing list