Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

final fixes #11

Merged
merged 3 commits into from Aug 9, 2022
Merged

final fixes #11

merged 3 commits into from Aug 9, 2022

Conversation

nikitalita
Copy link
Contributor

I added back the custom rules and changed them so that:

  1. the custom rules that duplicated global rules were commented out
  2. the minimum clearance rule for the inner layers has been changed so it doesn't apply to zones, so it won't have the same copper fill bug that it did before

The custom rules catch stuff that the global rules don't, but fortunately, we didn't have anything to fix here.

I also length matched the D- and D+ lines by making D- and D+ both use vias to cross each other; I also adjusted the spacking and width of the tracks of the main run based on this calculation from eeweb:
image

With this, I think we're finally finished with this revision 馃コ

@hansfbaier
Copy link
Owner

Thanks, great 馃憤馃徎 Will look at it soon. Do you want me to put your name or GitHub handle in the board?

@nikitalita
Copy link
Contributor Author

Sure, "Nikita Lita" will be fine.

@hansfbaier hansfbaier merged commit 9a263a7 into hansfbaier:main Aug 9, 2022
@hansfbaier
Copy link
Owner

@nikitalita The Diff Pair traces on the top and bottom PHY look different. Are you sure you adjusted them both?
Which ones are the one you adjusted, the bottom one, as it looks?
image

@hansfbaier
Copy link
Owner

Yes, the bottom one as it looks. I will adjust the top one too. Ok, did it. WIll commit soon 0.09mm gap is hard on the edge of JLCPCB manufacturing capability. Let's hope it turns out right.
The bottom PHY worked great with 0.3mm width and standard spacing, and I think for that short length it is not that critical.
But it never hurts to do things properly!

@nikitalita
Copy link
Contributor Author

the spacing limit is technically 3.5mil, which is 0.0889mm, so that means we have a full 0.0011 mm to play with 馃槃
But if you think it may be an issue, 0.1mm spacing and a width of 0.38mm will also work:
image

@hansfbaier
Copy link
Owner

@nikitalita I am a bit surprised, though, that you didn't have to input the dielectric constant in this calculator.
This is a very crucial number, because it affects the whole geometry.

@nikitalita
Copy link
Contributor Author

That's what Substrate Dielectric is:
h4bMx46yED

@nikitalita
Copy link
Contributor Author

Urgh, I did only apply it to the bottom one, and I forgot to length match the top one. I just fixed this and adjusted the values of the main run according to the new calc above, PR incoming.

@hansfbaier
Copy link
Owner

That's what Substrate Dielectric is: h4bMx46yED

Ah I overlooked this.

@hansfbaier
Copy link
Owner

@nikitalita please pull first, I already adjusted the top one.

@nikitalita
Copy link
Contributor Author

I did, the top one had the main run adjusted but not the back part.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants