Jan Niehusmann

> As a side note, I was surprised that the API doesn't let you set up a chain before starting the transfer. The current API is awkward if you know...

It's not obvious to me what's going wrong here. `#[inline(always)]` in combination with a ram function looks strange, but I don't think that's the issue here as when it's called...

While I do not yet know what exactly causes your code to fail, some ideas where I'd investigate further: - your `write_flash` function calls several other functions. Why the function...

Uh, even worse, it looks like rustc doesn't inline the sio `read`/`read_blocking` calls: ``` 20000000: b5d0 push {r4, r6, r7, lr} 20000002: af02 add r7, sp, #0x8 20000004: b082 sub...

Glad to hear that it works now, and thanks for posting the working code!

The failing check is caused by cargo-hack no longer being compatible to our MSRV. https://github.com/rp-rs/rp-hal/pull/666 is a possible fix.

This is partly a design decision: Do we want to make it as easy as possible to write some one-off firmware using rp2040-hal? Then we should obviously provide those methods....

Seems to be a good tradeoff. In many cases, the word size will be 8 bits anyway, so calling the new API gets simpler. And for other word sizes, the...

> EDIT: I made this: [ithinuel@90c6c82](https://github.com/ithinuel/rp-hal/commit/90c6c820090c796a3e6551c03acfcc5f10f9ad34) I only managed to get rid of the turbofish by introducing a sorta builder type. > > Let me know your thoughts. I'm not...

Another option could be to make DS a conventional generic parameter, with different types for the available bit sizes. This would have the additional advantage that it would become impossible...