search

golemparts / rppal

A Rust library that provides access to the Raspberry Pi's GPIO, I2C, PWM, SPI and UART peripherals.
MIT License
1.15k stars 89 forks source link

Open drain pin emulation #39

Open rumatoest opened 4 years ago

rumatoest commented 4 years ago

Hello.

I've spent some time trying link my HAL based driver for DHT sensors with rppal.

I think that rppal missing mostly 2 things right now:

BTW I can create pull request if you are OK with open drain pin emulation

golemparts commented 4 years ago

Thanks for the suggestions! Sorry for the delayed response. digital::v2 traits have been implemented, and will be part of the next release.

Software-based open drain emulation sounds like it could be useful to have. This shouldn't be hard to implement using IoPin. I'll add it to the list for the next release as well. I appreciate the PR offer, but I prefer to give it a go myself first since it's a relatively small addition. If I run into any issues I'll give you a shout for some help. 😄

David-OConnor commented 4 years ago

This seems to be required for compatibility with the popular DS18B20 sensor as well, on the OneWire protocol.

LeCyberDucky commented 11 months ago

Howdy, I'd like to chime in here, as I'm looking for exactly this functionality. I'm not entirely sure whether any progress has been made here, but I'd like to add the following:

I've been looking at using an IoPin to simulate open drain, by switching between an input pin and a low output pin. When switching to output, I would like to set the level to low beforehand, to ensure that I don't start out in an undesirable state. The IoPin documentation, however, mentions that:

Depending on the mode, some methods may not have any effect. For instance, calling a method that alters the pin’s output state won’t cause any changes when the pin’s mode is set to Input.

This sounds like it's not possible to set the level beforehand. It seems like this problem has been looked into already in https://github.com/golemparts/rppal/issues/88, but I believe that does not consider IoPin, meaning that I wouldn't be able to simply switch back and forth between input and low output.

What's the best way forward here? Am I overlooking some functionality that is already there? If not, I'd be happy try implementing the needed functionality, in case that would be welcomed.