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

Added USB_C_Plug_JAE_DX07P024AJ1.kicad_mod #1365

Merged
merged 8 commits into from
Oct 6, 2019

Conversation

Misca1234
Copy link
Collaborator

This is a repush of an assumed abandoned push
#507

DX07P024AJ1 (Slim Plug)
http://www.jae.com/z-en/pdf/MB-0301-2E_DX07_PLUG.pdf

The easiest way to review this is to move the bottom F.Fab line to x, y=0,0 via "move exactly".

bild

bild

bild

bild


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.

@poeschlr poeschlr added Addition Adds new footprint to library Pending reviewer A pull request waiting for a reviewer labels Feb 3, 2019
@poeschlr
Copy link
Collaborator

Is the line that is currently on the fab layer intended to be the edge cut? If so then move it to the edge cuts layer please.

My assumption would be that the part is then outside of that outline. So that side might be best included as well to show the user where the board ede can not return. (Would suggest the full outline on the fab layer to be honest. But no silk as it is fully outside the board.)


The A side side pads should not have paste (See note 3)

I am not so sure the large shield pins are on both the A and B side. (They are not in the drawing that i assume to show the A side.)


A few dimensions are not quite right. And the drawing has at least two degrees of freedom (For the large S1 pads. Y direction is not fixed at all. The orange dimensions.)

screenshot from 2019-02-10 19-59-44

@poeschlr poeschlr self-assigned this Feb 10, 2019
@poeschlr poeschlr added Pending changes and removed Pending reviewer A pull request waiting for a reviewer labels Feb 10, 2019
@Misca1234
Copy link
Collaborator Author

Is the line that is currently on the fab layer intended to be the edge cut? If so then move it to the
edge cuts layer please.

Is it not possible to "draw line" in Edge.Cuts, nor change a line to that type, do you want me to add it manually via an text editor ?

@poeschlr
Copy link
Collaborator

poeschlr commented Feb 10, 2019

Yes the text editor might be required. (Wayne was against adding this to v5 for some strange reason. Even if it would be a hidden feature. https://bugs.launchpad.net/kicad/+bug/1251393)

Edit: To be honest i kind of gave up trying to get dev buy in. They seem to be dead set on not allowing the edge layer from within the footprint editor. I have not gotten a single satisfactory reason for it.
They even said edge cuts are ok in footprints but for some reason they do not want to give us an interface to create such footprints.

@Misca1234
Copy link
Collaborator Author

It looks like this now

bild

@Misca1234 Misca1234 added Pending reviewer A pull request waiting for a reviewer and removed Pending changes labels Feb 20, 2019
@poeschlr poeschlr added Ready for review Use this to mark pull requests that are updated but you could not review instantly and removed Pending reviewer A pull request waiting for a reviewer labels Feb 21, 2019
@poeschlr
Copy link
Collaborator

Check the edgcuts outline. There are a few points where the endpoints of the next elements do not quite meet. (marked pink)
There is also a short line in there that should not be there. (marked red, reported by DRC when trying to use the footprint.)

Screenshot from 2019-03-21 20-56-07

I did not check the measurements again as i would need to check them again after you fixed the endpoint alignment.

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

Now so, it should have been fixed

@Misca1234 Misca1234 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 6, 2019
@poeschlr
Copy link
Collaborator

poeschlr commented May 5, 2019

It seems like the JAE datasheet is not available today. Will check it again later but we might need to search an alternative.

@Misca1234
Copy link
Collaborator Author

They seemsed to have rearranged their sire, I find it again and have updated the foot print
https://www.jae.com/en/searchfilter/?topics_keyword=DX07P024AJ1&mainItemSelect=1

@poeschlr
Copy link
Collaborator

The large shield pads are only in the bottom side of the drawing. You have them on both sides.
There is also still the question on how you came up with their dimension and positioning in y direction.

There are still some dimension errors.

usb svg

@poeschlr poeschlr removed the Ready for review Use this to mark pull requests that are updated but you could not review instantly label May 30, 2019
@Misca1234
Copy link
Collaborator Author

Made some modification after review

bild

@poeschlr
Copy link
Collaborator

poeschlr commented Jun 2, 2019

And the question about the y x direction (orange dimensions in my drawing)?

Edit: Noticed that i now wrote y for the second time when i meant x

@Misca1234
Copy link
Collaborator Author

I can not find any width for the pads, more than looking how they "cover" the pads below,

I went to their site and asked for their 3D model so we can "attach" it to the foot print and see how good it looks, I have not yet got the download link for the model, I guess it is manually accepted

@poeschlr
Copy link
Collaborator

poeschlr commented Jun 2, 2019

My guess would be that the pads should have the same distance from the edge cuts on both sides so possibly move both to the left. (If you did not have any reference for them. Maybe also ask for clarification as it is kind of strange that they leave 3 degrees of freedom in their documentation.)

@Misca1234
Copy link
Collaborator Author

I got hold of the 3D model, when I apply it to the foot print and mess around a little it is clear that the "wide" shield pad is to large

bild

I could not find hoe to change the thickness of the PCB in the 3D viewer so I moved it around a little manually

bild

bild

@Misca1234
Copy link
Collaborator Author

Going to away for 5 days, will not be able to fix anything until then

@Misca1234
Copy link
Collaborator Author

@poeschlr
Should I change the footprint so it fits the 3D model?

@poeschlr
Copy link
Collaborator

Only things that are not dimensioned in the datasheet.

@Misca1234
Copy link
Collaborator Author

As you can see in the last image the left pad does not really fit, but the foot print should be according to data sheet, otherwise I can not see any changes

@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 Jun 19, 2019
@poeschlr
Copy link
Collaborator

poeschlr commented Sep 4, 2019

The edge cuts measurement i marked in my last review is still not exact.
(The leftmost edge cuts line is on -4.3025 in x direction but should be 4.3000)

Edit: The edge cuts line is also still not closed (check where the leftmost line connects to the next one.)

@poeschlr poeschlr added Pending changes and removed Ready for review Use this to mark pull requests that are updated but you could not review instantly labels Sep 4, 2019
…d point in y direction to match next edge cut line
@Misca1234
Copy link
Collaborator Author

Done

Changed the leftmost edge cut x coordinate to -4.3 and aligned the end point in y direction to match next edge cut line

@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 Oct 5, 2019
@poeschlr
Copy link
Collaborator

poeschlr commented Oct 6, 2019

thanks

@poeschlr poeschlr merged commit 3269cac into KiCad:master Oct 6, 2019
@poeschlr poeschlr removed the Ready for review Use this to mark pull requests that are updated but you could not review instantly label Oct 6, 2019
pull bot pushed a commit to maximeborges/kicad-footprints that referenced this pull request Oct 6, 2019
@antoniovazquezblanco antoniovazquezblanco added this to the 5.1.5 milestone Oct 7, 2019
@Misca1234 Misca1234 deleted the USB_C_Plug_JAE_DX07P024AJ1 branch October 9, 2019 17:33
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.

3 participants