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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add protection resistor between fpga and the level shifter for the sync connector #126

Closed
wants to merge 1 commit into from

Conversation

electroniceel
Copy link
Member

My implementation for #125.

I had to move U32 and some tracks a bit to make room for the resistor. But I think there is enough clearance everywhere now.
top
3d

@marcan
Copy link
Member

marcan commented Apr 13, 2019

I know why you moved the via next to U24, and I don't blame you for it, but I'm going to try to find a way of doing this without that :-)

@marcan
Copy link
Member

marcan commented Apr 13, 2019

My take:

image

image

This has the nice property that it also orients the shifter the same as the others.

@electroniceel
Copy link
Member Author

what is the problem with moving the via?
I think there is enough clearance between the via and the pad so that no solder is sucked into the via. Or is it the beauty of the vias being in a line & symmetrical?

I think the downside of your solution is the 3v3-via between the pads of U32: it seems to touch the pads, so solder could be sucked down.

@marcan
Copy link
Member

marcan commented Apr 13, 2019

The vias ought to be tented, so that shouldn't be an issue. We already have a few other vias closer to the pads than that without trouble.

It's a beauty thing; all the shifters have an identical layout. If I really wanted to move that via closer I'd end up moving them all to match.

@electroniceel
Copy link
Member Author

Ok, yours is more beautiful.

But you could still move the 3v3-via a bit up & right without problems. So even if the tenting isn't complete it doesn't create a problem. Via tenting tends to be not that reliable in my experience. You need plugged vias for that, but that is $$$

@marcan marcan closed this in 4bc3dee Apr 13, 2019
@electroniceel electroniceel deleted the sync-r branch April 22, 2019 10:30
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