Route all 47 AM3352 signals with the public bus_lanes phase - #4237
Merged
seveibar merged 6 commits intoSep 30, 2026
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
seveibar
marked this pull request as ready for review
September 30, 2026 21:43
seveibar
merged commit Sep 30, 2026
998f772
into
docs/bus-lanes-integrated-dogbones
16 checks passed
This branch was successfully deployed
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.
The original AM3352/RAM board now routes all 47 signals with zero native DRC errors through the public phase:
The fixture preserves the reference TSX's footprints, placement, 47 connections, two byte buses, three differential pairs, and timing constraints. It contains no custom algorithm or saved route plan. Automatic dogbones apply only to untouched component pads; existing fanout handoffs stay fixed.
Start review with
tests/features/autorouter-bus-lanes-am3352.test.tsxand its three signal-layer SVG snapshots. Every snapshot is written only after these gates pass:The source-bus assertion includes both byte buses and all three exported differential-pair memberships. The snapshot views explicitly select inner1, inner2, and bottom so the carrier routing is visible. No partial/unrouted AM3352 artifacts are included.
Validation: fresh TSX integration run passed in 281 seconds with 169 assertions; six smaller bus_lanes integration tests and core typecheck pass. The test timeout is 900 seconds for this dense regression. The solver has 49 passing tests and four passing DDR benchmarks; see solver #3.
Dependencies: props uses the built, commit-pinned PR preview from props #875 in both devDependencies and overrides. Replace this preview with its npm release before merge. The dist build and smoke test pass from a clean CI install. The dense AM3352 test has a dedicated Ubuntu ARM job matching the snapshot-update runner, with all 169 assertions and zero image-diff tolerance. Other tests continue in the ten x86 shards. ARM and x86 can select different valid control paths due to floating-point geometry differences. The tested solver is pinned to
de425372f6413e024010d24b64341d64c83a6a12, which uses fanout-solver #252. This PR is stacked on planning #4232.CI is green on
fa67b539ed24f804820a4e8af5172d5b81a9bf92: all ten x86 test shards, the dedicated AM3352 ARM routing/snapshot job (169 assertions), dist build/smoke test, and Vercel preview pass. Test run. No routing assertions, DRC gates, image tolerances, or snapshot geometry were weakened or changed.