Skip to content

Commit 29914f8

Browse files
author
gychang
committed
Remove neopixel_write workaround
Turns out it was a workaround for a bad LED on one particular board.
1 parent 77b6137 commit 29914f8

File tree

2 files changed

+0
-14
lines changed

2 files changed

+0
-14
lines changed

ports/espressif/boards/espressif_esp32c3_lyra_v2/mpconfigboard.mk

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,3 @@ CIRCUITPY_ESP_FLASH_FREQ = 80m
88
CIRCUITPY_ESP_FLASH_SIZE = 4MB
99

1010
CIRCUITPY_ESP_USB_SERIAL_JTAG = 0
11-
12-
ESPRESSIF_ESP32C3_LYRA_STATUS_LED_HOLD = 1
13-
14-
CFLAGS += -DESPRESSIF_ESP32C3_LYRA_STATUS_LED_HOLD=$(ESPRESSIF_ESP32C3_LYRA_STATUS_LED_HOLD)

ports/espressif/common-hal/neopixel_write/__init__.c

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -126,21 +126,11 @@ void common_hal_neopixel_write(const digitalio_digitalinout_obj_t *digitalinout,
126126
// Update the next start to +2 ticks. It ensures that we've gone 300+ us.
127127
next_start_raw_ticks = port_get_raw_ticks(NULL) + 2;
128128

129-
#if defined(ESPRESSIF_ESP32C3_LYRA_STATUS_LED_HOLD)
130-
// Hold the pin, deleting the channel seems to glitch pixels into turning off.
131-
gpio_hold_en(digitalinout->pin->number);
132-
#endif
133-
134129
// Free channel again
135130
rmt_del_encoder(encoder);
136131
rmt_disable(channel);
137132
rmt_del_channel(channel);
138133
CHECK_ESP_RESULT(result);
139134
// Swap pin back to GPIO mode
140135
gpio_set_direction(digitalinout->pin->number, GPIO_MODE_OUTPUT);
141-
142-
#if defined(ESPRESSIF_ESP32C3_LYRA_STATUS_LED_HOLD)
143-
// Release hold
144-
gpio_hold_dis(digitalinout->pin->number);
145-
#endif
146136
}

0 commit comments

Comments
 (0)