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

x300: Add API access to GPSDO UART #657

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

jholtom
Copy link

@jholtom jholtom commented Jan 25, 2023

x300: Add API Access to GPSDO UART

I added the ability to send arbitrary UART commands to the GPSDO from the UHD API.

For my use case, this enables sending commands to allow the GPSDO clock to free-run, while GPS is still locked, providing time information, but with continuous phase from the crystal. This enables interesting applications.
It also allows for getting other information, and configuring the GPSDO more than the default settings without recompiling UHD.

Which devices/areas does this affect?

x300/x310, has no affect unless the user sends a command to change the behavior of the GPSDO, and only then when the GPSDO is selected as a time or clock source.

Testing Done

Tested on 11 x300/x310 units on latest firmware.
Compared clock output with oscilloscope to check expected behavior from sending commands to the GPSDO, and comparing GPSDO responses. Behaves correctly.
Have run many long experiments on the systems since adding code, no failure condition has been observed.

Checklist

  • I have read the CONTRIBUTING document.
  • My code follows the code style of this project. See CODING.md.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes, and all previous tests pass.
  • I have checked all compat numbers if they need updating (FPGA compat,
    MPM compat, noc_shell, specific RFNoC block, ...)

@github-actions
Copy link

github-actions bot commented Jan 25, 2023

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@jholtom
Copy link
Author

jholtom commented Jan 25, 2023

I have read the CLA Document and I hereby sign the CLA

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.

1 participant