Re: [PATCH 0/8] counter: fix, improvements and stm32 timer events support

[William Breathitt Gray <william.gray@linaro.org>]

[-- Attachment #1: Type: text/plain, Size: 2063 bytes --]

On Tue, Aug 29, 2023 at 03:40:21PM +0200, Fabrice Gasnier wrote:
> This series combines some fix and improvements to the counter interface,
> found while stm32 timer counter driver developements.
> It also introduces a new tool that can be used for testing.
> 
> Then, it improves the stm32 timer counter driver by introducing new signals,
> e.g. counting frequency, and missing channels.
> It also adds support for interrupt based events using the chrdev interface.
> Two event types are added in this series: overflows and capture.
> 
> Up to now, stm32 timer counter driver focused mainly on quadrature
> encoder feature. With this series, all timer instances can be enabled
> for simple counting (with overflow and capture events).
> 
> Fabrice Gasnier (8):
>   counter: chrdev: fix getting array extensions
>   counter: chrdev: remove a typo in header file comment
>   tools/counter: add a flexible watch events tool
>   mfd: stm32-timers: add support for interrupts
>   counter: stm32-timer-cnt: rename quadrature signal
>   counter: stm32-timer-cnt: introduce clock signal
>   counter: stm32-timer-cnt: populate capture channels and check encoder
>   counter: stm32-timer-cnt: add support for events
> 
>  drivers/counter/counter-chrdev.c     |   4 +-
>  drivers/counter/stm32-timer-cnt.c    | 585 ++++++++++++++++++++++++++-
>  drivers/mfd/stm32-timers.c           |  46 +++
>  include/linux/mfd/stm32-timers.h     |  26 ++
>  include/uapi/linux/counter.h         |   2 +-
>  tools/counter/Build                  |   1 +
>  tools/counter/Makefile               |   8 +-
>  tools/counter/counter_watch_events.c | 348 ++++++++++++++++
>  8 files changed, 998 insertions(+), 22 deletions(-)
>  create mode 100644 tools/counter/counter_watch_events.c
> 
> -- 
> 2.25.1
> 

Hi Fabrice,

There are a number of precursor changes in this series that are somewhat
independent, so I'll be taking some patches separately to simplify
things and review this patchset a little at a time.

Thanks,

William Breathitt Gray

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]