Skip to content
This repository has been archived by the owner on Oct 7, 2020. It is now read-only.

Add VQFN-32-1EP_5x5mm_P0.5mm_EP3.15x3.15mm #619

Merged
merged 5 commits into from
Sep 30, 2020

Conversation

justyn
Copy link
Contributor

@justyn justyn commented Sep 10, 2020

This is for the VQFN-32 with a nominal 3.15x3.15mm EP.

It's used in this TI part:
https://www.ti.com/lit/ds/slvs589d/slvs589d.pdf#page=33

I used VQFN-32-1EP_5x5mm_P0.5mm_EP3.1x3.1mm as a basis, updating from the datasheet and removing commented lines. However I'm not sure if the conventions have changed since that part was defined.

Apologies if I made some incorrect assumptions (or indeed if the whole part is redundant).

image

image

image

image

@codeclimate
Copy link

codeclimate bot commented Sep 10, 2020

Code Climate has analyzed commit 6329435 and detected 0 issues on this pull request.

View more on Code Climate.

Comment on lines 599 to 601
paste_via_clearance: 0.1
EP_paste_coverage: 0.7
grid: [1.2, 1.2]
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you choose those values?

Copy link
Contributor Author

@justyn justyn Sep 26, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry I had not updated all the values in this section from VQFN-32-1EP_5x5mm_P0.5mm_EP3.1x3.1mm.

The grid values I have updated to 1.0, 1.0 to match the datasheet.

I note that paste_via_clearance and EP_paste_coverage have default values in the script, so for now I have removed them from this definition entirely. If that is not correct I would appreciate some guidance on how to select the values.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good choice on using defaults for the EP parameters.
I suggest to do the same with the grid.

This footprint is supposed to be build according to IPC spec (it has no Texas_ prefix). With the current values (grid, thermal_vias, EP_num_paste_pads) you are mixing both. Resulting in vias below the paste which will lead to solder loss.
Without the grid-setting, the vias are placed no-overlapping with the paste.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the explanation, I have removed the grid line to allow the defaults, as you say the vias now don't overlap with the paste.

I've added a screenshot of the ThermalVias variant to the top of the PR.

@cpresser cpresser self-assigned this Sep 26, 2020
@cpresser cpresser merged commit 6d172aa into pointhi:master Sep 30, 2020
@cpresser
Copy link
Collaborator

Is there a corresponding PR in the footprints repo. I failed to find it.

@justyn
Copy link
Contributor Author

justyn commented Sep 30, 2020

@cpresser thanks for the assistance, I have created a pull request in kicad-footprints here:

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants