Skip to content

setup and la voltage save the I/O voltage to the wiring profile; drop the committed binary - #23

Merged
wardviaene merged 1 commit into
mainfrom
setup-saves-voltage
Oct 9, 2026
Merged

wardviaene merged 1 commit into
mainfrom
setup-saves-voltage

Conversation

@wardviaene

Copy link
Copy Markdown
Member

Follow-up to #22 (these changes were pushed to its branch after it was merged, so they never reached main).

  • benchpod setup and benchpod la voltage (network or embeddedci:) save la_mv to the pod's stored wiring profile on embeddedci.com when signed in and the pod is on your account: read-modify-write via GET/PUT /api/benchpod/devices/{id}/wiring, only la_mv changes, nothing written when it already matches. The server re-applies it on every connect and the web setup checklist ticks. A failed save warns and continues. Over USB, la voltage sets the pod only.
  • Removes cmd/benchpod-cli/benchpod-cli (12 MB build output committed by discover: say whether a registered pod is on the signed-in account #21) and adds it to .gitignore.
  • README and help updated.

Tests: go vet ./..., go test ./... (fwrefusals with BENCHPOD_FIRMWARE_DIR = firmware origin/main). Live on benchpod-baea06 under the pod lock: stored wiring went {"uart_rx":3} → {"la_mv":3300,"uart_rx":3}, nothing else changed.

🤖 Generated with Claude Code

…; drop the committed binary

When signed in and the pod is on your account, benchpod setup (step 3, and
after registering) and benchpod la voltage over the network also write la_mv
to the pod's stored wiring profile on embeddedci.com (read-modify-write, only
la_mv changes), so the server re-applies it on every connect and the web
checklist ticks. A failed save warns and continues. Over USB, la voltage sets
the pod only.

Also removes cmd/benchpod-cli/benchpod-cli, a 12 MB build output committed by
#21, and ignores it.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@wardviaene
wardviaene merged commit 245d0dd into main Oct 9, 2026
2 checks passed
@wardviaene
wardviaene deleted the setup-saves-voltage branch October 9, 2026 20:03
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.

1 participant