-
Notifications
You must be signed in to change notification settings - Fork 743
Conversation
Symbol is good for any magjack. Hide pin numbers, remove default footprints. Signed-off-by: Karl Palsson <[email protected]>
Same symbol as existing magjacks, just updated names and pin numbers. HanRun still lists this part, but doesn't offer the pdf download anymore, link to lcsc as a reputable vendor instead. Signed-off-by: Karl Palsson <[email protected]>
see also: KiCad/kicad-footprints#1091 |
Travis explanations: For the hanrun part itself: |
Have a look at my "how to make a symbol" tutorial over at the forum. I used a recent nightly and showed how to set the documentation fields. TlDr: The documentation and keyword stuff are now managed in the symbol properties dialog. For the main symbol all of it is done in the general tab. For aliases the same fields can be managed in the aliases tab. Edit: I asked the devs for clarification just in case that you are right and this can indeed not be done anymore (and i have the feeling you are right) For generic parts the datasheet field can be left intentionally empty. For this place a "~" character in it. The footprint field should be left completely empty. But the footprint filters need to be set. |
So... I just ignore the travis errors from S6.2/S6.3 then?
ok, so, not blank, if you want blank, a special not blank, blank value. ok got it :) (this is not covered in the KLC, and is completely undiscoverable, why would one not leave a value blank when one wants it to be blank?) |
For now you would need to either manually move the datasheet field from the lib file to the dcm file or use a stable release. |
even if I do that, (highly doubtful) what's the roadmap going forward? how is this meant to work in the future? You're not planning on making another repo for the symbols again, and forking it all again are you? |
Why should we make a new repo? Last time we made a new one was because the old one had the 3d models in it as well and the name kicad-library is not really fitting for something that only contains symbols. Regarding the dcm file problem: we can not accept the stuff as it is now. The devs are aware of it and there will be some fix in the future. (Will depend on how busy they are with other stuff) For now we need to accept that we need to manually change some files in a text editor until a real fix exists. (with we i mean contributors running nightly builds) |
Hi @karlp, thanks for contributing I have not yet reviewed the symbol because I want to ask @poeschlr what we should do with this contribution ? This is a new RJ45 magjack, but for me a generic one (to be reworked according to me) is enough, isn't it ? Only specific footprints should be accepted in my opinion. What do you think ? Screenshots for remembering: Existing Amphenol symbol in the lib (to be removed according to me): Generic symbol proposed by this PR: HanRun specific symbol also proposed by this PR: Cheers, |
What still needs to be done here to get this merged? I need the HanRun jack and I'd like to avoid PRing the same thing all over again. EDIT: This is also noncompliant with KLC since the HanRun magjack has a footprint, but no associated symbol |
There's no symbol, because it relies on a resolution on whether there'll be a generic magjack or not, (see the linked PR) from my point of view, take any part of this you like and submit it as your own, I consider my workk in this area to be licensed under the receivers choice of public domain, ISC, MIT, X11, Apache2, BSD 2Clause or GPLv2+ whatever amkes their life easier. I'll get back to the project this needed eventually, but the hoops here are not the top of my todo list at present. |
@poeschlr can you have a look to this PR to get a decision, particlularly at my comment #1146 (comment) ? Thanks. |
No news of the original author, indicate the PR is abandoned. |
Aren't we waiting on @poeschlr for a decision on a generic magjack? It's not abandonded. |
@myfreescalewebpage i agree with you on the idea of using a generic symbol for future additions. We might want to go with a similar pin "numbering" scheme as used with the audio jacks. Meaning use logical "names" which should then also be used on the footprint side as i guess manufacturers will have differing pin numbering schemes used in their datasheets. At least the symbols shown above give this impression. (These pin numbers would need to be documented in the KLC, see KiCad/kicad-website#433) My suggestion would be using the same "numbers" as the current names except for the shield pin where i would follow our normal standard of using pin number "SH". |
So, @myfreescalewebpage is going to take refactoring the existing symbols to a generic magjack symbol (is that tracked anywhere or is this it?), this PR can be closed, and the footprint PR KiCad/kicad-footprints#1091 can then just be rebased on @myfreescalewebpage 's new symbol when it becomes available? Have I understood correctly? |
I do not plan to make a symbol, but you can do it in this PR :) The footprint is independent. Since the beginning of this PR I personally proposed SIM card connectors (#2519). @poeschlr I have doubt common numbering can be done for Ethernet and SIM connectors, there is plenty of different possible devices. For SIM card connectors there is also double and combo connectors, etc. @karlp maybe you can make a proposition of standard Ethernet connector symbol and we can discuss. |
I'm not currently planning on spending any more time on these. |
Replace this line with your commit message! Please provide a description of your pull request
^^ commits contain commit messages, github doesn't fill them for multi commits
Thanks for creating a pull request to contribute to the KiCad libraries! To speed up integration of your PR, please check the following items: