raiden00pl commented on PR #15079: URL: https://github.com/apache/nuttx/pull/15079#issuecomment-2527330202
I see that PCNT can count pulses and edges. `capture.c` can be used to count edges OR freq OR duty (capture low-level ops all are optional). Another option is to add optional `CAPIOC_PULSES` to `capture.c` instead of providing another driver with similar functionality. The question is wheter `CAPIOC_EDGES` should be added to `capture.c` at all (9e388d1d00dc3aca2ee24fa97b330d240ce1073c). I agree with this PR that capture and pulse count are different functionalities. One measures events, the other time. If we add a separate driver to count pulses, then `CAPIOC_EDGES` should be removed from `capture.c`. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
