-
Notifications
You must be signed in to change notification settings - Fork 63
Added pico-w blink example #11
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
Conversation
pico-w-blink-sdk/BridgingHeader.h
Outdated
// | ||
// This source file is part of the Swift open source project | ||
// | ||
// Copyright (c) 2023 Apple Inc. and the Swift project authors. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you update the year to 2024 for the files added?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you scale down the photo of the board? (it’s now an 8 MB jpeg) |
@kubamracek |
Looks great, and I confirmed it works on a Pico W. Thanks!!! |
I really thank you all for the hard work done in this repo and also in this repo! I do have some Picos at home and I really want to get an extra time for playing with it. Thanks a lot guys! You are awesome! |
I'm out of vacation right now and won't be able to repro, feel free to merge this 😃 |
@kubamracek @rauhul |
Thanks @yochidros, this is a great contribution :) |
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!
I recently close PR #6, but @kubamracek commented #6 (comment). I appreciate that. I added pico-w-blink-sdk entry into the top-level README.