From: Eugen Hristev <eugen.hristev-UWL1GkI3JZL3oGB3hsPCZA@public.gmane.org>
To: nicolas.ferre-UWL1GkI3JZL3oGB3hsPCZA@public.gmane.org,
alexandre.belloni-UWL1GkI3JZL3oGB3hsPCZA@public.gmane.org,
linux-iio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
lars-Qo5EllUWu/uELgA04lAiVw@public.gmane.org,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
ludovic.desroches-UWL1GkI3JZL3oGB3hsPCZA@public.gmane.org,
jic23-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org
Cc: eugen.hristev-UWL1GkI3JZL3oGB3hsPCZA@public.gmane.org
Subject: [PATCH v3 0/4] iio: adc: sama5d2_adc hw triggers and buffers
Date: Thu, 25 May 2017 14:17:10 +0300 [thread overview]
Message-ID: <1495711034-28410-1-git-send-email-eugen.hristev@microchip.com> (raw)
This patch implements the hardware triggers support and buffer management
for sama5d2.
The DT modifications ( [PATCH 1/3] ARM: dts: at91: sama5d2_xplained:
enable ADTRG pin) are for demonstration purposes of the feature,
setting the pinctrl for the ADC hw trigger pin,should go through
at91 maintainers.
As discussed on the mailing list, since we have a single external trigger
namely the ADTRG, but three different possible edges that can trigger it,
will have a single trigger in the IIO subsystem and three possible edges
which can be selected from the device tree.
Thus I created new bindings in the device tree for having the possible
edge types for the ADTRG pin.
The documentation for the bindings is in the patch
[PATCH v3 2/4] Documentation: dt: iio: at91-sama5d2_adc: add adtrg
trigger binding
The modifications in the DT are in the patch
[PATCH v3 3/4] ARM: dts: at91: sama5d2: add ADC hw trigger
Changes in v3:
- No longer have all three possible triggers registered in the subsystem and
available in the sysfs. Only registering one trigger, and the possible edge
is being read from the device tree. In the device tree, any possible edge
can be set. There are three already made nodes for each possible edge which
can be selected.
- Fixed a bug where software triggered conversion could be started even if
the hardware trigger was enabled.
- Preallocate enough space for the buffer, considering all the channels
and timestamp
Changes in v2:
- Moved buffer allocation and freeing into the preenable and postdisable
callbacks.
We have a total of scan bytes that can vary a lot depending on each channel
enabled at a certain point.
- made the at91 trigger list part of state structure
- made the iio trigger list preallocated in state structure
- moved irq enabling/disabling into the try_reenable callback
- on trigger disable must write disable registries as well
- Modified trigger name length to 64
Eugen Hristev (4):
ARM: dts: at91: sama5d2_xplained: enable ADTRG pin
Documentation: dt: iio: at91-sama5d2_adc: add adtrg trigger binding
ARM: dts: at91: sama5d2: add ADC hw trigger
iio: adc: at91-sama5d2_adc: add hw trigger and buffer support
.../bindings/iio/adc/at91-sama5d2_adc.txt | 17 ++
arch/arm/boot/dts/at91-sama5d2_xplained.dts | 16 +-
arch/arm/boot/dts/sama5d2.dtsi | 11 +
drivers/iio/adc/at91-sama5d2_adc.c | 229 ++++++++++++++++++++-
4 files changed, 267 insertions(+), 6 deletions(-)
--
2.7.4
next reply other threads:[~2017-05-25 11:17 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-05-25 11:17 Eugen Hristev [this message]
2017-05-25 11:17 ` [PATCH v3 2/4] Documentation: dt: iio: at91-sama5d2_adc: add adtrg trigger binding Eugen Hristev
2017-05-28 15:26 ` Jonathan Cameron
2017-05-31 17:59 ` Rob Herring
[not found] ` <1495711034-28410-1-git-send-email-eugen.hristev-UWL1GkI3JZL3oGB3hsPCZA@public.gmane.org>
2017-05-25 11:17 ` [PATCH v3 1/4] ARM: dts: at91: sama5d2_xplained: enable ADTRG pin Eugen Hristev
2017-05-25 11:17 ` [PATCH v3 3/4] ARM: dts: at91: sama5d2: add ADC hw trigger Eugen Hristev
2017-05-25 11:17 ` [PATCH v3 4/4] iio: adc: at91-sama5d2_adc: add hw trigger and buffer support Eugen Hristev
[not found] ` <1495711034-28410-5-git-send-email-eugen.hristev-UWL1GkI3JZL3oGB3hsPCZA@public.gmane.org>
2017-05-28 15:23 ` Jonathan Cameron
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=1495711034-28410-1-git-send-email-eugen.hristev@microchip.com \
--to=eugen.hristev-uwl1gki3jzl3ogb3hspcza@public.gmane.org \
--cc=alexandre.belloni-UWL1GkI3JZL3oGB3hsPCZA@public.gmane.org \
--cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=jic23-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
--cc=lars-Qo5EllUWu/uELgA04lAiVw@public.gmane.org \
--cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
--cc=linux-iio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=ludovic.desroches-UWL1GkI3JZL3oGB3hsPCZA@public.gmane.org \
--cc=nicolas.ferre-UWL1GkI3JZL3oGB3hsPCZA@public.gmane.org \
/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).