m-labs / artiq

A leading-edge control system for quantum information experiments
https://m-labs.hk/artiq
GNU Lesser General Public License v3.0
414 stars 192 forks source link

kasli: add satellite WRPLL clock recovery #2418

Closed MorganTL closed 1 month ago

MorganTL commented 1 month ago

ARTIQ Pull Request

Description of Changes

Merge Part 1 before merging this PR. This is part 2 of the WRPLL port, focusing on adding satellite WRPLL clock recovery

Changes to the zynq#282 version:

Lock test

Phase noise

TAG_OFFSET calibration

INFO(board_artiq::si549::wrpll): calibration successful, error zone: 7848 -> 9128, width: 31488 (345deg), middle of working region: 24872
INFO(board_artiq::si549::wrpll): calibration successful, error zone: 7128 -> 8424, width: 31472 (345deg), middle of working region: 24160
INFO(board_artiq::si549::wrpll): calibration successful, error zone: 6488 -> 7840, width: 31416 (345deg), middle of working region: 23548
INFO(board_artiq::si549::wrpll): calibration successful, error zone: 6504 -> 7888, width: 31384 (344deg), middle of working region: 23580
INFO(board_artiq::si549::wrpll): calibration successful, error zone: 6240 -> 7568, width: 31440 (345deg), middle of working region: 23288
INFO(board_artiq::si549::wrpll): calibration successful, error zone: 6680 -> 7960, width: 31488 (345deg), middle of working region: 23704
INFO(board_artiq::si549::wrpll): calibration successful, error zone: 6376 -> 7688, width: 31456 (345deg), middle of working region: 23416
INFO(board_artiq::si549::wrpll): calibration successful, error zone: 7664 -> 9024, width: 31408 (345deg), middle of working region: 24728
INFO(board_artiq::si549::wrpll): calibration successful, error zone: 7288 -> 8632, width: 31424 (345deg), middle of working region: 24344
INFO(board_artiq::si549::wrpll): calibration successful, error zone: 6824 -> 8152, width: 31440 (345deg), middle of working region: 23872
INFO(board_artiq::si549::wrpll): calibration successful, error zone: 6792 -> 8184, width: 31376 (344deg), middle of working region: 23872
INFO(board_artiq::si549::wrpll): calibration successful, error zone: 6192 -> 7632, width: 31328 (344deg), middle of working region: 23296

Type of Changes

Type
:sparkles: New feature
:hammer: Refactoring

Steps (Choose relevant, delete irrelevant before submitting)

All Pull Requests

Code Changes

Git Logistics

Licensing

See copyright & licensing for more info. ARTIQ files that do not contain a license header are copyrighted by M-Labs Limited and are licensed under LGPLv3+.

sbourdeauducq commented 1 month ago

no lose of lock during UART info!

The UART isn't using interrupts, so why would it impact the lock?

MorganTL commented 1 month ago

no lose of lock during UART info!

The UART isn't using interrupts, so why would it impact the lock?

On zynq, using UART will disable the interrupt. So I also test that just in case.