Skip to content

7 add neopixel status led driver#47

Open
Ananya-Jajodia wants to merge 15 commits intomainfrom
7-add-neopixel-status-led-driver
Open

7 add neopixel status led driver#47
Ananya-Jajodia wants to merge 15 commits intomainfrom
7-add-neopixel-status-led-driver

Conversation

@Ananya-Jajodia
Copy link
Copy Markdown

Adds status leds to indicate configuration state of power gate, wifi, and sensors (imu/lidar)

@Ananya-Jajodia Ananya-Jajodia linked an issue Apr 22, 2026 that may be closed by this pull request
Copy link
Copy Markdown
Member

@usedhondacivic usedhondacivic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Couple of comments

Comment thread mote-ffi/mote_link/mote_ffi.pyd Outdated
Comment thread mote-firmware/src/tasks/status_led/led_driver.rs Outdated
Comment thread mote-firmware/src/tasks/status_led.rs Outdated
Comment thread mote-firmware/src/tasks/status_led.rs Outdated
Comment thread mote-firmware/src/tasks/status_led.rs Outdated
Comment thread mote-firmware/src/main.rs Outdated
Copy link
Copy Markdown
Member

@usedhondacivic usedhondacivic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two small things

Comment thread mote-ffi/mote_link/mote_ffi.pdb Outdated
Comment thread mote-ffi/Cargo.toml Outdated
Copy link
Copy Markdown
Member

@usedhondacivic usedhondacivic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add neopixel status LED driver

2 participants