fix(lcc): default to 1.27mm LCC/PLCC pitch so pads stop overlapping - #795
Closed
zkasuran wants to merge 1 commit into
Closed
fix(lcc): default to 1.27mm LCC/PLCC pitch so pads stop overlapping#795zkasuran wants to merge 1 commit into
zkasuran wants to merge 1 commit into
Conversation
lcc set pl and pw but not p, so it inherited the generic quad 0.5mm pitch. With 0.6mm-wide pads on a 0.5mm pitch, neighbouring pins overlapped by 0.1mm, a short in the emitted PCB. LCC and PLCC are a 1.27mm pitch family (JEDEC MO-047 / MS-034), which the lcc68 parity reference already uses. Default the pitch to 1.27mm when the caller gives none, the same way pl and pw are defaulted. Explicit pitches still win. Adds a regression test that no lcc pads overlap and a KiCad parity test for lcc44 against PLCC-44.
Author
|
Closing this. Refocusing on building real boards rather than footprint tweaks. Thanks for the steer. |
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.
Closes #794
What changed
lcc set pl and pw but never p, so it fell back to the generic quad default pitch of 0.5mm. With 0.6mm-wide pads on a 0.5mm pitch, neighboring pins overlap by 0.1mm, which is a short in the emitted PCB. Every bare lcc size is affected (lcc20, lcc28, lcc44, lcc68).
LCC and PLCC are a 1.27mm pitch family (JEDEC MO-047 / MS-034). The repo already uses that pitch in the lcc68 parity test against PLCC-68_24.2x24.2mm_P1.27mm. This PR defaults the lcc pitch to 1.27mm when the caller gives none, in the same block that already defaults pl and pw. An explicit pitch still wins, so lcc68_..._p1.27mm and any custom pitch are unchanged.
Repro (before this PR)
After the fix the same pads sit 1.27mm apart with 0.67mm of clearance.
Tests
Verification
Run locally on this branch:
AI assistance (Claude) was used in developing this change. The design, review and verification were done by the author. Verified locally before submitting: bun test (524 pass, 0 fail), bunx biome format . (clean), bun run build (success).