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

Incorrect usage of the overload bit in the skiq_rx_block_t to detect an "overflow". #3

Open
crroush opened this issue Feb 14, 2022 · 0 comments

Comments

@crroush
Copy link

crroush commented Feb 14, 2022

The sidekiq_core/inc/sidekiq_types.h defines overload as:

volatile uint64_t overload:1;     /**< @brief RF Overload (1 bit) indicating whether or
                                               * not the RF input was overloaded for the received
                                               * sample block */

In Streaming.cpp the overload is being treated as if it means overflow. There is an error message that gets returned when you call skiq_receive that will indicate there is an overrun, which from the logic I believe that is what you are trying to catch to reset things.
skiq_rx_status_error_overrun return check will give you the desired functionality. The way the logic works right now, it will reset buffers prematurely when overload state occurs, which is when the RF input is in excess of 0dBm.

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

No branches or pull requests

1 participant