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

feat(targets): ✨ Add Raspberry Pi Pico W #113

Open
wants to merge 2 commits into
base: Main-Trunk
Choose a base branch
from

Conversation

ZZ-Cat
Copy link
Owner

@ZZ-Cat ZZ-Cat commented Apr 15, 2024

Overview

This Pull Request adds support for the Raspberry Pi Pico W

@ZZ-Cat ZZ-Cat added ✨️ Enhancement ✨️ New feature or request ...in progress 🚧 Development on this is in progress labels Apr 15, 2024
@ZZ-Cat ZZ-Cat added this to the Version 1.1.0 milestone Apr 15, 2024
@ZZ-Cat ZZ-Cat self-assigned this Apr 15, 2024
@ZZ-Cat ZZ-Cat linked an issue Apr 15, 2024 that may be closed by this pull request
5 tasks
@ZZ-Cat ZZ-Cat mentioned this pull request Apr 15, 2024
29 tasks
@ZZ-Cat
Copy link
Owner Author

ZZ-Cat commented Apr 15, 2024

Right-oh, @britannio.

Can you check out the ZZ-Cat/issue109 branch, flash one of the examples onto your Raspberry Pi Pico W and let me know if it's working for you?

If anything crops up, you may go into CFA_Config.hpp and turn on CRSF_DEBUG_ENABLED and CRSF_DEBUG_ENABLE_COMPATIBILITY_TABLE_OUTPUT, re-flash your target, screenshot the results in the Serial Monitor and post those screenshots here.

Aside from #108, by rights there should not be any other issues.

@britannio
Copy link

Thanks, I'll test it out tomorrow (was away for a few days).
I'm now using https://github.com/britannio/pico_crsf but it's fair that I ensure the feature I requested works.

@ZZ-Cat
Copy link
Owner Author

ZZ-Cat commented Apr 17, 2024

Yeha.

I plan on adding more RP2040 targets soon, because this is something that's requested a lot (it's a close second to ESP32 targets).
What caused me to hold off on it was an apparent falling out between PlatformIO and Raspberry Pi Trading; and about a year ago, there wasn't any way to add Earle Phillhower's Arduino-Pico to PlatformIO... until I found out very recently that it's been integrated.

It's a slightly similar situation with STM32 support as well. I want to add in STM32duino support either alongside what I already have or replace it entirely. As far as I am aware, there is yet to be PlatformIO integration for STM32duino.

I believe that the more targets CRSF for Arduino is compatible with, this increases the number of applications it can be used in, and people can pick whichever hardware fits their budgets and needs.

@ZZ-Cat ZZ-Cat added the ⤴️ Rebase Needed ⤴️ This needs to be rebased. label Apr 19, 2024
@ZZ-Cat ZZ-Cat removed the ⤴️ Rebase Needed ⤴️ This needs to be rebased. label Apr 26, 2024
@ZZ-Cat ZZ-Cat added ⤴️ Rebase Needed ⤴️ This needs to be rebased. and removed ...in progress 🚧 Development on this is in progress labels Jul 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨️ Enhancement ✨️ New feature or request ⤴️ Rebase Needed ⤴️ This needs to be rebased.
Projects
Status: On hold
Development

Successfully merging this pull request may close these issues.

Raspberry Pi Pico W
2 participants