Skip to content

align CIPO message to COPI and CS#7

Merged
mbreiser merged 1 commit into
mainfrom
tx-timing
Jun 3, 2026
Merged

align CIPO message to COPI and CS#7
mbreiser merged 1 commit into
mainfrom
tx-timing

Conversation

@floesche
Copy link
Copy Markdown
Member

@floesche floesche commented Jun 3, 2026

  • align TX to RX
  • add more debug messages
  • version currently used for 10 panel test

@floesche floesche requested a review from mbreiser June 3, 2026 00:38
@mbreiser
Copy link
Copy Markdown

mbreiser commented Jun 3, 2026

Bench-validated against Frank's arena main (acd5286, 25 MHz), both panels on this branch (pico_v0xx_spidiag):

  • CIPO reads back 100% byte-perfect (01 30 62) on both SPI buses (50/50 each), cipo_realign=1; panels received ~15k frames/run, all sampled validity gates passing, 0 parity/queue/skip errors.
  • The busy_wait_us(2) [VERIFY] drain showed no byte truncation at 25 MHz — tag can be retired with a pointer to this run.
  • Confirms this PR composes cleanly with Frank's realignCipo (panel emits aligned bytes; arena compensates the 1-bit wire delay).

LGTM to merge. One latent edge case for a follow-up (not a blocker): prime_tx_confirmation() busy-waits unbounded with no TX-FIFO flush, so a sub-5-byte/zero-clock CS window could overflow the 8-deep FIFO and hang.

@mbreiser mbreiser merged commit 907fbe0 into main Jun 3, 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