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

CLKGEN (Si570) driver could get out of frequency lock #19

Open
kaklik opened this issue Aug 8, 2021 · 0 comments
Open

CLKGEN (Si570) driver could get out of frequency lock #19

kaklik opened this issue Aug 8, 2021 · 0 comments

Comments

@kaklik
Copy link
Member

kaklik commented Aug 8, 2021

There is a wrong condition in checking "small frequency change"

Instead of this, the driver should check the frequency change against the last rough tuning.

The second issue is the driver tunes from the last tunned frequency. Therefore it accumulates the frequency error.

Both problems could be resolved by maintaining the internal tuning status in the CLKGEN object.

  • save calibration frequency read by recall_nvm ()
  • save frequency of the last rough tuning

Prvně teda v set_freq porovná novou frekvenci vůči poslednímu pomalýmu ladění, a pokud musí znovu pomalu ladit, tak si vezme kalibraci poznamenanou v recall_nvm, a ignoruje tu současnou frekvenci předanou jako jeden z argumentů pro set_freq

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

No branches or pull requests

1 participant