-
Notifications
You must be signed in to change notification settings - Fork 7
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
clock link AMC-to-RTM #66
Comments
Can't the FPGA do that clock fanout? Or are you worried about noise? |
Having the fpga do the fanout was deemed too noisy. |
@jordens Can you remind me why we can't use the SI5324 on the RTM for this. |
That's one level of FPGA plus serial link plus FPGA plus SI5324 noisier. But fine by me if the jitter is still good enough. I forgot that we had that thing there as well. |
And I would like to seize this opportunity to point out again that this desire to squeeze so many channels into this has lead to a big and otherwise unnecessary increase in complexity, delays, and mission creep. |
"might as well be hanged for a sheep as a lamb"... presumably, once we've been through one set of fibre-cdr-SI324, the added jitter from going through it all a second time will only be ~3dB, which isn't likely to be significant. IHMO, this is a reasonable price to pay for reducing the amount of AMC-RTM wiring -- if we actually cared about the noise in this clock path, then it'd be better to use a lower-noise component than the SI5324... @jordens what frequency are SI_CLK_OUT_1/SI_CLK_OUT_2? Which one should we use as the 100MHz reference for the HMC830? |
Note that the SI5324 on the RTM side might operate at a submultiple of the RTIO clock, or even asynchronously to the RTIO clock in the first gateware implementations. |
@sbourdeauducq Okay, would you prefer to use the AMC SI5324 then, and route the recovered 100MHz clock over the AMC-RTM connector? If so, which output from the AMC SI5324 would you like to route to the RTM? |
@hartytp I don't see how that's a significant simplification. It's adding a fan-out on the AMC and routing one signal over the connector versus adding a fan-out on the RTM and routing it there. The frequencies will be dictated by the RTM-AMC data link clocking. We want both available to the FPGAs. |
@jordens It's not a significant simplification, just a minor convenience: it seems sensible not to route a signal over the AMC<=> RTM connector if we are already generating it on the RTM. But, if there's a reason to prefer to use the SI5234 on the AMC then go for it. Can you confirm which output (1/2) from which SI5324 (AMC/RTM) you want to be used as the 100MHz source in CDR mode, please? |
The ones going to the fabric. SI_CLK_OUT2 on the RTM or SI_CLK_OUT1 on the AMC in 1.0rc1. With fan-outs. |
Any preference over RTM/AMC (or, are they equivalent as far as you're concerned)? |
Doesn't make much of a difference to me. @sbourdeauducq should weigh in regarding the likelihood of actually spitting out 100 MHz on either. AFAICT the frequency doesn't need to be 100 MHz because the HMC830 can cope. |
@jordens True, the HMC830's N- and R-dividers are 19-bit and 14-bit respectively, so this is unlikely to be a problem for any vaguely sensible SI5234 frequency. In that case, I propose the following:
Objections? |
AFAICT for our purposes we can always arrange the gateware for the AMC-RTM link clock to be derived from the same oscillator as the RTIO clock. |
@sbourdeauducq ACK. Modified proposal is then:
|
To make life easier for @gkasprow, I'll close this issue and raise a new summary issue... |
2016-11-11 google hangouts consensus item:
For stand-alone operation (in a minimal enclosure or on the bench), it's useful and convenient to not have to provide a phase locked 100 MHz clock on the minimal rf backplane or the RTM frontpanel but to be able to use the recovered and jitter-cleaned RTIO clock to drive the clock tree on the RTM.
Therefore, the one output of the SI5324 synth/jitter cleaner on the AMC that goes to FPGA general purpose should go into a fan-out and then into an SMA that can be connected to another SMA on the RTM to feed the HMC830.
This obviously assumes that there is no backplane that's in the way of the SMA cable.
The text was updated successfully, but these errors were encountered: