Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add senseBox MCU-S2 #9263

Merged
merged 2 commits into from
Feb 21, 2024
Merged

Conversation

mariopesch
Copy link
Contributor

add senseBox MCU-S2 with ESP32S2.
We already added the board to the 3.0 release, but would also like to integrate it to the latest 2.0.15 stable release.


Description of Change

Adds a new variant directory for senseBox MCU-S2 with ESP32S2. TinyUF2 boot loader is included (see: adafruit/tinyuf2#360) Add sensebox_mcu_esp32s2 in boards.txt

Tests scenarios

Tested with Arduino-ESP32 Core v2.0.14 and senseBox MCU-S2

Related links

Website: https://sensebox.de/

Allocation for PID under Espressif VID used in this PR: espressif/usb-pids#121

add senseBox MCU-S2 with ESP32S2
Copy link
Member

@P-R-O-C-H-Y P-R-O-C-H-Y left a comment

Choose a reason for hiding this comment

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

Just one small fix needed for back-porting it to 2.x

#define USB_FW_MSC_PRODUCT_REVISION "1.00" // max 4 chars
#define USB_FW_MSC_VOLUME_NAME "senseBox" // max 11 chars
#define USB_FW_MSC_SERIAL_NUMBER 0x00000000

Copy link
Member

Choose a reason for hiding this comment

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

As its 2.x branch, you are missing this part:

#define EXTERNAL_NUM_INTERRUPTS 46
#define NUM_DIGITAL_PINS        48
#define NUM_ANALOG_INPUTS       20

#define analogInputToDigitalPin(p)  (((p)<20)?(analogChannelToDigitalPin(p)):-1)
#define digitalPinToInterrupt(p)    (((p)<48)?(p):-1)
#define digitalPinHasPWM(p)         (p < 46)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@P-R-O-C-H-Y thank you! Just added the missing stuff.

@P-R-O-C-H-Y P-R-O-C-H-Y added the Status: Pending Merge Pull Request is ready to be merged label Feb 19, 2024
@me-no-dev me-no-dev merged commit 381d7ef into espressif:release/v2.x Feb 21, 2024
42 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Pending Merge Pull Request is ready to be merged Type: 3rd party Boards
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants