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

IQ amplitude phase correction #3

Open
wants to merge 64 commits into
base: master
Choose a base branch
from
Open

Conversation

F6GNJ
Copy link

@F6GNJ F6GNJ commented Feb 8, 2017

Hello,
I use RTL-FM on a raspberry pi3. I notice practically always the presence of a weak carrier accompanying the emission received in AM mode and interferences in NFM mode. When i use rtl-tcp and sdr# or sdrconsole the demodulation is perfect. I assume that IQ amplitude phase correction is involved. I saw in the source of rtlsdr:
IQ estimation / compensation (en_iq_comp, en_iq_est) * /
Rtlsdr_demod_write_reg (dev, 1, 0xb1, 0x1b, 1);
I suppose we use an internal chipset function? Is this function good enough? I did not find anything about it in RTL-FM but I'm not a specialist. Is an improvement possible?
Regards,
Arnaud

keenerd and others added 28 commits September 1, 2014 20:12
This scans 0 - 3GHz looking for frequencies that can be tuned to without
error, and refines the band edges further once a tuneable band is found. It
should work with any tuner that correctly reports tuning errors. It takes
about 1.5 minutes to complete on a R820T.
This fixes out-of-range frequencies that would set the PLL
registers to incorrect values rather than failing outright.
Remove unused variables.
Use proper type specifier.
Explicitly cast to correct type to suppress MSVC warnings.
- Link librtlsdr dynamically also on Windows.
pinkavaj pushed a commit that referenced this pull request Oct 3, 2022
@pinkavaj
Copy link
Owner

pinkavaj commented Oct 3, 2022

Hi, is this MR still relevant? I have adopted latest changes from https://github.com/librtlsdr/librtlsdr, so mey-be it will be more appropriate to open MR into that repo, althought it is not visibly upstream here.

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

Successfully merging this pull request may close these issues.

9 participants