Skip to content

Conversation

@abc8747
Copy link
Contributor

@abc8747 abc8747 commented Nov 2, 2025

Fixes compile issue:

$ cargo check --features rtl_sdr_blog
error[E0384]: cannot assign twice to immutable variable `div_buf_cur`
   --> src/tuners/r82xx.rs:906:13
    |
886 |                 div_buf_cur = 0x30; /* 11, 150u */
    |                 ------------------ first assignment to `div_buf_cur`
...
906 |             div_buf_cur = 0xa0;
    |             ^^^^^^^^^^^^^^^^^^ cannot assign twice to immutable variable
    |

abc8747 added a commit to xoolive/ship162 that referenced this pull request Nov 2, 2025
- the public-facing `RtlSdrReceiver` and `AsyncRtlSdrReceiver` are now thin
  wrappers around `IqSource` and `AsyncIqSource`.
- improved error handling by propagating I/O errors through the `Iterator`
  and `Stream` as `Result` items
- bumped `rtl-sdr-rs` to 0.2.0 to support R828D tuner
- test: fix false positive in `test_message_serialisation`

Can confirm it works with rtl-sdr blog v4 (r828d) with this patch applied:
ccostes/rtl-sdr-rs#24

```sh
cargo run --release --example rtlsdr
cargo run --release --example rtlsdr_async
```
xoolive pushed a commit to xoolive/ship162 that referenced this pull request Nov 2, 2025
- the public-facing `RtlSdrReceiver` and `AsyncRtlSdrReceiver` are now thin
  wrappers around `IqSource` and `AsyncIqSource`.
- improved error handling by propagating I/O errors through the `Iterator`
  and `Stream` as `Result` items
- bumped `rtl-sdr-rs` to 0.2.0 to support R828D tuner
- test: fix false positive in `test_message_serialisation`

Can confirm it works with rtl-sdr blog v4 (r828d) with this patch applied:
ccostes/rtl-sdr-rs#24

```sh
cargo run --release --example rtlsdr
cargo run --release --example rtlsdr_async
```
@ccostes
Copy link
Owner

ccostes commented Nov 2, 2025

Thanks for the fix!

@ccostes ccostes merged commit a04a97b into ccostes:main Nov 2, 2025
4 checks passed
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.

2 participants