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

Inconsistent ADS7924 P28V0_CH readback per bias tuning #388

Open
HarryMakes opened this issue Dec 7, 2021 · 8 comments
Open

Inconsistent ADS7924 P28V0_CH readback per bias tuning #388

HarryMakes opened this issue Dec 7, 2021 · 8 comments

Comments

@HarryMakes
Copy link

HarryMakes commented Dec 7, 2021

By running the bias tuning command, Booster will report different values of Ids (i.e. "the most recent P28V rail current measurements" by ADS7924) for the same given Vgs. Note that these RF channels are powered up for at least 2 minutes before their bias are tuned, which should give enough time for stabilisation for each power-cycle.

I wonder if the bias current has actually not stabilised before tuning can be done, or if the ADS7924 ADC cannot update the voltage register properly.

Examples

(Update: I'm providing a more comprehensive set of data to show that (Vgs, Ids) can vary across all channels)

Channel 0, Ids varies in 2 consecutive tuning runs given the same Vgs:

Vgs = -0.919 V, Ids = 52.45 mA
Vgs = -0.919 V, Ids = 80.52 mA

Channel 1, Ids varies in 2 consecutive tuning runs given the same Vgs:

Vgs = -0.900 V, Ids = 51.67 mA
Vgs = -0.900 V, Ids = 78.96 mA

Channel 2, Ids varies in 2 consecutive tuning runs given the same Vgs:

Vgs = -0.980 V, Ids = 50.30 mA
Vgs = -0.980 V, Ids = 70.97 mA

Channel 3, Ids varies in 2 consecutive tuning runs given the same Vgs:

Vgs = -0.960 V, Ids = 49.72 mA
Vgs = -0.960 V, Ids = 69.41 mA

Channel 4, Ids varies in 2 consecutive tuning runs given the same Vgs:

Vgs = -1.080 V, Ids = 51.67 mA
Vgs = -1.080 V, Ids = 71.94 mA

Channel 5, Ids varies in 2 consecutive tuning runs given the same Vgs:

Vgs = -0.740 V, Ids = 51.86 mA
Vgs = -0.740 V, Ids = 76.62 mA

Channel 6, Ids varies in 2 consecutive tuning runs given the same Vgs:

Vgs = -0.680 V, Ids = 49.72 mA
Vgs = -0.680 V, Ids = 73.31 mA

Channel 7, Ids varies in 2 consecutive tuning runs given the same Vgs:

Vgs = -1.080 V, Ids = 51.86 mA
Vgs = -1.080 V, Ids = 76.82 mA
@jordens
Copy link
Member

jordens commented Dec 7, 2021

Looks related to #368

@HarryMakes
Copy link
Author

Thanks @jordens. We are also looking at increasing the bias current above 50 mA to reduce output distortion.

I just retried tuning the bias multiple times in a row, and it looks like this ~20 mA discrepancy only happen on the first (Vgs, Ids) readback via the bias command. This issue seems to happen only once every time the RF channel transitions from Disabled to Powerup, and regardless of the duration from the moment the state becomes Powerup to the moment the bias current is read.

I would've assumed that it's only that the firmware doesn't report the correct bias current only on the first read after the RF changes its state to Powerup. However, during bias tuning, multiple reads from ADS7924 are occuring for each bias voltage applied, so it's unlikely the firmware simply reads a delayed ADC value.

@gkasprow
Copy link
Member

gkasprow commented Dec 8, 2021

@HarryMakes The ADC has an external lowpass filter after the mux. so it may be affected by the setting time of the mux and the filter. I assume you don't change the mux during next ADC readouts

@gkasprow
Copy link
Member

gkasprow commented Feb 7, 2022

did additional delay solve the issue?

@jordens
Copy link
Member

jordens commented Feb 8, 2022

We haven't continued debugging the hardware. I would first try to resolve #368 and then revisit this here. But it should be looked at by an RF amplifier expert.

@gkasprow
Copy link
Member

gkasprow commented Feb 8, 2022

I didn't observe such issue with old firmware.

@jordens
Copy link
Member

jordens commented Feb 8, 2022

I don't think the legacy firmware does anything like this. You would not see it.

@gkasprow
Copy link
Member

gkasprow commented Feb 8, 2022

That could be related to the AVDD issue. I replaced this resistor in new release

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

No branches or pull requests

3 participants