rtl: integrate packetized LSC-1 on ULX3S#24
Conversation
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_d1092f2e-2494-4a9b-82cb-dd6320520f1d) |
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_819cf02f-505d-43aa-a8b4-99c5d85fc0ec) |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a119ce2c15
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
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".
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_5cfdfee5-b1d1-4031-8915-cae81c6fd2bf) |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4c7f4a3e80
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
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".
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_1602aa5e-fcfc-4d76-8c96-af8bbd6d0eb4) |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1f9d185ade
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
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".
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_53be3f7d-c87f-4206-8342-a21ab7579a48) |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: df8ca93e7a
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
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".
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_8d81ce6b-a54a-46dd-bc14-7e54b0bccfbe) |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ddfc84345b
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
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".
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_1149c8f0-8ddf-4f80-b28e-b506fbd21e7b) |
What changed
lean_silicon_lsc1the packetized product top while preserving a separate raw MinCore diagnostic toplsc1_field_encoder, reusing the streaming multiplier to verify encoded JUMP destinations0xffreception after framing errorsc2ea7f1b299d5e293eaa1524335e671d78dc4999Root causes fixed
The initial adapter deadlocked SET/XOR because those MinCore operations emit output during payload ingestion. The UART receiver also re-armed while BREAK was still low, producing a spurious
0xffandBAD_SOF. Finally, the 94-byte packet buffer could not hold the 103-byte JUMP request. These paths now have regression coverage.Validation at head
33b4c81416d367e230b4ecb28b1c857d43f4bbabPhysical ULX3S evidence
SRAM-only validation was performed on LFE5U-85 IDCODE
0x41113043at 1 Mbaud; no flash write was performed.af78dcea8ac2a493b50f1a7fc293c20ed81f4052bdb58c6443c58e46da76d723assert_set_xor_mulcompletes all 13 records, including JUMP, reaches PC 15 and reports fullMATCHThis physical image was produced before the final merge-only reconciliation commit. The interrupted exact-
33b4c81rebuild must be rerun on the new hardware host before claiming an authenticated exact-head package.Current limits / handoff
Note
High Risk
Large RTL and protocol-surface change to the product top, transaction semantics, and host runtime; correctness relies on simulation/differential evidence rather than full formal refinement.
Overview
lean_silicon_lsc1is now the packetized product top: it instantiateslsc1_packet_frontendinstead of rawleanvm_b_stream_alu, with the old stream-ALU harness moved tolean_silicon_lsc1_mincorefor diagnostics.New RTL glue (
lsc1_stream_adapter,lsc1_field_encoder) drives MinCore over the existing 8-bit stream and re-encodes pointer witnesses for DEREF/JUMP. The packet frontend grows to the full scalar subset—NEGOTIATE, STATUS, SET/XOR/MUL, DEREF Cell/Pc/Fp, JUMP, staged RETIRE—via the stream adapter instead of a dedicated GF128 multiplier block; RX payload storage widens to 256 bytes for JUMP frames.Host and validation extend through DEREF/JUMP (frozen fixture now full
MATCH), plus RTL↔Python byte-exact differential tests, packet UART sims, and a physicallsc1_packet_uartdriver. FPGA adds packet-mode UART (BREAK abort, no0x7fin-band abort),ulx3s_packet_top/ build recipe, and CI Yosys closure over the packet RTL stack.Reviewed by Cursor Bugbot for commit dcd6bf3. Bugbot is set up for automated code reviews on this repo. Configure here.