-
Notifications
You must be signed in to change notification settings - Fork 739
Cologne Chip GateMate CCGM1A1 FPGA symbol is created #2578
base: master
Are you sure you want to change the base?
Conversation
a4e0a58
to
b62a598
Compare
Closing/opening to refresh the Travis test. |
Hi @ak-fau , thanks for contributing, A few comments I have during my review:
Unit A:
Unit G:
Units H and I:
I attach the excel sheet I used for the review (extract of datasheet): Cheers, |
Everything should be correct now, with possible exception of a footprint name -- still waiting for a review comments on a separate PR with the footprint.
Hello Joel, thank you for your review and comments.
Understood, a PR with the footprint is still pending review.
It is now unit G
An error in the data sheet with pin J1 (IO_W2_B3) seems to
Best regards, Anton |
Thanks for the fixes! I think now just need to wait the footprint. |
Unfortunately there will be more changes. But it has to wait for new documentation from CologneChip to become available. |
@cp-aquila thanks for the info. This is exactly why we always wait for the footprint to be integrated before merging the symbol, because the definition of the footprint may change. In this case of course the symbol PR should be updated. |
Cologne Chip GateMate CCGM1A1 FPGA symbol is created
A separate PR is created for the kicad-footprints library to add BGA-320 footprint
(18x18 matrix with 4 balls removed), KiCad/kicad-footprints#2183.
Datasheet
Schematic page with all parts of the CCGM1A1 symbol
KLC violations:
S3.1 for unit 8 -- to make the unit itself symmetrical
S4.2 too many ground and power pins to place on top/bottom of a unit
S5.1 corresponding footprint is submitted with pull request KiCad/kicad-footprints#2183
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: