-
-
Notifications
You must be signed in to change notification settings - Fork 38k
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 scorpio_rev2 to QMK #22906
Add scorpio_rev2 to QMK #22906
Conversation
|
||
#define WS2812_PWM_DRIVER PWMD1 | ||
#define WS2812_PWM_CHANNEL 3 | ||
#define WS2812_PWM_COMPLEMENTARY_OUTPUT |
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.
#define WS2812_PWM_COMPLEMENTARY_OUTPUT |
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.
Datasheet indicates that B15 only supports TIM1_CH3N, so this would require the WS2812_PWM_COMPLEMENTARY_OUTPUT
define to properly function. Master still requires the pwm advnaced config (as I targeted develop for that patch).
Since I suspect that this won't work at all, without this config, the PR should probably target develop, and have this define re-added.
Thank you for your contribution! |
Description
Types of Changes
Issues Fixed or Closed by This PR
Checklist