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

Added TQFP-48-1EP_7x7mm_P0.5mm_EP3.62x3.62mm #359

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

jstjst
Copy link

@jstjst jstjst commented May 17, 2019

@pointhi
Copy link
Owner

pointhi commented May 23, 2019

Please reference the footprint PR in KiCad

@poeschlr
Copy link
Collaborator

It does not seem like there is a footprint open over on the footprint repo that would fit this contribution.

@codeclimate
Copy link

codeclimate bot commented Aug 4, 2019

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

View more on Code Climate.

TQFP-48-1EP_7x7mm_P0.5mm_EP3.62x3.62mm:
size_source: 'http://www.ti.com/lit/ml/mpqf051a/mpqf051a.pdf'
body_size_x:
nominal: 7
Copy link
Collaborator

Choose a reason for hiding this comment

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

Use dimensioning from the datasheet. This should be 6.8mm min and 7.2mm max.

num_pins_y: 12

EP_size_x:
minimum: 3.52
Copy link
Collaborator

Choose a reason for hiding this comment

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

Only the nominal size is given for the EP, so get rid of min and max.

maximum: 5.72
# EP_paste_coverage: 0.65
EP_num_paste_pads: [2, 2]
#heel_reduction: 0.1 #for relatively large EP pads (increase clearance)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Get rid of this comment line and the two lines below.

@evanshultz
Copy link
Collaborator

@jstjst

  1. You need to use the part datasheet or add that document since the EP size isn't shown in the package drawing you currently link. Add `#page=421 to the end of the datasheet link.
  2. I also added comments above.

@evanshultz
Copy link
Collaborator

@jstjst
Are you planning to come back to this PR soon?

@chschlue
Copy link
Collaborator

#422

@evanshultz
Copy link
Collaborator

@jstjst
Are you available to make the fixes above?

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

Successfully merging this pull request may close these issues.

5 participants