Repository navigation
TL_RESTART (realign) emits on the P4 writer (ESP-026) - #139
Merged
Merged
Conversation
…tart (ESP-026) Realign was an explicit no-op in ks_tick.c: transport_launch produced TL_RESTART but plan.transport[] held one action per output, so the writer dropped it and emitted nothing on the bar line. Add TRANSPORT_RESTART; ks_tick_out_bytes expands it to 0xFC then 0xFA, both before the downbeat 0xF8 (the ESP-038 transport-before-clock invariant). Closes the ESP-026 TL_RESTART acceptance criterion. TDD (test/test_ks_tick.c): out_bytes emits [0xFC,0xFA]; the pair precedes the first clock; and a realign posted while running + on the bar line drives TRANSPORT_RESTART through the full tick. All 60 host suites green. Claude-Session: https://claude.ai/code/session_01F7YsQgso3KNpeXbP5vNVj6
Skeyelab
enabled auto-merge (squash)
July 17, 2026 04:51
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Makes realign work on the ESP-IDF clock writer — it was an explicit no-op (
ks_tick.ccase TL_RESTART: break;).transport_launchalready producedTL_RESTART, but the plancarried one action per output so the writer emitted nothing on the bar line.
TRANSPORT_RESTARTadded toTransportAction;ks_tick_out_bytesemits0xFCthen0xFA, both before the downbeat0xF8(ESP-038 transport-before-clock invariant).ks_tick.cmaps theTL_RESTARTlaunch action toTRANSPORT_RESTART(+ tr sync).test_ks_tick.c; all 60 suites green.TL_RESTARTacceptance criterion. Additive enum value; P4/Arduinobehavior otherwise unchanged.
https://claude.ai/code/session_01F7YsQgso3KNpeXbP5vNVj6