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 s2 mini dev board support #364

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

lewibt01
Copy link

Adding support for ESP32-S2FN4R2 mini dev boards. Serial0 defaults to pins 43/44, which aren't made available to the end user on the breakout board. I've added a new board definition that allows users to selectively change these pins to ones that are accessible on the dev board.

justcallmekoko and others added 4 commits October 17, 2023 17:14
Created a new board definition for ESP32-S2 mini dev boards

Conditionally changed the default Serial0 pins to ones that are available on the mini dev board. Other boards are not affected.

Added a preprocessor directive to exclude softwareserial in GpsInterface.h, as it caused compilation issues with ESP32-S2FN4R2 boards.
@yyewolf
Copy link

yyewolf commented Jan 8, 2024

Have you managed to make this work ? I have a modified version of the firmware that doesn't work.. I disabled BT because the ESP32 S2 N4R2 doesn't have it and got to the CLI ">" displayed on the Flipper (on reset) but commands don't work..

@lordrak007
Copy link

Hi,
several months ago i change only one line in code (Serial.begin(115200) -> Serial.begin(115200, SERIAL_8N1, 17, 18);) and i was able to use Marauder with thi device. (Lolin S2 mini based on ESP32-S2FN4R2)
But with latest source code does not working bacause something changed and i am getting error
esp32_marauder:190:42: error: no matching function for call to 'USBCDC::begin(int, SerialConfig, int, int)' Serial.begin(115200, SERIAL_8N1, 17, 18);
Changed Serial to Serial1 also does not working.
Do you have any sugestion to to solve it? Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants