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

Make SYNC pullup smaller #51

Closed
whitequark opened this issue May 14, 2018 · 10 comments
Closed

Make SYNC pullup smaller #51

whitequark opened this issue May 14, 2018 · 10 comments
Labels
hardware Component: hardware revB Hardware revision: B
Milestone

Comments

@whitequark
Copy link
Member

The FPGA has ridiculously powerful drivers (what, 10 mA?) so if we use a pullup smaller than 1k we can go well into MHz range. Useful, because the PLL has a minimum frequency of 16 MHz--can we get that high?

@whitequark whitequark added the revB Hardware revision: B label May 14, 2018
@whitequark whitequark added this to the Preview 1 milestone May 14, 2018
@awygle
Copy link
Contributor

awygle commented May 22, 2018

I think 300 Ohms is the lowest we can safely go. That'll give us 300 mV of rise above ground at 3.3V with 10 mA drivers.

@whitequark
Copy link
Member Author

It's actually 24 mA. On the other hand, Vil(max) for LVCMOS 3.3 inputs is 0.8 V, so 0.3 V seems fine.

@awygle
Copy link
Contributor

awygle commented May 22, 2018

Okay, that'd give us 125 Ohms. 130 for ease of sourcing. Math math math.

@whitequark
Copy link
Member Author

Can you verify that this value actually gives a decent signal? I don't have my resistor set at hand...

@awygle
Copy link
Contributor

awygle commented May 22, 2018

I have some 120 Ohm resistors, I can look at what those give.

@awygle
Copy link
Contributor

awygle commented Jun 25, 2018

Where'd you find 24 mA as the value? I'm looking at
image and reading 8 mA.

(Just for completeness that's 375 Ohms by math and probably 390 by sourcing)

@whitequark
Copy link
Member Author

Hrm, I might have been looking at the wrong package. 300 mV rise seems pretty low too, that's not enough to sync using e.g. external LMVCMOS 3.3 circuitry... Let's aim at 800 mV rise above ground then.

@awygle
Copy link
Contributor

awygle commented Jul 10, 2018

Because it's an open drain output, 300 mV is Vol in this case - the voltage for a logic 0. The high voltage will be the full VIO voltage.

@whitequark
Copy link
Member Author

Oh right, I derped. That's fine then, let's go with 300 mV.

@awygle awygle closed this as completed in 0ecd8ce Jul 21, 2018
@awygle
Copy link
Contributor

awygle commented Jul 22, 2018

Whoops:

image

@awygle awygle reopened this Jul 22, 2018
@awygle awygle closed this as completed in 3b602a5 Jul 22, 2018
@whitequark whitequark added the hardware Component: hardware label Apr 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hardware Component: hardware revB Hardware revision: B
Projects
None yet
Development

No branches or pull requests

2 participants