tib

> I finally got around to rebasing this against the current version. I'll try and find some time to rewrite it to use the embedded-hal implementation soon. > > @tib888...

@TheZoq2 have you tested the read_raw implementation? Repeatedly calling it while waiting for the result will run the first two lines of code many times. I'm afraid that it will...

@TheZoq2 What about this solution below? It is using type states to block starting a new conversion while an other is running. Also in this case you do not need...

> @tib888 That's defenitively a solution that I would prefer over runtime errors. However, as far as I can tell, it is not compatible with the `Adc` trait in `embedded_hal`....

What if into_open_drain_output would return not only an OutputPin, but also an InputPin? This would be more clear than the specialization of the is_low() in OpenDrain mode. The only drawback...

> Thanks. I am not quite sure about "partial layer switch". I see it simply as activating another layer, even if a small one consisting of only 10 keys. So...

I'm looking for this functionality too since I like to use US layout for coding because all brackets and operators are easy to accces and I use English in work...

How is this related to the ZMK "behavior-mod-morph"? ``` //this is used to enter "á" or "Á" depending on the shift's state: hun_aaAA_: hun_aaAA_ { compatible = "zmk,behavior-mod-morph"; #binding-cells =...

Yep, how nice, that Rust has sum-types! :) Instead of RotaryEncoderEvent I would propose: ``` //for an EC11 encoder pub struct RelativeMoveEvent { pub id: u8, pub value: i16, //...

I was assuming that 'id' is the sensor ID. So if he uses 3 identical rotary wheels and 2 identical trackballs, each instance would have different ids. So when the...