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

Add footprint for RTC RV-1805-C3 #1505

Merged
merged 8 commits into from
Apr 9, 2019
Merged

Conversation

jneiva08
Copy link
Contributor

Created footprint for RTC RV-1805-C3 from MicroCrystal

Symbol: KiCad/kicad-symbols#1687

Datasheet: https://www.microcrystal.com/fileadmin/Media/Products/RTC/Datasheet/RV-1805-C3.pdf

Footprint:
rv_1805-c3_footprint

Didn't know where to put, have created RTC.pretty because the footprint did't fit in the others


All contributions to the kicad library must follow the KiCad library convention

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
  • Give a reason behind any intentional library convention rule violation.

@jneiva08 jneiva08 mentioned this pull request Mar 27, 2019
@myfreescalewebpage myfreescalewebpage added Addition Adds new footprint to library Pending reviewer A pull request waiting for a reviewer labels Mar 27, 2019
@jneiva08
Copy link
Contributor Author

jneiva08 commented Apr 2, 2019

Was seeing other model and appear the package is the same.

RV-8523: https://www.microcrystal.com/fileadmin/Media/Products/RTC/App.Manual/RV-8523-C3_App-Manual.pdf

Probably the footprint name should be: RTC_SMD_MicroCrystal_C3_2.5x3.7mm ?

@evanshultz evanshultz self-assigned this Apr 4, 2019
@evanshultz
Copy link
Collaborator

Since it's specific to this company and it appears they call this package C3, I think what you wrote above is fine and the dimensions are even optional. Up to you. Please put the dimensions in the description and update the keywords too.

  1. Courtyard height should be +/-2.1mm.
  2. Perhaps push the silk down closer to the fab lines and make it shorter to fit between the pads.

Changed F.CrtYd and F.SilkS hight
Updated description and keywords
@jneiva08
Copy link
Contributor Author

jneiva08 commented Apr 5, 2019

Updated footprint

rtc_c3

@evanshultz
Copy link
Collaborator

Thanks! All but the silk looks good. Silk needs to clear the pads by at least the line width, or 0.12mm in this case (note that the line will then lay 0.18mm away to account for half of the line width). It moved too close to the pads now but it's placed nicely just outside the fab lines. Perhaps shorten the bottom silk line between the pads and make the top right do the same. Then on the top left the silk could "dogleg" up and over pin 1 to mark it? I can make an example if words aren't so easy to understand.

@jneiva08
Copy link
Contributor Author

jneiva08 commented Apr 7, 2019

Have changed the F.SilS
In the bottom maintained in the same place but reduced width.
In the top moved a bit (0.06mm) and made a slope to indicate pin 1.

rtc_c3

@jneiva08
Copy link
Contributor Author

jneiva08 commented Apr 7, 2019

Moved back the slope, to mask pin 1

how it were was under the component

rtc_c3

@evanshultz evanshultz mentioned this pull request Apr 8, 2019
6 tasks
@evanshultz
Copy link
Collaborator

We would still like to get 0.12mm clearance from silk to pads. Here is an idea:

  1. Move the X position ends of the top and bottom silk lines to +/-0.56mm.
  2. Move the top silk line down to 1.85mm in the Y direction (same as the bottom line).
  3. The line above pin 1 should be 2.03 in the Y direction to clear the pad by 0.12mm.
    image

Lastly, did you not find a suitable existing library for this footprint to live? I think Package_DFN_QFN would be fine.

@jneiva08
Copy link
Contributor Author

jneiva08 commented Apr 9, 2019

@evanshultz have made the modifications to have the slope 0.12mm of the pad.

C3

Still didn't changed yet the location of footprint.
I returned to see the datasheet and not the manual application, and refer in the point "terminations and processing" that the footprint is SON-10 and processing IPC / JEDEC J-STD-020C.

Probably should change to Package_SON\SON-10_3.7x2.5mm_P0.8 ?

@evanshultz
Copy link
Collaborator

Silk looks good!

Because this is a specific footprint and probably not generic, let's keep the footprint name. I'm OK to move it to the SON library, though, and please note this package name in the description.

Lastly, there are now quote marks around all strings in the footprint name. I suppose this is because you are using a new nightly? The footprint opens fine in 5.1.0 for me but syntax changes like this are something librarians need to keep in mind. In this case it's seems backwards-compatible.

Moved footprint to Package_SON
@evanshultz
Copy link
Collaborator

Thanks!

@evanshultz evanshultz merged commit 59575d3 into KiCad:master Apr 9, 2019
@jneiva08
Copy link
Contributor Author

jneiva08 commented Apr 9, 2019

Thanks for your help

@myfreescalewebpage myfreescalewebpage removed the Pending reviewer A pull request waiting for a reviewer label Apr 9, 2019
@antoniovazquezblanco antoniovazquezblanco modified the milestones: 6.0.0, 5.1.1 Apr 10, 2019
@jneiva08 jneiva08 deleted the RTC_RV-1805 branch April 16, 2019 11:29
DaToBSn pushed a commit to DaToBSn/kicad-footprints that referenced this pull request Jul 2, 2019
* 'master' of github.com:KiCad/kicad-footprints:
  Add 3.5x3.5mm QFN-20 with 2mm pad. (KiCad#1543)
  RF_Module: Fix issues of HOPERF_RFM9XW_THT (KiCad#1346) (KiCad#1347)
  Added USB_A SS-52100-001 connector (KiCad#1468)
  Add Texas RGE0024C footprint (KiCad#1395)
  Add shield pads (KiCad#1390)
  Add RF_WiFi folder and footprint support of USR-C322 3MBps WiFi module
  Add Texas RNN0018A for TPS568215 (KiCad#1466)
  Add footprint for RTC RV-1805-C3 (KiCad#1505)
  Oscillator: Add Oscillator_SMD_DiodesPericom_FN-4Pin_7.0x5.0mm (KiCad#1538)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Addition Adds new footprint to library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants