Skip to content

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