Skip to content

fix: lock I2C mutex while draining NotePing input#246

Merged
zfields merged 1 commit into
blues:ray/autobaudfrom
zakoverflow:zak/pr239-i2c-mutex
Jun 5, 2026
Merged

fix: lock I2C mutex while draining NotePing input#246
zfields merged 1 commit into
blues:ray/autobaudfrom
zakoverflow:zak/pr239-i2c-mutex

Conversation

@zakoverflow

Copy link
Copy Markdown
Contributor

Targets #239.

This addresses only the I2C mutex handling in NotePing():

  • locks the I2C mutex around the pre-ping I2C drain path
  • unlocks on normal and error exits from that drain path
  • adds focused NotePing coverage that verifies I2C receive runs while the mutex is held and serial ping does not take the I2C mutex

Architecture docs: not updated; this is a local mutex-scope fix and does not change module boundaries, public APIs, or transport contracts.

Tests:

  • docker CI image: ./scripts/run_unit_tests.sh (166/166 passed)

@zfields zfields merged commit 9910a5d into blues:ray/autobaud Jun 5, 2026
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.

2 participants