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

Correct the amplitude response of the ADC #5

Open
jonschumacher opened this issue Oct 16, 2017 · 6 comments
Open

Correct the amplitude response of the ADC #5

jonschumacher opened this issue Oct 16, 2017 · 6 comments
Assignees

Comments

@jonschumacher
Copy link
Collaborator

The CIC-Compiler has a non-flat amplitude response. This can be compensated according to this article. Additionally the input stage has some issues according to this forum entry. The original image includes a compensation filter which uses the values from this forum entry. Ideally both filters should be combined in order to achieve a minimal delay.

@jonschumacher jonschumacher self-assigned this Oct 16, 2017
jonschumacher added a commit that referenced this issue Oct 20, 2017
…the filter from the original RP image is used to overcome the flaws of the input stage.
@amarrei
Copy link
Collaborator

amarrei commented Apr 9, 2021

Hello,
I have measured the ADC inputs to calculate offset and gain correction to be done on the client side. My values are similar to the ones Tobias mentioned in another issue, so this looked promising.
This works for DC and very low frequencies, but already with a few 10s of kHz, I see an attenuation when plotting the acquisition.
After undoing my HW modifications I can see that this is not coming from my additions. I haven't yet done further measurements, but the attenuation is not negligible.
I am not sure if this is related to this dfilt1 filter or not. New to FPGA, I have not yet figured out where/how it is actually implemented in this project.
Has this issue been resolved and I have a different problem?
Thankful for any help,
Amir

@tknopp
Copy link
Owner

tknopp commented Apr 10, 2021

no, we have not resolved this issue.

@jonschumacher
Copy link
Collaborator Author

@nHackel @jusack Is this resolved now?

@nHackel
Copy link
Collaborator

nHackel commented Mar 17, 2023

Yes, the implementation is finished and will be included in the upcoming release. Only the documentation of the FIR filter is missing and will be added later

@nHackel nHackel closed this as completed Mar 17, 2023
@jonschumacher
Copy link
Collaborator Author

@jusack noticed that there is still an issue with the input stage when trying to capture square waves. This might still be related to the missing compensation filter. The link from the original post is broken but the file can be found here: https://github.com/RedPitaya/RedPitaya-FPGA/blob/e5f70e1dee2c633eab5c162cca25a085d10c7b55/rtl/red_pitaya_dfilt1.sv

@jonschumacher jonschumacher reopened this Sep 28, 2023
@jonschumacher
Copy link
Collaborator Author

@jusack Is this done?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants