-
Notifications
You must be signed in to change notification settings - Fork 676
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
There is one phase difference in UHD 3.15.0.0, but there are four phase differences in UHD 4.1.0.1 #524
Comments
Note to self: USRP-2945 == X310 + 2x TwinRX. |
@zchengyun when you say four phase differences, between which channels are you measuring the phase? |
|
@mbr0wn I use the first channel of two usrps to collect the same signal source and calculate the phase difference between the two channels. There will be four phase differences, but it has only one phase difference on uhd3.15 |
Were you able to resolve this issue? I am having the same issue with the same setup. I believe it's related to the time_sync signal passed to the fe_control module in x300_core.v: uhd/fpga/usrp3/top/x300/x300_core.v Line 610 in 8daa80c
In UHD 4.0 and beyond, it's left empty. In UHD 3.15 it's connected to a time sync signal that synchronizes the frontend when you sync to PPS. It resets the quarter_rate_downconverter module that mixes down the ADC input to baseband. The quarter_rate_downconverter cycles through 0, 90, 180, 270 degrees, so if you're not sync'd you'll see a phase offset of one of these vlues between channels. |
Definitely fixed since c70b479 |
Issue Description
Setup Details
1.CDA-2990 provides internal reference clock and PPS to four usrps.
2.Pxie-5651 and Pxie-5652 are used to provide two-stage local oscillator for four USRPs.
Expected Behavior
Using UHD 4.1.0.1, there is only one phase difference.
Actual Behaviour
There are four phase differences using UHD 4.1.0.1.
Steps to reproduce the problem
After synchronizing the reference clock, PPS and lo, use UHD 4.1.0.1 to collect one channel of different USRP and calculate the phase difference.
Additional Information
The text was updated successfully, but these errors were encountered: