The noise LFSR can be reset, which SIDFourier0008.prg attempts to do (but which MiSTer may not implement), see here for implementation https://github.com/daglem/reDIP-SID/commit/ab5f54f02152300a225e5aa6517ef64c269de342
You're right, I tried to pass that flag via .cargo/config.toml, but it failed silently when I passed it incorrectly and I thought it didn't help. Thanks for bringing it back to my attention and helping me get things working.
@dtolnay The issue boils down to cc-rs defaulting to the MSVC /MD runtime library. Integrating cxx into a /MT static library is therefore impossible in the current state. This is configurable through the following cc-rs function, perhaps you could expose a way to allow the /MT library to be used https://docs.rs/cc/latest/cc/struct.Build.html#method.static_crt
As discussed with @sajattack
CHANGELOG.mdfor the BSP or HAL updated
broken by https://github.com/atsamd-rs/atsamd/pull/478 Correct code should be
let usb_gclk = clocks.get_gclk(GEN_A::GCLK2).unwrap(); let usb_clock = &clocks.usb(&usb_gclk).unwrap();
We also have a chatroom where we can discuss if you have more questions https://matrix.to/#/#atsamd-rs:matrix.org
If I were to do it, I would create a
slave module under the existing i2c directory https://github.com/atsamd-rs/atsamd/tree/master/hal/src/sercom/i2c
I think the main reason we don't have it yet is mainly a lack of standardization for the interface in the embedded-hal crate (last I checked). But that's not exactly a good reason why we can't or shouldn't support it.