sinara-hw / Phaser

Quad channel 1GS/s RF generator card with dual IQ upconverter and dual 5MS/s ADC and FPGA in EEM form factor
13 stars 4 forks source link

Phaser Baseband AA Filter Specification #158

Open Jonah-Foley opened 2 months ago

Jonah-Foley commented 2 months ago

Hi, i'm just trying to get a handle on what the actual specification for the anti-aliasing filter used on Phaser (basedband) is. The Phaser classic gateware provides a block diagram of the system where it quotes the AA filter to have a 340MHz cut-off frequency, however upon measuring this the following spectrum is seen:

image

This shows the frequency response in the range [100MHz, 500MHz] and at 300MHz we see we are about 35dB(m) down from the passband (which is correctly at ~0dB), with the actual cut-off around 200MHz.

I have looked through #4, #119. #130 and so on to try to find a clear definition of the spec but its hard to tell what decisions were made. In addition i'm having a hard time telling why the quoted bandwidth is 340MHz in the wiki if the actual bandwidth is significantly lower. Any clarity on this issue is appreciated.

gkasprow commented 1 month ago

How did you measure the filter frequency response?

Jonah-Foley commented 1 month ago

Hi Greg, the experimental setup was as follows:

The result is the trace above.

We now have a solution for this issue and my colleague will be providing full details on Tuesday 7th May.

nqtim commented 1 month ago

It looks like the issue is to do with scaling the capacitor values for a differential filter. In this ticket we can see the motivations for the original filter design. However, when scaling the single-ended filter for differential, the capacitors were doubled, whereas they actually need to be halved.

We also accounted for the 5pF parasitic capacitance on each DAC output pin (according to the datasheet) which adds an extra 2.5pF equivalent differential, so we reduced the first capacitor from 3pF to 0.5pF.

Here are the values we used: image

We modified a baseband board to these values. Here is the measured result to 500MHz with these values (ignore the bumps at DC and 500MHz): Phaser_mod_AA_filter_response

Probably a little fine-tuning or more detailed accounting of parasitics would help to level out the passband, but we didn't bother since this is good enough for us.

jordens commented 1 month ago

Ah sure. Thanks for the debugging!

gkasprow commented 1 month ago

Right! We have two capacitors connected in series with virtual ground between them. My bad!

kaolpr commented 2 weeks ago

We've modified one of our devices with C1=0.5p, C2=4.7p, and C3=3p and obtained a similar result: image The question is - is the bandwidth characteristic obtained by @nqtim satisfactory, or do we need some more work on that?