forked from pinkavaj/rtl-sdr
-
Notifications
You must be signed in to change notification settings - Fork 100
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
Requesting pull for improved accuracy/features of mutability fork into keenerd. #19
Open
Xeddius
wants to merge
31
commits into
keenerd:master
Choose a base branch
from
mutability:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ion, not at kHz resolution. Also replace the manual divison loop with a simpler fixed-point calculation.
Make the tuner interface return the raw LO frequency so we can infer the necessary IF value to program the R2832 with without having to know ahead of time what IF offset the tuner uses. Remove R82XX_IF_FREQ since it's no longer needed. Round the IF divisor in rtlsdr_set_if_freq. When calculating the actual tuner frequency, round it correctly. Compute the effective tuned frequency and feed that back to the application via rtlsdr_get_center_freq on request. Try to handle various corner cases (direct sampling, offset mode) correctly. I don't have devices to test this so YMMV.
…tlsdr Conflicts: include/tuner_r82xx.h src/librtlsdr.c src/tuner_r82xx.c
Set the tuner IF filter to the actual IF used after tuning, for the case where the tuner could not tune close to the preferred IF. Set spectrum inversion in the RTL2832 based on tuner LO vs. target frequency Refuse to try to tune the R820T PLL below 27.7MHz which is where the PLL appears to get unstable.
…ntical register values.
…t fall out of the loop with div_num unset.
high-side or low-side mixing.
Treat direct sampling as a tuner with a LO of 0. Disabled no-mod direct sampling; you can now get the same effect by directly tuning now (between ~ 12-25Mhz) Move responsibility for setting the IF filter entirely into the tuner; it sets the filter as part of tuning, not as a separate operation, as the target frequency is always known.
…xes from 06af0f9 This, at least, doesn't just give me a reflected spectrum from >15MHz ant more; but I'm not sure it gives me a useful signal either.
Note that the in-spec VCO range implies a wider-than-specified FLO range.
…ime automatically.
This scans 0 - 3GHz adaptively and reports the tuneable bands it finds.
It doesn't really hurt to set the high bound too high, and some dongles do go this high.
cfstras
pushed a commit
to cfstras/rtl-sdr
that referenced
this pull request
Oct 28, 2016
Fix misc. compilation issues, update gitignore
+1 looks useful |
Would be nice to merge these and become the most up to date and accurate of all the forks... |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Higher precision tuning, tweaks, and improvements can be implemented hopefully.