ESP32 PCNT: add support for all counting units #72933
Hallootto1100
started this conversation in
RFC
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
The current driver for the PCNT (pulse counting) can only capture the first counting unit.
Furthermore, the driver uses the sensor api and implements the SENSOR_CHAN_ROTATION without actually converting the unit to degrees.
zephyr/drivers/sensor/espressif/pcnt_esp32/pcnt_esp32.c
Line 111 in a2ebe49
Before I start implementing my own driver or start to improve the one above, am I missing something?
The devicetree binding seem to have multiple counter units in mind.
https://github.com/zephyrproject-rtos/zephyr/blob/2c1da15e3565af76f60fd87407c33d59231f7c0c/dts/bindings/sensor/espressif,esp32-pcnt.yaml
Beta Was this translation helpful? Give feedback.
All reactions