Skip to content

fix(led): handle expected KTD2026 reset NACK#218

Merged
Ruben-Lohberg merged 3 commits into
2.2.4from
codex/fix-led-i2c-nack-191
Jun 29, 2026
Merged

fix(led): handle expected KTD2026 reset NACK#218
Ruben-Lohberg merged 3 commits into
2.2.4from
codex/fix-led-i2c-nack-191

Conversation

@TobiasRoeddiger

@TobiasRoeddiger TobiasRoeddiger commented May 30, 2026

Copy link
Copy Markdown
Member

Summary

  • Treat KTD2026 reset-command NACKs as expected so boot/reset no longer logs false I2C write warnings.
  • Replace hardware register reads with a local KTD2026 register cache so getColor() avoids the unsupported/problematic read path.
  • Keep the cached public LED color in sync with steady, blink, pulse, reset, and shutdown states.

Closes #191

@github-actions

Copy link
Copy Markdown

Build output available:
openearable_v2_firmware.elf.zip
openearable_v2_fota.zip

@TobiasRoeddiger TobiasRoeddiger requested review from habibialireza and removed request for DennisMoschina June 1, 2026 07:19

@mkuettner97 mkuettner97 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

works but be aware that the registers are not actually read but are cached.

@github-actions

Copy link
Copy Markdown

Build output available:
openearable_v2_firmware.elf.zip
openearable_v2_fota.zip

@github-actions

Copy link
Copy Markdown

Build output available:
openearable_v2_firmware.elf.zip
openearable_v2_fota.zip

@Ruben-Lohberg Ruben-Lohberg changed the base branch from main to 2.2.4 June 29, 2026 07:37
@Ruben-Lohberg Ruben-Lohberg merged commit f3697d8 into 2.2.4 Jun 29, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

FW: LED driver falsly claims I2C errors

5 participants