junkfix / esp32-ds18b20

Arduino Library for ESP32 DS18B20 Non Blocking Onewire RMT protocol, no dependencies, minimal
MIT License
15 stars 4 forks source link

Reference to documentation #12

Closed sirpete83 closed 1 month ago

sirpete83 commented 1 month ago

Hi, Maybe it is important, maybe not but the documentation of rmt module says:

Due to a software limitation in the GPIO driver, when both TX and RX channels are bound to the same GPIO, ensure the RX Channel is initialized before the TX Channel. If the TX Channel was set up first, then during the RX Channel setup, the previous RMT TX Channel signal will be overridden by the GPIO control signal.

https://docs.espressif.com/projects/esp-idf/en/stable/esp32/api-reference/peripherals/rmt.html

You did it the other way any it works. Eventually it is worth to look over this.

Best regards Peter

junkfix commented 1 month ago

This is based on esp idf 4.x RMT driver which now deprecated. it does not have that information. https://docs.espressif.com/projects/esp-idf/en/v4.4.7/esp32/api-reference/peripherals/rmt.html

idf 5.x is a new driver which i may port it later.

sirpete83 commented 1 month ago

Ah. Ok. Understand.. But maybe it is silicon related restriction.

junkfix commented 1 month ago

This is rewritten with new api in the latest update