Skip to content

Fix build issue with status led #2600

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.

Sign up for GitHub

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

Merged
merged 1 commit into from
Jul 28, 2025
Merged

Fix build issue with status led #2600

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions src/rp2_common/pico_status_led/status_led.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ static bool colored_status_led_on;

static PIO pio;
static uint sm;
static int offset = -1;
static uint offset;

// Extract from 0xWWRRGGBB
#define RED(c) (((c) >> 16) & 0xff)
Expand All @@ -52,7 +52,7 @@ static int offset = -1;
#define WHITE(c) (((c) >> 24) && 0xff)

bool set_ws2812(uint32_t value) {
if (offset > -1) {
if (pio) {
#if PICO_COLORED_STATUS_LED_USES_WRGB
// Convert to 0xWWGGRRBB
pio_sm_put_blocking(pio, sm, WHITE(value) << 24 | GREEN(value) << 16 | RED(value) << 8 | BLUE(value));
Expand Down Expand Up @@ -164,9 +164,9 @@ void status_led_deinit(void) {
status_led_context = NULL;
#endif
#if COLORED_STATUS_LED_USING_WS2812_PIO
if (offset >= 0) {
if (pio) {
pio_remove_program_and_unclaim_sm(&ws2812_program, pio, sm, offset);
offset = -1;
pio = NULL;
}
#ifdef PICO_DEFAULT_WS2812_POWER_PIN
gpio_put(PICO_DEFAULT_WS2812_POWER_PIN, false);
Expand Down
Loading