Add LSC-1 Tiny Tapeout ttsky26c integration lane - #31
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_21890cd6-61aa-4e01-91b0-952e16bf7d37) |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: be3588b9d4
ℹ️ 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".
| The ttsky26c integration top is `tt_um_lfglabs_lean_silicon_lsc1` in `src/`. | ||
| It is a thin wrapper around the unchanged `lean_silicon_lsc1` boundary in |
There was a problem hiding this comment.
Point the tapeout guide at the active LSC-1u top
Update this guide to name tt_um_lfglabs_lsc1u: the reviewed info.yaml and src/user_config.tcl both select that reduced profile, while tt_um_lfglabs_lean_silicon_lsc1 is an orphan full-core wrapper excluded from the declared source list. As written, anyone using this document to inspect or reproduce the ttsky26c integration is directed to a different design with a different protocol and source boundary.
Useful? React with 👍 / 👎.
Summary
tt_um_*ttsky26c wrapper around the unchangedasic_core/LSC-1 boundaryenais lowImmutable base
fda85f5efc6f5b460d72204b4d9c0d882960ec29df42a6211828d856e9a21ba61ecc2cd1b9dc4e71asic_core/diff from base: emptyLocal PASS evidence
make -C testcocotb Tiny Tapeout lane test: exit 0make check: exit 0make sim: exit 0tt_tool.py --create-user-config: exit 0make checksum-check: exit 0Unresolved / authoritative CI
make lean: exit 2; real Lake binary unavailablemake formal: exit 2;sbyunavailablett_tool.py --harden: exit 1 before physical design; LibreLane module unavailable andPDK_ROOTunsetTinyTapeout/tt-gds-action@ttsky26cjobs on this draft PR are the authoritative external attemptThis PR does not claim ASIC readiness and must not be merged based only on RTL results.
Note
Medium Risk
Touches tapeout metadata, active CI top, and new formal boundaries for retained arithmetic; physical signoff still depends on external GDS/GL jobs, and the profile is not interchangeable with full LSC-1 host packet logic.
Overview
Adds the LSC-1µ (LSC-1 Micro) Tiny Tapeout lane as a deliberately smaller sub-core: streamed byte opcodes for GF(2¹²⁸) XOR/MUL/SET only, with
info.yamlnow targetingtt_um_lfglabs_lsc1uon a 1×2 tile and a minimal RTL closure (lsc1u_coreplus reused multiplier blocks). Fulllean_silicon_lsc1inasic_core/stays the FPGA/larger-ASIC boundary; a separatett_um_lfglabs_lean_silicon_lsc1wrapper remains for the canonical top without changing that RTL.Verification and CI expand around the new profile: unbounded
lsc1u_protocoland cover-basedlsc1u_reachabilityformal harnesses (multiplier over-approximated where control does not depend on product),formal-mutations/check_mutations.pyin CI, a Pythonlsc1u_modeloracle with unit tests, and cocotb RTL/GL tests (Makefile.tt,test_tt.py, power-pin fixes for Icarus GL). New workflows run GDS/precheck/gl_test/viewer and a dedicated Tiny Tapeout RTL test job; consistency tools and docs (LSC1U_ARCHITECTURE.md,TINY_TAPEOUT_TTSKY26C.md) describe pins, exclusions, and evidence-driven tile sizing—not a claim that full LSC-1 packet execution is on silicon.Reviewed by Cursor Bugbot for commit be3588b. Bugbot is set up for automated code reviews on this repo. Configure here.