Skip to content

Added pico-w blink example #6

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

Closed
wants to merge 1 commit into from

Conversation

yochidros
Copy link
Contributor

Hi! I tried using the pico-blink-sdk with a Pico-W, but it didn't work because the LED module is on the wireless chip. For the Pico-W, the pico-sdk doesn't define PICO_DEFAULT_LED_PIN; instead, it uses CYW43_WL_GPIO_LED_PIN.
Ref: Pico-SDK

I also tried using CYW43_WL_GPIO_LED_PIN to make the Pico-W blink. By linking several necessary modules, I was able to get it working successfully!

I thought this example might be useful to other developers, so I added it!

@yochidros
Copy link
Contributor Author

I apologize, I missed the following text.

Note that the SDK integration examples (Pico SDK, Zephyr SDK, etc.) are not recommendations or endorsement, the same is true for build system choice (Make, CMake, SwiftPM, shell scripts). Embedded Swift aims to be versatile and to allow for integration into more existing SDKs and build systems, and the example projects are merely showing the possibilities.

Therefore, I will close this PR.

@yochidros yochidros closed this May 12, 2024
@yochidros yochidros deleted the add/pico-w-blink-sdk branch May 12, 2024 15:31
@kubamracek
Copy link
Collaborator

@yochidros my apologies for not responding sooner, the only reason is that I wanted to try building and running it myself and I didn't find a time to do that... but I think this actually is a great contribution and I'd love to get it added! Pico W is a common board, and I previously did not realize that the LED is wired up differently from the Pico.

Would you be willing to re-open this PR? And perhaps even add an entry into the top-level README?

@yochidros yochidros restored the add/pico-w-blink-sdk branch May 13, 2024 09:44
@yochidros
Copy link
Contributor Author

@kubamracek
Thank you comments. I'm glad to hear that. I tried to re-open this PR, but I ended up incorporating upstream changes and adding more to the README, which led me to create a new PR. you could review this new one #11 . I'm sorry for the inconvenience.

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.

2 participants