From: Alexandre Belloni <alexandre.belloni@free-electrons.com>
To: Shawn Guo <shawn.guo@linaro.org>, Jonathan Cameron <jic23@cam.ac.uk>
Cc: jimwall@q.com, brian@crystalfontz.com,
Maxime Ripard <maxime.ripard@free-electrons.com>,
linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-doc@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
devicetree-discuss@lists.ozlabs.org,
Rob Landley <rob@landley.net>,
Rob Herring <rob.herring@calxeda.com>,
Grant Likely <grant.likely@secretlab.ca>,
Alexandre Belloni <alexandre.belloni@free-electrons.com>
Subject: [PATCHv3 0/3] Add support for the Nuvoton NAU7802 ADC to the cfa10049
Date: Mon, 24 Jun 2013 19:24:56 +0200 [thread overview]
Message-ID: <1372094699-3832-1-git-send-email-alexandre.belloni@free-electrons.com> (raw)
Hi,
This is v3 of the patchset that adds support for the 3 Nuvoton NAU7802 ADCs
present on the CFA-10049 board from Crystalfontz. The first patch adds the
driver, the next ones are adding them in the DT after switching the second i2c
bus to bitbanging following a lot of timeout issues we have on that bus.
Changes in v3:
- removed the min_conversion knob in sysfs, it is now hardcoded to 6
- added error checking in probe
- corrected error checking
- corrected possibly uninitialized vldo
- used ARRAY_SIZE(nau7802_chan_array) instead of 2
- added that vldo refers to the internal voltage regulator
Changes in v2:
- channels are now statically allocated
- no wrappers around i2c functions
- sampling_freq is a shared attribute
- setting gain is done through scale
- cosmetics changes (s/idev/indio_dev/, blank lines before returns)
- use of sign_extend32 to extend the sign
- reading scale now returns IIO_VAL_FRACTIONAL_LOG2
- no dt lookup for interrupts, this is done in of_i2c.c but that was mask due
to a bug.
what I didn't address is setting the external reference voltage through a fixed
regulator and I kept the vldo attribute to set the internal voltage reference in
the DT as Jonathan seemed to be ok with that. If anybody is using a nau7802 with
an external reference, it will still be time to add the fixed regulator. It
shouldn't be difficult but I don't have any hardware to test.
Regards,
Alexandre Belloni (1):
iio: Add Nuvoton NAU7802 ADC driver
Maxime Ripard (2):
ARM: mxs: cfa10049: Switch bus i2c1 to bitbanging
ARM: mxs: cfa10049: Add NAU7802 ADCs to the device tree
.../bindings/iio/adc/nuvoton-nau7802.txt | 18 +
arch/arm/boot/dts/imx28-cfa10049.dts | 126 +++--
drivers/iio/adc/Kconfig | 9 +
drivers/iio/adc/Makefile | 1 +
drivers/iio/adc/nau7802.c | 576 +++++++++++++++++++++
5 files changed, 687 insertions(+), 43 deletions(-)
create mode 100644 Documentation/devicetree/bindings/iio/adc/nuvoton-nau7802.txt
create mode 100644 drivers/iio/adc/nau7802.c
--
1.8.1.2
next reply other threads:[~2013-06-24 17:25 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-06-24 17:24 Alexandre Belloni [this message]
2013-06-24 17:24 ` [PATCHv3 1/3] iio: Add Nuvoton NAU7802 ADC driver Alexandre Belloni
2013-07-04 9:08 ` Lars-Peter Clausen
2013-07-06 10:24 ` Jonathan Cameron
2013-07-09 14:04 ` Alexandre Belloni
2013-07-09 17:43 ` Jonathan Cameron
2013-06-24 17:24 ` [PATCHv3 2/3] ARM: mxs: cfa10049: Switch bus i2c1 to bitbanging Alexandre Belloni
2013-07-02 2:45 ` Fabio Estevam
2013-07-02 11:35 ` Alexandre Belloni
2013-07-02 11:45 ` Fabio Estevam
2013-07-02 11:50 ` Alexandre Belloni
2013-07-02 14:06 ` Alexandre Belloni
2013-07-02 16:33 ` Marek Vasut
2013-07-02 17:15 ` Alexandre Belloni
2013-07-06 10:26 ` Jonathan Cameron
2013-07-07 8:47 ` Alexandre Belloni
2013-06-24 17:24 ` [PATCHv3 3/3] ARM: mxs: cfa10049: Add NAU7802 ADCs to the device tree Alexandre Belloni
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=1372094699-3832-1-git-send-email-alexandre.belloni@free-electrons.com \
--to=alexandre.belloni@free-electrons.com \
--cc=brian@crystalfontz.com \
--cc=devicetree-discuss@lists.ozlabs.org \
--cc=grant.likely@secretlab.ca \
--cc=jic23@cam.ac.uk \
--cc=jimwall@q.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-iio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=maxime.ripard@free-electrons.com \
--cc=rob.herring@calxeda.com \
--cc=rob@landley.net \
--cc=shawn.guo@linaro.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
all inboxes | Powered by JetHome®