evidence: qualify historical ULX3S LSC-1 physical run#19
Draft
Th0rgal wants to merge 9 commits into
Draft
Conversation
# Conflicts: # .gitignore # SHA256SUMS
Member
Author
|
@codex review |
|
Codex Review: Didn't find any major issues. Chef's kiss. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
…nto agent/add-ulx3s-lsc1-harness
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
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.
Summary
This draft is stacked on PR #16 and adds qualified historical physical-run evidence plus a restricted compiled-program runner. It now also integrates PR #16 head
91ec84a, rebuilds the maintained artifacts, and records a bounded physical run of the final 1 Mbaud UART path on one ULX3S-85F.MinCoreSerialDriver,0x7fresynchronization, payload guards, and maintained RTLassert_set_xor_mulprefix through the maintained bitstreamMaintained build identity
The integrated source was rebuilt from commit
9d5741961a3851c584fc2882ae7eccd41a1ec404with manifests reportinginputs-match-revision: yes.96eb9eda7421bac902eacaeced21eee0db9a80b8f2f2effdb52b515d68e0b2e3efa908bc6b4285d5461a4d6c2c65081ce7063efc7ede080a867ddbde81edde0eThe previously documented
eb3d81...and7272b0...values did not match the committed binaries; the archive and documentation now describe the exact rebuilt and tested bytes.Physical run
All FPGA loads were volatile SRAM-only via
openFPGALoader -b ulx3s; no command used-f.0x41113043, Lattice ECP5 LFE5U-8501010f08, proving the UART SRAM image was no longer activeDisable configuration: DONE01010f08PREFIX_MATCH, explicitly not full-program PASSThe authenticated record is in
results/fpga-pr16-pr19-20260726/program-run.json. It recordsrepo_dirty: truebecause regenerated build evidence existed before the transaction; this is not presented as clean-source transaction provenance.Historical evidence and limits
The original candidate evidence remains in
results/fpga-lsc1-20260726/. Its recorded physical digest and the later reconstructed candidate digest still differ, and its USB product text/PCB revision discrepancy remains documented. It is not reclassified as a run of PR #16.Neither evidence set proves packet-v1, the unsupported JUMP suffix, complete leanVM-b execution, flash persistence, a second board, or clean source-to-bitstream identity beyond the exact manifests and tested digests.
Validation
make check: 242 tests passedmake sim: maintained stream-ALU and UART bridge simulations passedgit diff --checkpassed for source/documentation changesIncremental diff
PR #16 remains the owner of the maintained UART RTL, build flow, and driver. PR #19 owns the restricted host program runner, qualified historical evidence, maintained-bitstream physical follow-up, regression assertions, and accompanying documentation. After #16 merges, this PR can be retargeted to
mainwithout changing that ownership boundary.