myriadrf / LimeSuite

Driver and GUI for LMS7002M-based SDR platforms
https://myriadrf.org/projects/lime-suite/
Apache License 2.0
470 stars 186 forks source link

SISODDR for LimeSDR-Mini #204

Closed gasparka closed 6 years ago

gasparka commented 6 years ago

49d518e706a6a5c64e7ebaace81432ea9580847a adds interpolation/decimation support for SISODDR mode, but only for LimeSDR. Any reasons to not support LimeSDR-Mini?

Code that disables SISODDR for LimeSDR-Mini: https://github.com/myriadrf/LimeSuite/blob/master/src/API/LimeSDR_mini.cpp#L211

IgnasJarusevicius commented 6 years ago

If you judge by SetRate() function, then LimeSDR-USB does not support SISODDR at all as this mode never gets automatically enabled. That commit added correct PLL configuration for situation when interpolation/decimation with SISODDR mode is selected but did not enable automatic selection of such mode anywhere. So if you configure SISODDR and interpolation/decimation from LimeSuiteGUI, it will work correctly with any board.

I agree that now, when interpolation/decimation with SISODDR mode is supported, it makes sense to update SetRate() for LimeSDR-Mini to always select SISODDR mode. So I will update it soon.

gasparka commented 6 years ago

Good to hear. This should reduce the heating problem of the board, so maybe the bandwidth restrictions could be relaxed?

IgnasJarusevicius commented 6 years ago

this was addressed with ddae8b9892013cb2783d19348f823