cat-in-136 / ws2812-esp32-rmt-driver

WS2812 driver using ESP32 RMT for Rust
MIT License
40 stars 21 forks source link

Fix `rmt_item32_t` type name to work on ESP32-C3 #11

Closed faern closed 2 years ago

faern commented 2 years ago

The first commit just fixes a typo ws2821 -> ws2812.

The second commit changes rmt_item32_s into rmt_item32_t. rmt_item32_t is the correct type according to the ESP-IDF docs.

rmt_item32_s worked fine on the Xtensa based ESPs, since there bindgen generated a type with that name, But that generated type does not exist in esp-idf-sys when built for the RISC-V based ESP32-C3 models. I guess it could be up to exact bindgen version used or some other minor quirk. Anyway rmt_item32_s is just something bindgen made up from what I can find.