-
Notifications
You must be signed in to change notification settings - Fork 41
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
External down converter support ? #127
Comments
This is a cool idea. Using an SI5351 would certainly simplify things. You could then tune the Tayloe detector to a fixed frequency that is an exact division of the clock frequency. I'm almost tempted to say that this should be a project in its own right, would be very cool to build a general coverage receiver that goes up to VHF, and it would still be relatively simple. |
One of the things that got me excited about this project is the low component count and the components that you need are quite easy to source aka "a crystal radio for today" |
On GitHub there are Pico libs for si5351 (ex. one by user "kholia"). |
Yes, I know at least one person has reported success recieving 2m and 70cm using external oscillator and mixer. I would definitely like to try this sometime. |
Mister Kholia's Sî5351 routines work fine, but when I tried to integrate them |
Is it just a matter of using extern "C" {#include "xyz"}? |
I am a bit confused about that... |
Someone may help about the matter (compile Sî5351 support into picorx code ) ? |
Sure, happy to help, can you give me some more details? |
Well, like "maurmari" I tried "kholia" code (I work in Win under Visual Studio). |
Nedless to say, I'm quite a newbie about c/c++ ! |
This one? I don't use visual studio, but it should just be a matter of adding the .c file to the CMakelist.txt.
It has already been mentioned that the si5351 library is written in C, whereas PiPicoRx is written in C++. If you call C code from C++ code you need to wrap the function declarations in an extern "C" {} declaration. This tells the compiler/linker that it is calling a C function rather than a C++ function. You can either add this to si5351.h (I did this in with the SSD1306 library, so you could use this as a template. https://github.com/dawsonjon/PicoRX/blob/master/ssd1306.h#L36 ). Alternatively you can add the declaration when you include the file like this:
Hope this helps. Cheers Jon |
Bingo ! Thank you very much for the tips. |
Is there any interest in supporting an external down converter ?
I suspect there's 3 levels of "support" we could do
configurable frequency offset in the UI
IF inversion to support low side and high side mixing
flip LSB and USB if required
Control of an external 1st IF
Si5351 over I2C
A separate NCO output
The text was updated successfully, but these errors were encountered: