zephyrproject-rtos / gsoc-2022-arduino-core

Arduino Core Zephyr Module (GSoC 2022 Project)
Apache License 2.0
44 stars 11 forks source link

Move board definitions to `variants` folder #16

Closed beriberikix closed 2 years ago

beriberikix commented 2 years ago

Currently the pin mapping for the nano 33 ble is located in the Arduino core itself. Per the Arduino platform specification, these kind of files should be in their own directory under a variants folder (see mbed for an example.)

9 should be updated once complete.

DhruvaG2000 commented 2 years ago

Is this the structure we are aiming for @beriberikix ?


└─❯ tree . -L 3
.
├── CMakeLists.txt
├── CODE_OF_CONDUCT.md
├── cores
│   ├── arduino
│   │   ├── api -> /home/dhruva/temp/ArduinoCore-API/api
│   │   ├── Arduino.h
│   │   ├── CMakeLists.txt
│   │   ├── main.cpp
│   │   └── zephyrCommon.cpp
│   └── CMakeLists.txt
├── Kconfig
├── LICENSE
├── README.md
├── samples
│   ├── blinky_arduino
│   │   ├── CMakeLists.txt
│   │   ├── prj.conf
│   │   ├── README.rst
│   │   ├── sample.yaml
│   │   └── src
│   ├── button_press_led
│   │   ├── CMakeLists.txt
│   │   ├── prj.conf
│   │   ├── README.rst
│   │   ├── sample.yaml
│   │   └── src
│   └── hello_arduino
│       ├── CMakeLists.txt
│       ├── prj.conf
│       ├── README.rst
│       ├── sample.yaml
│       └── src
├── variants
│   ├── ARDUINO_NANO33BLE
│   │   ├── arduino_nano_33_ble.overlay
│   │   └── arduino_nano_ble_sense_pinmap.h
│   └── CMakeLists.txt
└── zephyr
    └── module.yml
beriberikix commented 2 years ago

Yes, that looks great!

beriberikix commented 2 years ago

Closed with #17