Skip to content

Commit

Permalink
boards,sys/arduino: major clean up
Browse files Browse the repository at this point in the history
- Rename all `arduino_pinmap.h` to `arduino_iomap.h`
    - An empty `arduino_pinmap.h` that just includes `arduino_iomap.h`
      is provided for backward compatibility
    - Move all info from `arduino_board.h` into the new file as trivial
      macros, so that they can also be used outside of sketches
    - The new name reflects the fact not just pin mappings, but also
      other I/O features such as PWMs are mapped
- Drop all `arduino_board.h`
    - `arduino_board.h` and `arduino_iomap.h` now provide the exact
      same information, just in a different format
    - a generic `arduino_board.h` is provided instead that just
      uses the info in `arduinio_iomap.h` and provides them in the
      format the code in `sys/arduino` expects it
- Add fine grained features to indicate for mappings
    - availability of mappings for analog pins, DAC pins, PWM pins,
      UART devices, SPI/I2C buses to the corresponding RIOT
      identification can now be expressed:
        - `arduino_pins`: `ARDUINO_PIN_0` etc. are available
        - `arduino_analog`: `ARDUINO_A0` etc. are available
        - `arduino_pwm`: `ARDUINO_PIN_13_PWM_DEV` etc. are available
        - `arduino_dac`: `ARDUINO_DAC0` etc. are available
        - `arduino_uart`: `ARDUINO_UART_D0D1` or similar are available
        - `arduino_spi`: `ARDUINO_SPI_ISP` or similar are available
        - `arduino_i2c`: `ARDUINO_I2C_UNO` or similar are available
    - mechanical/electrical compatibility with specific form factors
      can now be expressed as features:
        - `aruino_shield_nano`: Arduino NANO compatible headers
        - `aruino_shield_uno`: Arduino UNO compatible headers
        - `aruino_shield_mega`: Arduino MEGA compatible headers
        - `aruino_shield_isp`: ISP header is available

This provides the groundwork to implement shield support as modules
that can rely on the I/O mappings, rather than having to provide a
configuration per board.
  • Loading branch information
maribu committed Jun 25, 2023
1 parent 970734e commit 7bdc012
Show file tree
Hide file tree
Showing 195 changed files with 4,309 additions and 4,889 deletions.
3 changes: 2 additions & 1 deletion boards/adafruit-grand-central-m4-express/Kconfig
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,8 @@ config BOARD_ADAFRUIT_GRAND_CENTRAL_M4_EXPRESS
bool
default y
select CPU_MODEL_SAMD51P20A
select HAS_ARDUINO
select HAS_ARDUINO_ANALOG
select HAS_ARDUINO_PINS
select HAS_HIGHLEVEL_STDIO
select HAS_PERIPH_ADC
select HAS_PERIPH_DAC
Expand Down
3 changes: 2 additions & 1 deletion boards/adafruit-grand-central-m4-express/Makefile.features
Original file line number Diff line number Diff line change
Expand Up @@ -15,5 +15,6 @@ FEATURES_PROVIDED += periph_uart
FEATURES_PROVIDED += periph_usbdev

# other board features
FEATURES_PROVIDED += arduino
FEATURES_PROVIDED += arduino_analog
FEATURES_PROVIDED += arduino_pins
FEATURES_PROVIDED += sdcard_spi
73 changes: 0 additions & 73 deletions boards/adafruit-grand-central-m4-express/include/arduino_board.h

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,8 @@
* @author Gunar Schorcht <gunar@schorcht.net>
*/

#ifndef ARDUINO_PINMAP_H
#define ARDUINO_PINMAP_H
#ifndef ARDUINO_IOMAP_H
#define ARDUINO_IOMAP_H

#include "periph/gpio.h"
#include "periph/adc.h"
Expand Down Expand Up @@ -49,12 +49,25 @@ extern "C" {
#define ARDUINO_PIN_14 GPIO_PIN(PB, 20)
#define ARDUINO_PIN_15 GPIO_PIN(PB, 21)

#define ARDUINO_PIN_A0 GPIO_PIN(PA, 3)
#define ARDUINO_PIN_A1 GPIO_PIN(PC, 0)
#define ARDUINO_PIN_A2 GPIO_PIN(PC, 3)
#define ARDUINO_PIN_A3 GPIO_PIN(PC, 1)
#define ARDUINO_PIN_A4 GPIO_PIN(PC, 4)
#define ARDUINO_PIN_A5 GPIO_PIN(PC, 5)
#define ARDUINO_PIN_16 GPIO_PIN(PA, 3)
#define ARDUINO_PIN_17 GPIO_PIN(PC, 0)
#define ARDUINO_PIN_18 GPIO_PIN(PC, 3)
#define ARDUINO_PIN_19 GPIO_PIN(PC, 1)
#define ARDUINO_PIN_20 GPIO_PIN(PC, 4)
#define ARDUINO_PIN_21 GPIO_PIN(PC, 5)

#define ARDUINO_PIN_LAST 21 /** D21 (A5) is the last digital pin */
/** @} */

/**
* @name Aliases for analog pins
*/
#define ARDUINO_PIN_A0 ARDUINO_PIN_16 /**< Arduino pin A0 */
#define ARDUINO_PIN_A1 ARDUINO_PIN_17 /**< Arduino pin A1 */
#define ARDUINO_PIN_A2 ARDUINO_PIN_18 /**< Arduino pin A2 */
#define ARDUINO_PIN_A3 ARDUINO_PIN_19 /**< Arduino pin A3 */
#define ARDUINO_PIN_A4 ARDUINO_PIN_20 /**< Arduino pin A4 */
#define ARDUINO_PIN_A5 ARDUINO_PIN_21 /**< Arduino pin A5 */
/** @} */

/**
Expand All @@ -67,11 +80,13 @@ extern "C" {
#define ARDUINO_A3 ADC_LINE(3)
#define ARDUINO_A4 ADC_LINE(4)
#define ARDUINO_A5 ADC_LINE(5)

#define ARDUINO_ANALOG_PIN_LAST 5 /**< A5 is the last analog pin */
/** @} */

#ifdef __cplusplus
}
#endif

#endif /* ARDUINO_PINMAP_H */
#endif /* ARDUINO_IOMAP_H */
/** @} */
1 change: 1 addition & 0 deletions boards/arduino-duemilanove/Kconfig
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ config BOARD_ARDUINO_DUEMILANOVE
select BOARD_COMMON_ARDUINO_ATMEGA
select CPU_MODEL_ATMEGA328P
select MODULE_BOARDS_COMMON_ARDUINO-ATMEGA if TEST_KCONFIG
select HAS_ARDUINO_SHIELD_UNO

config BOARD
default "arduino-duemilanove" if BOARD_ARDUINO_DUEMILANOVE
Expand Down
1 change: 1 addition & 0 deletions boards/arduino-duemilanove/Makefile.features
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
CPU = atmega328p

FEATURES_PROVIDED += arduino_shield_uno
include $(RIOTBOARD)/common/arduino-atmega/Makefile.features
1 change: 1 addition & 0 deletions boards/arduino-leonardo/Kconfig
Original file line number Diff line number Diff line change
Expand Up @@ -13,5 +13,6 @@ config BOARD_ARDUINO_LEONARDO
select BOARD_COMMON_ARDUINO_ATMEGA
select CPU_MODEL_ATMEGA32U4
select MODULE_BOARDS_COMMON_ARDUINO-ATMEGA if TEST_KCONFIG
select HAS_ARDUINO_SHIELD_UNO

source "$(RIOTBOARD)/common/arduino-atmega/Kconfig"
1 change: 1 addition & 0 deletions boards/arduino-leonardo/Makefile.features
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
CPU = atmega32u4

FEATURES_PROVIDED += arduino_shield_uno
include $(RIOTBOARD)/common/arduino-atmega/Makefile.features
2 changes: 2 additions & 0 deletions boards/arduino-mega2560/Kconfig
Original file line number Diff line number Diff line change
Expand Up @@ -13,5 +13,7 @@ config BOARD_ARDUINO_MEGA2560
select CPU_MODEL_ATMEGA2560
select BOARD_COMMON_ARDUINO_ATMEGA
select MODULE_BOARDS_COMMON_ARDUINO-ATMEGA if TEST_KCONFIG
select HAS_ARDUINO_SHIELD_MEGA
select HAS_ARDUINO_SHIELD_UNO

source "$(RIOTBOARD)/common/arduino-atmega/Kconfig"
3 changes: 3 additions & 0 deletions boards/arduino-mega2560/Makefile.features
Original file line number Diff line number Diff line change
@@ -1,3 +1,6 @@
CPU = atmega2560

FEATURES_PROVIDED += arduino_shield_mega
FEATURES_PROVIDED += arduino_shield_uno

include $(RIOTBOARD)/common/arduino-atmega/Makefile.features
5 changes: 0 additions & 5 deletions boards/arduino-mkr1000/include/board.h
Original file line number Diff line number Diff line change
Expand Up @@ -29,11 +29,6 @@
extern "C" {
#endif

/**
* @brief The on-board LED is connected to pin 6 on this board
*/
#define ARDUINO_LED (6U)

/**
* @name LED pin definitions and handlers
* @{
Expand Down
5 changes: 0 additions & 5 deletions boards/arduino-mkrfox1200/include/board.h
Original file line number Diff line number Diff line change
Expand Up @@ -40,11 +40,6 @@ extern "C" {
#define ATA8520E_PARAM_RESET_PIN GPIO_PIN(PA, 27)
/** @} */

/**
* @brief The on-board LED is connected to pin 6 on this board
*/
#define ARDUINO_LED (6U)

/**
* @name LED pin definitions and handlers
* @{
Expand Down
5 changes: 0 additions & 5 deletions boards/arduino-mkrwan1300/include/board.h
Original file line number Diff line number Diff line change
Expand Up @@ -30,11 +30,6 @@
extern "C" {
#endif

/**
* @brief The on-board LED is connected to pin 6 on this board
*/
#define ARDUINO_LED (6U)

/**
* @name LED pin definitions and handlers
* @{
Expand Down
1 change: 1 addition & 0 deletions boards/arduino-nano/Kconfig
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ config BOARD_ARDUINO_NANO
select BOARD_COMMON_ARDUINO_ATMEGA
select CPU_MODEL_ATMEGA328P
select MODULE_BOARDS_COMMON_ARDUINO-ATMEGA if TEST_KCONFIG
select HAS_ARDUINO_SHIELD_NANO

config BOARD
default "arduino-nano" if BOARD_ARDUINO_NANO
Expand Down
1 change: 1 addition & 0 deletions boards/arduino-nano/Makefile.features
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
CPU = atmega328p

FEATURES_PROVIDED += arduino_shield_nano
include $(RIOTBOARD)/common/arduino-atmega/Makefile.features
1 change: 1 addition & 0 deletions boards/arduino-uno/Kconfig
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ config BOARD_ARDUINO_UNO
select CPU_MODEL_ATMEGA328P
select BOARD_COMMON_ARDUINO_ATMEGA
select MODULE_BOARDS_COMMON_ARDUINO-ATMEGA if TEST_KCONFIG
select HAS_ARDUINO_SHIELD_UNO

config BOARD
default "arduino-uno" if BOARD_ARDUINO_UNO
Expand Down
1 change: 1 addition & 0 deletions boards/arduino-uno/Makefile.features
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
CPU = atmega328p

FEATURES_PROVIDED += arduino_shield_uno
include $(RIOTBOARD)/common/arduino-atmega/Makefile.features
7 changes: 6 additions & 1 deletion boards/common/arduino-atmega/Kconfig
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,13 @@ config BOARD_COMMON_ARDUINO_ATMEGA
select HAS_PERIPH_TIMER
select HAS_PERIPH_UART
# Various other features (if any)
select HAS_ARDUINO
select HAS_ARDUINO_ANALOG
select HAS_ARDUINO_I2C
select HAS_ARDUINO_PINS
select HAS_ARDUINO_PWM
select HAS_ARDUINO_SHIELD_ISP
select HAS_ARDUINO_SPI
select HAS_ARDUINO_UART

select HAVE_SAUL_GPIO

Expand Down
7 changes: 6 additions & 1 deletion boards/common/arduino-atmega/Makefile.features
Original file line number Diff line number Diff line change
Expand Up @@ -7,5 +7,10 @@ FEATURES_PROVIDED += periph_timer
FEATURES_PROVIDED += periph_uart

# Various other features (if any)
FEATURES_PROVIDED += arduino
FEATURES_PROVIDED += arduino_analog
FEATURES_PROVIDED += arduino_i2c
FEATURES_PROVIDED += arduino_pins
FEATURES_PROVIDED += arduino_pwm
FEATURES_PROVIDED += arduino_shield_isp
FEATURES_PROVIDED += arduino_spi
FEATURES_PROVIDED += arduino_uart
Loading

0 comments on commit 7bdc012

Please sign in to comment.