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

add SFP/SFP+ connector and cage #1214

Merged
merged 6 commits into from
May 5, 2019
Merged

add SFP/SFP+ connector and cage #1214

merged 6 commits into from
May 5, 2019

Conversation

aewallin
Copy link
Contributor

@aewallin aewallin commented Dec 25, 2018 •
edited
Loading

SMD connector and THT cage for single SFP/SFP+ transciever.
This is a standardized/generic connector - standards:

SFP is for 1 Gbit, while SFP+ is for 10 Gbit - same footprint AFAIK.

Cage e.g. TE MPN 2227302-1: https://www.te.com/commerce/DocumentDelivery/DDEController?Action=srchrtrv&DocNm=2227302&DocType=Customer+Drawing&DocLang=English
SMD Connector e.g. TE MPN 1367073-4: https://www.te.com/commerce/DocumentDelivery/DDEController?Action=srchrtrv&DocNm=1367073&DocType=Customer+Drawing&DocLang=English
SFP+ standard e.g. "SFF-8431" from https://www.snia.org/technology-communities/sff/specifications?field_doc_status_value=All&combine=8431

This footprint has a lot going on - so I will fix things one by one as I get review comments - thanks ;)

Some open questions to discuss:

  • here I suggest a footprint with SMD+THT parts together. If they would be separate it is very hard in PCB design to get the SMD-part and THT-cage positioned correctly. OTOH some designs will use dual- or quad-cages and this combo-footprint might not be useful there.
  • The TE documents suggest a keepout-area. I have not drawn it yet but it can be added.
  • Some Cage holes are optionally NPTH or Plated. I have used plated everywhere.
  • Cage-pin is a named pin - should be matched on symbol side. (alternative name it "21" - one more than the SMD pads?)

Symbol: KiCad/kicad-symbols#1327

sfp_footprint

sfp_large

sfp_small

Thanks for creating a pull request to contribute to the KiCad libraries! To speed up integration of your PR, please check the following items:

  • Provide a URL to a datasheet for the footprint(s) you are contributing
  • An example screenshot image is very helpful
  • If there are matching symbol or 3D model pull requests, provide link(s) as appropriate
  • Check the output of the Travis automated check scripts - fix any errors as required

@aewallin
Copy link
Contributor Author

only remaining travis error is now about footprint anchor. will wait for reviewer comments before (possibly) fixing.

  • Footprint center calculated as (-18.675,0.0)mm

@aewallin aewallin mentioned this pull request Dec 25, 2018
Merged
5 tasks
@DanSGiesbrecht DanSGiesbrecht added Addition Adds new footprint to library Pending reviewer A pull request waiting for a reviewer labels Dec 25, 2018
@poeschlr poeschlr self-assigned this Jan 2, 2019
@poeschlr poeschlr removed the Pending reviewer A pull request waiting for a reviewer label Jan 2, 2019
@poeschlr poeschlr removed their assignment Jan 2, 2019
@poeschlr poeschlr added the Pending reviewer A pull request waiting for a reviewer label Jan 2, 2019
@poeschlr
Copy link
Collaborator

poeschlr commented Apr 2, 2019

Denote the keepout area in accordance to http://kicad-pcb.org/libraries/klc/F4.5/

The land pattern looks good otherwise:
Screenshot from 2019-04-02 21-35-09

@poeschlr poeschlr self-assigned this Apr 2, 2019
@poeschlr poeschlr added Pending changes and removed Pending reviewer A pull request waiting for a reviewer labels Apr 2, 2019
@aewallin
Copy link
Contributor Author

aewallin commented Apr 3, 2019

keepout added.
This follows the silk-layer and dimensions of the datasheet (where given).
In principle keepout-lines could be moved by some 0.01mm if one wants strictly to follow the datasheet drawing...
Connector_SFP_and_Cage

@poeschlr
Copy link
Collaborator

poeschlr commented Apr 3, 2019

The keepout in the datasheet is for all copper except ground. The component keepout is the courtyard layer so make sure the courtyard layer is drawn to take this area into account.

@aewallin
Copy link
Contributor Author

aewallin commented Apr 4, 2019

I have now extended the keepout-zone to +/-8.125mm in Y-direction as in the datasheet.
the description is also as per datasheet "component and trace keepout (except chassis ground)"

remaining KLC error is about origin of footprint, which can be changed but I'm not sure if it improves things or not..

Connector_SFP_and_Cage

@poeschlr poeschlr added Ready for review Use this to mark pull requests that are updated but you could not review instantly and removed Pending changes labels Apr 7, 2019
@poeschlr
Copy link
Collaborator

poeschlr commented May 5, 2019

looks good, thanks

@poeschlr poeschlr merged commit b90c94c into KiCad:master May 5, 2019
@poeschlr poeschlr removed the Ready for review Use this to mark pull requests that are updated but you could not review instantly label May 5, 2019
@aewallin
Copy link
Contributor Author

aewallin commented May 5, 2019

great - thanks for the review/merge!

@antoniovazquezblanco antoniovazquezblanco added this to the 5.1.3 milestone May 6, 2019
DaToBSn pushed a commit to DaToBSn/kicad-footprints that referenced this pull request Jul 2, 2019
Merge branch 'master' of github.com:KiCad/kicad-footprints
8f0f8c9 
* 'master' of github.com:KiCad/kicad-footprints:
  adds footprints for Xilinx Zynq-7000 BGAs (KiCad#1596)
  Button_Switch_SMD: improve slide switches descriptions
  Fix hole recommendation (KiCad#1588)
  Fix hole to recommendation (KiCad#1587)
  adds footprints for Xilinx Kintex-7 BGAs (KiCad#1584)
  adds BGA footprints for Xilinx Virtex-7 FPGAs (KiCad#1569)
  Added UDFN-9 package (KiCad#1572)
  add SFP/SFP+ connector and cage (KiCad#1214)
  Added script made foot print QFN-32-1EP_5x5mm_P0.5mm_EP3.7x3.7mm (KiCad#1344)
  Remove mpn and add height from footprint name
  Add Murata 1200RS inductor footprint (KiCad#1472)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@poeschlr poeschlr

Labels
Addition Adds new footprint to library
Projects
None yet
Milestone
5.1.3
Development

Successfully merging this pull request may close these issues.
4 participants
@aewallin @poeschlr @antoniovazquezblanco @DanSGiesbrecht