Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Sun, 14 Feb 2021 16:43:32 +0900
From:   William Breathitt Gray <vilhelm.gray@gmail.com>
To:     Oleksij Rempel <o.rempel@pengutronix.de>
Cc:     Rob Herring <robh+dt@kernel.org>, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org,
        Pengutronix Kernel Team <kernel@pengutronix.de>,
        David Jander <david@protonic.nl>,
        Robin van der Gracht <robin@protonic.nl>,
        linux-iio@vger.kernel.org,
        Linus Walleij <linus.walleij@linaro.org>,
        Jonathan Cameron <jic23@kernel.org>
Subject: Re: [PATCH v5 0/2] add support for GPIO or IRQ based evemt counter
Message-ID: <YCjUpIXOcZocQdbP@shinobu>
References: <20210208135347.18494-1-o.rempel@pengutronix.de>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="P1Ca1DfqvS/NCoUH"
Content-Disposition: inline
In-Reply-To: <20210208135347.18494-1-o.rempel@pengutronix.de>
Precedence: bulk


--P1Ca1DfqvS/NCoUH
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Mon, Feb 08, 2021 at 02:53:45PM +0100, Oleksij Rempel wrote:
> changes v5:
> - rename it to event counter, since it support different event sources

At the risk of bikeshedding, I feel "event counter" is too general of a
phrase to be useful here -- not to mention that the Counter subsystem
concept of an "event" does not necessarily correlate to unique
interrupts (a Counter driver may generate multiple events for a given
interrupt, or even events for non-interrupt state changes). Instead, if
I understand the behavior of this particular driver correctly, it is
more apt to call this an "interrupt counter" because it is counting
interrupts regardless of the source (GPIO or not).

William Breathitt Gray

> - make it work with gpio-only or irq-only configuration
> - update yaml binding
>=20
> changes v4:
> - use IRQ_NOAUTOEN to not enable IRQ by default
> - rename gpio_ from name pattern and make this driver work any IRQ
>   source.
>=20
> changes v3:
> - convert counter to atomic_t
>=20
> changes v2:
> - add commas
> - avoid possible unhandled interrupts in the enable path
> - do not use of_ specific gpio functions
>=20
> Add support for GPIO based pulse counter. For now it can only count
> pulses. With counter char device support, we will be able to attach
> timestamps and measure actual pulse frequency.
>=20
> Never the less, it is better to mainline this driver now (before chardev
> patches go mainline), to provide developers additional use case for the c=
ounter
> framework with chardev support.
>=20
> Oleksij Rempel (2):
>   dt-bindings: counter: add event-counter binding
>   counter: add IRQ or GPIO based event counter
>=20
>  .../bindings/counter/event-counter.yaml       |  56 ++++
>  drivers/counter/Kconfig                       |  10 +
>  drivers/counter/Makefile                      |   1 +
>  drivers/counter/event-cnt.c                   | 250 ++++++++++++++++++
>  4 files changed, 317 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/counter/event-count=
er.yaml
>  create mode 100644 drivers/counter/event-cnt.c
>=20
> --=20
> 2.30.0
>=20

--P1Ca1DfqvS/NCoUH
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIzBAABCgAdFiEEk5I4PDJ2w1cDf/bghvpINdm7VJIFAmAo1I4ACgkQhvpINdm7
VJJDLw/8Dwiip/OvRVt6b2J3aSHUjS1hmsJS8krQ2XXvWToPlbG3bYbGeO1hKHiZ
Pi7lcqivLAt/cudVuNHcH36Ts9LgmiqoAVJb4ODKQx1F96O+6Kx06+0mFusJJU3J
kIwT+4oB89TLEYWBBwkKr+L4W7MDk3XWPTKoh7yFAVdWlFf+F7WVdMmq1JHpHvBr
rpg3dQM5y58xJgJXiOyaN4fG+LknonSUJZpBkLq1Mq17AOWl8PcM1/kI6vPDhNpl
/Xv/BjYXQLQjIClexa8EG1t2U7YyjUF0/b4eITltm7LQSyXqKCq4I9Z7MUhWZvHA
u2S36lJzLFZvnCo0P+xx2lvmepr4M1CXpcRqIgeOqMS8j0htKSZv99Jn2jAIj/PA
h4fySHQiM4jzV07MM5kKlErQWmRTV4jtY1FwhJf6VGUUh+gGnhLXsaExlMTqlTSi
t1CKxMMWP5Lw0HqQoW53SHamRfM94OoNjEG9GladacznLtxhXcMF8HvWNIthVSSL
O7wjvCN1Nya2PJpFBKhTph4BTwMdKOF0KFSLxRNFMbKvrFaQdUzQQBIR21yaXviS
QKPaftkXqOZHDimpIvkin7u54BPYSBAvolUeS2ob0TlFjkfGSKkZbhpfVe8KZ33t
Yl+y9WAkxPt4oKsMip0ZyliukPfZiZC3OSUFrC/8hNVmr6P4T4Y=
=YStQ
-----END PGP SIGNATURE-----

--P1Ca1DfqvS/NCoUH--