From: William Breathitt Gray <vilhelm.gray@gmail.com>
To: Patrick Havelange <patrick.havelange@essensium.com>, jic23@kernel.org
Cc: Mark Rutland <mark.rutland@arm.com>,
devicetree@vger.kernel.org, linux-pwm@vger.kernel.org,
linux-iio@vger.kernel.org, linuxppc-dev@lists.ozlabs.org,
Daniel Lezcano <daniel.lezcano@linaro.org>,
linux-kernel@vger.kernel.org, Li Yang <leoyang.li@nxp.com>,
Rob Herring <robh+dt@kernel.org>,
Thierry Reding <thierry.reding@gmail.com>,
linux-arm-kernel@lists.infradead.org,
Thomas Gleixner <tglx@linutronix.de>,
Shawn Guo <shawnguo@kernel.org>,
Esben Haabendal <esben@haabendal.dk>
Subject: Re: [PATCH v2 0/7] FlexTimer Module Quadrature decoder counter
Date: Thu, 7 Mar 2019 21:03:13 +0900 [thread overview]
Message-ID: <20190307120224.GC7851@icarus> (raw)
In-Reply-To: <20190306111208.7454-1-patrick.havelange@essensium.com>
On Wed, Mar 06, 2019 at 12:12:01PM +0100, Patrick Havelange wrote:
> This patch serie is to be applied on top of
> https://patchwork.kernel.org/project/linux-iio/list/?series=147
> (a more recent version of the serie is available here :
> https://gitlab.com/vilhelmgray/iio/tree/generic_counter_v10 )
>
> Main changes in v2:
> The code is a bit simpler, thanks to more use of devm_* functions.
> The polling/32bit signed version has been dropped, as not needed and
> no other driver is doing that.
>
>
> Patrick Havelange (7):
> include/fsl: add common FlexTimer #defines in a separate header.
> drivers/pwm: pwm-fsl-ftm: use common header for FlexTimer #defines
> drivers/clocksource: timer-fsl-ftm: use common header for FlexTimer
> #defines
> dt-bindings: counter: ftm-quaddec
> counter: add FlexTimer Module Quadrature decoder counter driver
> counter: ftm-quaddec: Documentation: Add specific counter sysfs
> documentation
> LS1021A: dtsi: add ftm quad decoder entries
>
> .../ABI/testing/sysfs-bus-counter-ftm-quaddec | 16 +
> .../bindings/counter/ftm-quaddec.txt | 18 +
> arch/arm/boot/dts/ls1021a.dtsi | 28 ++
> drivers/clocksource/timer-fsl-ftm.c | 15 +-
> drivers/counter/Kconfig | 9 +
> drivers/counter/Makefile | 1 +
> drivers/counter/ftm-quaddec.c | 356 ++++++++++++++++++
> drivers/pwm/pwm-fsl-ftm.c | 44 +--
> include/linux/fsl/ftm.h | 88 +++++
> 9 files changed, 519 insertions(+), 56 deletions(-)
> create mode 100644 Documentation/ABI/testing/sysfs-bus-counter-ftm-quaddec
> create mode 100644 Documentation/devicetree/bindings/counter/ftm-quaddec.txt
> create mode 100644 drivers/counter/ftm-quaddec.c
> create mode 100644 include/linux/fsl/ftm.h
>
> --
> 2.19.1
Patrick,
I see you dropped the polling support in this version. If the need
arises in the future, we can discuss a possible ways of resolving your
latency issues; I imagine interrupts during overflow/underflow events to
be a common behavior among counter devices so those too may result in
latency issues as you discovered in your case.
Jonathan,
If you are satisfied with the changes in this patchset, let me know
which patches you like and I'll add respective Reviewed-by tags for you
for the next Counter subsystem introduction patchset submission.
William Breathitt Gray
prev parent reply other threads:[~2019-03-07 12:05 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-03-06 11:12 [PATCH v2 0/7] FlexTimer Module Quadrature decoder counter Patrick Havelange
2019-03-06 11:12 ` [PATCH v2 1/7] include/fsl: add common FlexTimer #defines in a separate header Patrick Havelange
2019-03-06 11:12 ` [PATCH v2 2/7] drivers/pwm: pwm-fsl-ftm: use common header for FlexTimer #defines Patrick Havelange
2019-03-06 11:12 ` [PATCH v2 3/7] drivers/clocksource: timer-fsl-ftm: " Patrick Havelange
2019-04-11 16:40 ` Daniel Lezcano
2019-03-06 11:12 ` [PATCH v2 4/7] dt-bindings: counter: ftm-quaddec Patrick Havelange
2019-03-12 19:09 ` Rob Herring
2019-03-16 14:21 ` Jonathan Cameron
2019-03-26 15:43 ` Arnout Vandecappelle
2019-03-26 16:06 ` Rob Herring
2019-03-06 11:12 ` [PATCH v2 5/7] counter: add FlexTimer Module Quadrature decoder counter driver Patrick Havelange
2019-03-07 11:32 ` William Breathitt Gray
2019-03-11 11:22 ` Patrick Havelange
2019-03-11 12:24 ` Jonathan Cameron
2019-03-06 11:12 ` [PATCH v2 6/7] counter: ftm-quaddec: Documentation: Add specific counter sysfs documentation Patrick Havelange
2019-03-07 11:42 ` William Breathitt Gray
2019-03-11 12:36 ` Jonathan Cameron
2019-03-06 11:12 ` [PATCH v2 7/7] LS1021A: dtsi: add ftm quad decoder entries Patrick Havelange
2019-03-07 12:03 ` William Breathitt Gray [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190307120224.GC7851@icarus \
--to=vilhelm.gray@gmail.com \
--cc=daniel.lezcano@linaro.org \
--cc=devicetree@vger.kernel.org \
--cc=esben@haabendal.dk \
--cc=jic23@kernel.org \
--cc=leoyang.li@nxp.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-iio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pwm@vger.kernel.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=mark.rutland@arm.com \
--cc=patrick.havelange@essensium.com \
--cc=robh+dt@kernel.org \
--cc=shawnguo@kernel.org \
--cc=tglx@linutronix.de \
--cc=thierry.reding@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).