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

Wider range of PLL usage #117

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Wider range of PLL usage #117

wants to merge 1 commit into from

Conversation

x44203
Copy link

@x44203 x44203 commented Feb 7, 2020

I experimentally tested the PLL range to be from 30-948 MHz and it doesn't lock when fpfd = 2 MHz and fvco = 600 MHz but it does when fvco = 300 MHz or smaller. For example CLKOP_DIV=1 CLKFB_DIV=25 CLKI_DIV=6 works while CLKOP_DIV=12 CLKFB_DIV=25 CLKI_DIV=6 does not.

I experimentally tested the PLL range to be from 30-948 MHz and it doesn't lock when fpfd = 2 MHz and fvco = 600 MHz but it does when fvco = 300 MHz or smaller. For example CLKOP_DIV=1 CLKFB_DIV=25 CLKI_DIV=6 works while CLKOP_DIV=12 CLKFB_DIV=25 CLKI_DIV=6 does not.

Signed-off-by: x44203 <[email protected]>
@smunaut
Copy link
Contributor

smunaut commented Feb 7, 2020

The user should probably be warned if operating outside of the officially supported specs. Something non-obvious could be broken ...

@daveshah1
Copy link
Contributor

@smunaut is definitely right here. I've also got no idea why you think one experiment, presumably on one device at one voltage and temperature, is more useful than a tested and guaranteed datasheet spec that Diamond also uses.

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

Successfully merging this pull request may close these issues.

3 participants