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

Added Texas Instruments CDCVF2505 PLL clock driver #608

Merged
merged 2 commits into from
May 22, 2018
Merged

Added Texas Instruments CDCVF2505 PLL clock driver #608

merged 2 commits into from
May 22, 2018

Conversation

endofexclusive
Copy link
Contributor

Datasheet:
http://www.ti.com/lit/ds/symlink/cdcvf2505.pdf

NOTE: CLKOUT and 1Y* are 3-state outputs according to the datasheet functional block diagram.

cdcvf

The part is available in SOIC-8 and TSSOP-8 with the same pinout.

NOTE: checklib.py says CDCVF2505:CLKOUT violates KLC S4.4. However, this
is a 3-state output according to the datasheet functional block diagram.
@CLAassistant
Copy link

CLAassistant commented May 21, 2018

CLA assistant check
All committers have signed the CLA.

@evanshultz
Copy link
Collaborator

Thanks for contributing to KiCad! It's because of a willing community like you that the library can grow and improve.

You will find a lost of rules and guidelines for the KiCad library at http://kicad-pcb.org/libraries/klc/. Please see that webpage for more information about the review items below.

This is an excellent symbol! Great first contribution!

  • Can you change the description to Clock Phase-Lock Loop Clock Driver, 24-200MHz, 3.3V, SOIC-8/TSSOP-8.
  • Please move the ref des to the top left and part type to the top right.

Description updated according to recommendation:
  "Clock Phase-Lock Loop Clock Driver, 24-200MHz, 3.3V, SOIC-8/TSSOP-8"

refdes at top left, name at top right
@endofexclusive
Copy link
Contributor Author

Sure. The position policy for refdes and name was not fully clear from the KLC illustrations. Here is a second try. :-)

@endofexclusive
Copy link
Contributor Author

up

@evanshultz
Copy link
Collaborator

Thanks! That text position is one of many things that KLC isn't clear about, but we strive to address issues as they come up.

I hope to see more contributions from you in the future!

By the way, your color scheme above is quite a departure from the default KiCad look.

@evanshultz evanshultz merged commit 3890fdb into KiCad:master May 22, 2018
@endofexclusive endofexclusive deleted the cdcvf2505 branch May 22, 2018 21:37
@myfreescalewebpage myfreescalewebpage added the Addition Adds new symbols to library label May 22, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Addition Adds new symbols to library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants