Fluid/fluidpop-v1
1
0
Fork 0
Code Issues 17 Pull requests Projects Releases Packages Wiki Activity Actions

rtl: PopLinkPHY_Behavioral skeleton + standalone sbt build #66

Closed
navigator wants to merge 1 commit from auto/issue-51-20260525T202210Z_issue51 into main
pull from: auto/issue-51-20260525T202210Z_issue51
merge into: Fluid:main
Conversation 2 Commits 1 Files changed 6 +158 -4
navigator commented 2026-05-25 17:29:45 -03:00
Owner
Copy link

Closes #51

Summary

First Chisel module + minimal sbt project under rtl/:

  • rtl/src/pop/PopLinkPHY_Behavioral.scala --- skeleton Module + PopLinkPHYIO bundle, CHARRUA-1.2 header.
    • linkUp materialised (Bool); per-lane symbol streams, extended handshake, BER / status counters left as documented // TBD per SPEC §Interface blocks. No widths or lane counts invented (docs/spec/poplink-phy-if.md is itself a Stub; PLAN.md §13.3 does not yet pin them).
    • Output drain via io <> DontCare; SPEC §Behavior / §Invariants items pointed at follow-up issues.
  • rtl/tests/PopLinkPHY_Behavioral/PopLinkPHY_BehavioralSpec.scala --- chiseltest AnyFlatSpec with the single should "elaborate" case, AGPL-3.0-or-later header.
  • rtl/build.sbt + rtl/project/build.properties + rtl/.gitignore --- standalone sbt project, Chisel 6.5.0 / chiseltest 6.0.0 pinned to the same versions as Chipyard 1.13.0 (see rtl/chipyard.hash / ~/chipyard/build.sbt). Compile/scalaSource and Test/scalaSource are overridden to match the src/pop and tests/<Module> layout from rtl/README.md rather than sbt defaults.
  • rtl/README.md --- stale "no Scala sources, no build.sbt" line updated.

Validation (local, agent host)

  • sbt compile --- green (1 source compiled).
  • sbt Test/compile --- green.
  • sbt test --- 1 test, 1 passed (PopLinkPHY_BehavioralSpec elaborates the module).

No Chipyard checkout was required for these --- the standalone build resolves Chisel + chiseltest from Maven Central.

Out of scope (deferred to follow-up issues)

  • Behavioural model itself (line-rate, latency, injectable BER, drop / re-order knobs --- SPEC §Behavior).
  • Synthesis-translate-off guard (SPEC §Invariants).
  • Pinning PopLinkPHYIO widths / lane count / training FSM (Phase-1 PHY deliverable #2 --- PHY interface spec).
  • Coverage runs (SPEC §Coverage targets is a Phase-3 close obligation; this PR is skeleton + elaborate only).
  • Parent wiring: leaf module, no parent.

Off-limits

No ADR-017 off-limits paths touched. No commit hook bypass.

Closes #51 ## Summary First Chisel module + minimal sbt project under `rtl/`: - `rtl/src/pop/PopLinkPHY_Behavioral.scala` --- skeleton `Module` + `PopLinkPHYIO` bundle, CHARRUA-1.2 header. - `linkUp` materialised (Bool); per-lane symbol streams, extended handshake, BER / status counters left as documented `// TBD per SPEC §Interface` blocks. No widths or lane counts invented (docs/spec/poplink-phy-if.md is itself a Stub; PLAN.md §13.3 does not yet pin them). - Output drain via `io <> DontCare`; SPEC §Behavior / §Invariants items pointed at follow-up issues. - `rtl/tests/PopLinkPHY_Behavioral/PopLinkPHY_BehavioralSpec.scala` --- chiseltest `AnyFlatSpec` with the single `should "elaborate"` case, AGPL-3.0-or-later header. - `rtl/build.sbt` + `rtl/project/build.properties` + `rtl/.gitignore` --- standalone sbt project, Chisel 6.5.0 / chiseltest 6.0.0 pinned to the same versions as Chipyard 1.13.0 (see `rtl/chipyard.hash` / `~/chipyard/build.sbt`). `Compile/scalaSource` and `Test/scalaSource` are overridden to match the `src/pop` and `tests/<Module>` layout from `rtl/README.md` rather than sbt defaults. - `rtl/README.md` --- stale "no Scala sources, no build.sbt" line updated. ## Validation (local, agent host) - `sbt compile` --- green (1 source compiled). - `sbt Test/compile` --- green. - `sbt test` --- 1 test, 1 passed (`PopLinkPHY_BehavioralSpec` elaborates the module). No Chipyard checkout was required for these --- the standalone build resolves Chisel + chiseltest from Maven Central. ## Out of scope (deferred to follow-up issues) - Behavioural model itself (line-rate, latency, injectable BER, drop / re-order knobs --- SPEC §Behavior). - Synthesis-translate-off guard (SPEC §Invariants). - Pinning `PopLinkPHYIO` widths / lane count / training FSM (Phase-1 PHY deliverable #2 --- PHY interface spec). - Coverage runs (SPEC §Coverage targets is a Phase-3 close obligation; this PR is skeleton + elaborate only). - Parent wiring: leaf module, no parent. ## Off-limits No ADR-017 off-limits paths touched. No commit hook bypass.
rtl: PopLinkPHY_Behavioral skeleton + standalone sbt build
All checks were successful
build / scalafmt-check (pull_request) Successful in 4s
Details
build / sbt-compile (pull_request) Successful in 6s
Details
build / shell-lint (pull_request) Successful in 31s
Details
74c0a9f43d 
First Chisel module landing in rtl/. Skeleton-only per issue #51:
PopLinkPHYIO bundle (linkUp materialised; per-lane symbol streams,
extended handshake, BER counters intentionally left as TBD with
SPEC §Interface references --- docs/spec/poplink-phy-if.md is itself
a stub and widths / lane count / training-FSM are not pinned).
Behaviour (line-rate, latency, injectable BER, drop / re-order knobs)
lands in follow-up issues per SPEC §Behavior / §Invariants.

build.sbt is standalone, pinned to the same Chisel 6.5.0 + chiseltest
6.0.0 versions Chipyard 1.13.0 uses (rtl/chipyard.hash). Non-standard
source layout (src/pop, tests/<Module>) preserved from rtl/README.md
via Compile/Test scalaSource overrides.

Validation: sbt compile, sbt Test/compile, sbt test all green
locally (the AnyFlatSpec "should elaborate" case passes).

Licensing: CHARRUA-1.2 on the Scala source per LICENSE.md mapping;
AGPL-3.0-or-later on the test.
navigator commented 2026-05-25 19:33:38 -03:00
Author
Owner
Copy link
No description provided.
<!-- agent:pinged ts=1779748417 -->
navigator commented 2026-05-25 19:48:51 -03:00
Author
Owner
Copy link

Closing as stale. Original branch was based off main from before #59/#60/#67 landed and the diff now collides with files those PRs added. Issue #51 can be re-queued; resolver will start from current main.

Closing as stale. Original branch was based off main from before #59/#60/#67 landed and the diff now collides with files those PRs added. Issue #51 can be re-queued; resolver will start from current main.
navigator closed this pull request 2026-05-25 19:48:59 -03:00
All checks were successful
build / scalafmt-check (pull_request) Successful in 4s
Details
build / sbt-compile (pull_request) Successful in 6s
Details
build / shell-lint (pull_request) Successful in 31s
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
adr

Architecture Decision Record

  
agent:blocked-ci

CI failed on the PR

  
agent:blocked-human

Operator escalated; awaiting Marcos input

  
agent:blocked-resolver

Resolver crashed, returned empty, or timed out

  
agent:done

PR merged; issue auto-closed

  
agent:in-progress

Resolver claimed and running

  
agent:no-touch

Human-reserved; swarm Opener/Dispatcher skip

  
agent:pinged

Operator already escalated; suppress repeat pings

  
agent:pr-open

Resolver opened a PR; awaiting review/merge

  
agent:queued

Issue ready for swarm Resolver to claim

  
agent:wip

PR work-in-progress; Approver skips

  
area:board

PCB / mechanical / cooling

  
area:funding

PACID / grants / RFQs

  
area:infra

Dev env, CI, containers

  
area:phy

Physical layer / SerDes

  
area:poplink

PopLink protocol

  
area:rtl

Chisel/RTL work

  
area:software

Driver / runtime / frameworks

  
area:supply-chain

Foundry, OSAT, PCB, components

  
area:verification

Test, formal, coverage

  
ci-failed

auto-merge.sh detected CI failure

  
ci-timeout

auto-merge.sh hit CI timeout

  
docs

Documentation

  
do-not-merge

Hard hold; PR Approver skips

  
human-approved

Marcos approved via Telegram; release governance hold

  
needs-human-approval

Touches governance path; awaits Marcos label human-approved

  
needs-rebase

PR is not mergeable; rebase needed

  
needs-triage

Issue from untrusted author; Marcos triage needed

  
phase:1

Phase 1 (Section 6 of PLAN.md)

  
ready-for-review

Resolver done; PR Approver triages

  
review:findings

PR semantic review flagged blocking findings

  
review:pass

PR semantic review passed; safe for pr-approver

  
risk:tripwire

Section 0.5 tripwire fired

  
swarm:quarantined

Escalation tripwire fired; resolver tried off-limits paths

No labels
adr
agent:blocked-ci
agent:blocked-human
agent:blocked-resolver
agent:done
agent:in-progress
agent:no-touch
agent:pinged
agent:pr-open
agent:queued
agent:wip
area:board
area:funding
area:infra
area:phy
area:poplink
area:rtl
area:software
area:supply-chain
area:verification
ci-failed
ci-timeout
docs
do-not-merge
human-approved
needs-human-approval
needs-rebase
needs-triage
phase:1
ready-for-review
review:findings
review:pass
risk:tripwire
swarm:quarantined
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Fluid/fluidpop-v1!66
No description provided.