* [PATCH v2 3/3] iio:pressure: initial zpa2326 barometer support
2016-09-01 16:25 [PATCH v2 0/3] iio: devm helpers and Murata zpa2326 barometer support Gregor Boirie
2016-09-01 16:25 ` [PATCH v2 1/3] iio:trigger: add resource managed (un)register Gregor Boirie
2016-09-01 16:25 ` [PATCH v2 2/3] iio: add resource managed triggered buffer init helpers Gregor Boirie
@ 2016-09-01 16:25 ` Gregor Boirie
2016-09-01 20:30 ` Linus Walleij
2 siblings, 1 reply; 6+ messages in thread
From: Gregor Boirie @ 2016-09-01 16:25 UTC (permalink / raw)
To: linux-iio
Cc: Jonathan Cameron, Hartmut Knaack, Lars-Peter Clausen,
Peter Meerwald-Stadler, Rob Herring, Mark Rutland,
Jonathan Corbet, Laxman Dewangan, Alexander Kurz, Tejun Heo,
Stephen Boyd, Akinobu Mita, Daniel Baluta, Ludovic Tancerel,
Vlad Dogaru, Linus Walleij, Marek Vasut, Crestez Dan Leonard,
Neil Armstrong, Masahiro Yamada, Arnd Bergmann, Gregor Boirie
Introduce driver for Murata ZPA2326 pressure and temperature sensor:
http://www.murata.com/en-us/products/productdetail?partno=ZPA2326-0311A-R
Signed-off-by: Gregor Boirie <gregor.boirie@parrot.com>
---
.../devicetree/bindings/iio/pressure/zpa2326.txt | 23 +
drivers/iio/pressure/Kconfig | 35 +
drivers/iio/pressure/Makefile | 3 +
drivers/iio/pressure/zpa2326.c | 1736 ++++++++++++++++++++
drivers/iio/pressure/zpa2326.h | 93 ++
drivers/iio/pressure/zpa2326_i2c.c | 97 ++
drivers/iio/pressure/zpa2326_spi.c | 101 ++
7 files changed, 2088 insertions(+)
create mode 100644 Documentation/devicetree/bindings/iio/pressure/zpa2326.txt
create mode 100644 drivers/iio/pressure/zpa2326.c
create mode 100644 drivers/iio/pressure/zpa2326.h
create mode 100644 drivers/iio/pressure/zpa2326_i2c.c
create mode 100644 drivers/iio/pressure/zpa2326_spi.c
diff --git a/Documentation/devicetree/bindings/iio/pressure/zpa2326.txt b/Documentation/devicetree/bindings/iio/pressure/zpa2326.txt
new file mode 100644
index 0000000..4b92906
--- /dev/null
+++ b/Documentation/devicetree/bindings/iio/pressure/zpa2326.txt
@@ -0,0 +1,23 @@
+Murata ZPA2326 pressure sensor
+
+Pressure sensor from Murata with SPI and I2C bus interfaces.
+
+Required properties:
+- compatible: "murata,zpa2326"
+- reg: the I2C address or SPI chip select the device will respond to
+
+Optional properties:
+- interrupt-parent: should be the phandle for the interrupt controller
+- interrupts: interrupt mapping for IRQ
+- vdd-supply: an optional regulator that needs to be on to provide VDD
+ power to the sensor
+
+Example:
+
+zpa2326@5c {
+ compatible = "murata,zpa2326";
+ reg = <0x5c>;
+ interrupt-parent = <&gpio>;
+ interrupts = <12>;
+ vdd-supply = <&ldo_1v8_gnss>;
+};
diff --git a/drivers/iio/pressure/Kconfig b/drivers/iio/pressure/Kconfig
index d130cdc..2134f20 100644
--- a/drivers/iio/pressure/Kconfig
+++ b/drivers/iio/pressure/Kconfig
@@ -187,4 +187,39 @@ config HP206C
This driver can also be built as a module. If so, the module will
be called hp206c.
+config ZPA2326
+ tristate "Murata ZPA2326 pressure sensor driver"
+ depends on I2C || SPI_MASTER
+ select REGMAP
+ select IIO_BUFFER
+ select IIO_TRIGGERED_BUFFER
+ help
+ Say Y here to build support for the Murata ZPA2326 pressure and
+ temperature sensor.
+
+ To compile this driver as a module, choose M here: the module will
+ be called zpa2326.
+
+config ZPA2326_I2C
+ tristate "support I2C bus connection"
+ depends on I2C && ZPA2326
+ select REGMAP_I2C
+ help
+ Say Y here to build I2C bus support for the Murata ZPA2326 pressure
+ and temperature sensor.
+
+ To compile this driver as a module, choose M here: the module will
+ be called zpa2326_i2c.
+
+config ZPA2326_SPI
+ tristate "support SPI bus connection"
+ depends on SPI_MASTER && ZPA2326
+ select REGMAP_SPI
+ help
+ Say Y here to build SPI bus support for the Murata ZPA2326 pressure
+ and temperature sensor.
+
+ To compile this driver as a module, choose M here: the module will
+ be called zpa2326_spi.
+
endmenu
diff --git a/drivers/iio/pressure/Makefile b/drivers/iio/pressure/Makefile
index 7f395be..fff7718 100644
--- a/drivers/iio/pressure/Makefile
+++ b/drivers/iio/pressure/Makefile
@@ -22,6 +22,9 @@ st_pressure-y := st_pressure_core.o
st_pressure-$(CONFIG_IIO_BUFFER) += st_pressure_buffer.o
obj-$(CONFIG_T5403) += t5403.o
obj-$(CONFIG_HP206C) += hp206c.o
+obj-$(CONFIG_ZPA2326) += zpa2326.o
+obj-$(CONFIG_ZPA2326_I2C) += zpa2326_i2c.o
+obj-$(CONFIG_ZPA2326_SPI) += zpa2326_spi.o
obj-$(CONFIG_IIO_ST_PRESS_I2C) += st_pressure_i2c.o
obj-$(CONFIG_IIO_ST_PRESS_SPI) += st_pressure_spi.o
diff --git a/drivers/iio/pressure/zpa2326.c b/drivers/iio/pressure/zpa2326.c
new file mode 100644
index 0000000..dd78866
--- /dev/null
+++ b/drivers/iio/pressure/zpa2326.c
@@ -0,0 +1,1736 @@
+/*
+ * Murata ZPA2326 pressure and temperature sensor IIO driver
+ *
+ * Copyright (c) 2016 Parrot S.A.
+ *
+ * Author: Gregor Boirie <gregor.boirie@parrot.com>
+ *
+ * This program is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License version 2 as published by
+ * the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
+ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for
+ * more details.
+ *
+ *
+ * README:
+ * ------
+ *
+ * This driver supports the following IIO modes: INDIO_DIRECT_MODE,
+ * INDIO_BUFFER_SOFTWARE and INDIO_BUFFER_TRIGGERED.
+ * A hardware trigger is also implemented to dispatch registered IIO trigger
+ * consumers upon "sample ready" interrupts.
+ *
+ * ZPA2326 hardware supports 2 sampling mode: one shot and continuous.
+ *
+ * A complete one shot sampling cycle gets device out of low power mode,
+ * performs pressure and temperature measurements, then automatically switches
+ * back to low power mode. It is meant for on demand sampling with optimal power
+ * saving at the cost of lower sampling rate and higher software overhead.
+ * This is a natural candidate for IIO read_raw hook implementation
+ * (INDIO_DIRECT_MODE). It is also used for triggered buffering support to
+ * ensure explicit synchronization with external trigger events
+ * (INDIO_BUFFER_TRIGGERED).
+ *
+ * The continuous mode works according to a periodic hardware measurement
+ * process continuously pushing samples into an internal hardware fifo (for
+ * pressure samples only). Measurement cycle completion may be signaled by a
+ * "sample ready" interrupt.
+ * Typical software sequence of operations :
+ * - get device out of low power mode,
+ * - setup hardware sampling period,
+ * - at end of period, upon data ready interrupt: pop pressure samples out of
+ * hardware fifo and fetch temperature sample
+ * - when no longer needed, stop sampling process by putting device into
+ * low power mode.
+ * This mode is used to implement INDIO_BUFFER_SOFTWARE mode if device tree
+ * declares a valid interrupt line. It is not compatible with trigger driven
+ * acquisition.
+ *
+ * Note that hardware sampling frequency is taken into account for
+ * INDIO_BUFFER_SOFTWARE mode only as the highest sampling rate seems to be the
+ * most energy efficient.
+ *
+ * TODO:
+ * - preset pressure threshold crossing / IIO events
+ * - differential pressure sampling
+ * - hardware samples averaging
+ */
+
+#include <linux/module.h>
+#include <linux/kernel.h>
+#include <linux/delay.h>
+#include <linux/interrupt.h>
+#include <linux/regulator/consumer.h>
+#include <linux/pm_runtime.h>
+#include <linux/regmap.h>
+#include <linux/iio/iio.h>
+#include <linux/iio/sysfs.h>
+#include <linux/iio/buffer.h>
+#include <linux/iio/trigger.h>
+#include <linux/iio/trigger_consumer.h>
+#include <linux/iio/triggered_buffer.h>
+#include "zpa2326.h"
+
+/* 200 ms should be enough for the longest conversion time in one-shot mode. */
+#define ZPA2326_CONVERSION_JIFFIES (HZ / 5)
+/* There should be a 1 ms delay (Tpup) after getting out of reset. */
+#define ZPA2326_TPUP_USEC_MIN (1000)
+#define ZPA2326_TPUP_USEC_MAX (2000)
+
+/* Hardware sampling frequency descriptor. */
+struct zpa2326_frequency {
+ /* Stringified frequency in Hertz. */
+ const char *hz;
+ /* Output Data Rate word as expected by ZPA2326_CTRL_REG3_REG. */
+ u16 odr;
+};
+
+/*
+ * Keep these in strict ascending order: last array entry is expected to
+ * correspond to the highest sampling frequency.
+ */
+static const struct zpa2326_frequency zpa2326_sampling_frequencies[] = {
+ { .hz = "1", .odr = 1 << ZPA2326_CTRL_REG3_ODR_SHIFT },
+ { .hz = "5", .odr = 5 << ZPA2326_CTRL_REG3_ODR_SHIFT },
+ { .hz = "11", .odr = 6 << ZPA2326_CTRL_REG3_ODR_SHIFT },
+ { .hz = "23", .odr = 7 << ZPA2326_CTRL_REG3_ODR_SHIFT },
+};
+
+/* Return the highest hardware sampling frequency available. */
+static const struct zpa2326_frequency *zpa2326_highest_frequency(void)
+{
+ return &zpa2326_sampling_frequencies[
+ ARRAY_SIZE(zpa2326_sampling_frequencies) - 1];
+}
+
+/* Per-device internal private state. */
+struct zpa2326_private {
+ /* Buffered samples ready datum. */
+ s64 timestamp;
+ /*
+ * Underlying I2C / SPI bus adapter used to abstract slave register
+ * accesses.
+ */
+ struct regmap *regmap;
+ /*
+ * Allows sampling logic to get completion status of operations
+ * interrupt handlers perform asynchronously.
+ */
+ int result;
+ /*
+ * Interrupt handler uses this to wake user context up at sampling
+ * operation completion.
+ */
+ struct completion data_ready;
+ /*
+ * Optional hardware / interrupt driven trigger used to notify external
+ * devices a new sample is ready.
+ */
+ struct iio_trigger *trigger;
+ /* Flag indicating whether or not device has just been powered on. */
+ bool waken;
+ /*
+ * Optional interrupt line: negative if not declared into DT, in
+ * which case sampling logic keeps polling status register to detect
+ * completion.
+ */
+ int irq;
+ /* Current hardware sampling frequency. */
+ const struct zpa2326_frequency *frequency;
+ /* Optional power / voltage reference */
+ struct regulator *vref;
+ /* Optional power supply. */
+ struct regulator *vdd;
+};
+
+#define zpa2326_err(_idev, _format, _arg...) \
+ dev_err(zpa2326_iio2slave(_idev), _format, ##_arg)
+
+#define zpa2326_warn(_idev, _format, _arg...) \
+ dev_warn(zpa2326_iio2slave(_idev), _format, ##_arg)
+
+#define zpa2326_dbg(_idev, _format, _arg...) \
+ dev_dbg(zpa2326_iio2slave(_idev), _format, ##_arg)
+
+static struct zpa2326_private *zpa2326_iio2priv(const struct iio_dev *indio_dev)
+{
+ return (struct zpa2326_private *)iio_priv(indio_dev);
+}
+
+/*
+ * Fetch single byte from slave register.
+ * indio_dev: IIO device the slave is attached to.
+ * address: address of register to read from.
+ *
+ * Returns the fetched byte when successful, a negative error code otherwise.
+ */
+static int zpa2326_read_byte(const struct iio_dev *indio_dev, u8 address)
+{
+ unsigned int val;
+ int err;
+
+ err = regmap_read(zpa2326_iio2priv(indio_dev)->regmap, address, &val);
+ if (err)
+ return err;
+
+ return val;
+}
+
+/*
+ * Store single byte to slave register.
+ * indio_dev: IIO device the slave is attached to
+ * address: address of register to write to
+ *
+ * Returns: zero when successful, a negative error code otherwise.
+ */
+static int zpa2326_write_byte(const struct iio_dev *indio_dev,
+ u8 address,
+ u8 value)
+{
+ return regmap_write(zpa2326_iio2priv(indio_dev)->regmap, address,
+ value);
+}
+
+/*
+ * Fetch multiple bytes from a block of contiguous slave registers.
+ * indio_dev: IIO device the slave is attached to.
+ * address: start address of contiguous registers block to read from.
+ * length: number of bytes to fetch.
+ * value: location to store fetched data into.
+ *
+ * Returns: zero when successful, a negative error code otherwise.
+ */
+static int zpa2326_read_block(const struct iio_dev *indio_dev,
+ u8 address,
+ u8 length,
+ u8 *value)
+{
+ return regmap_bulk_read(zpa2326_iio2priv(indio_dev)->regmap, address,
+ value, length);
+}
+
+bool zpa2326_isreg_writeable(struct device *dev, unsigned int reg)
+{
+ switch (reg) {
+ case ZPA2326_REF_P_XL_REG:
+ case ZPA2326_REF_P_L_REG:
+ case ZPA2326_REF_P_H_REG:
+ case ZPA2326_RES_CONF_REG:
+ case ZPA2326_CTRL_REG0_REG:
+ case ZPA2326_CTRL_REG1_REG:
+ case ZPA2326_CTRL_REG2_REG:
+ case ZPA2326_CTRL_REG3_REG:
+ case ZPA2326_THS_P_LOW_REG:
+ case ZPA2326_THS_P_HIGH_REG:
+ return true;
+
+ default:
+ return false;
+ }
+}
+EXPORT_SYMBOL_GPL(zpa2326_isreg_writeable);
+
+bool zpa2326_isreg_readable(struct device *dev, unsigned int reg)
+{
+ switch (reg) {
+ case ZPA2326_REF_P_XL_REG:
+ case ZPA2326_REF_P_L_REG:
+ case ZPA2326_REF_P_H_REG:
+ case ZPA2326_DEVICE_ID_REG:
+ case ZPA2326_RES_CONF_REG:
+ case ZPA2326_CTRL_REG0_REG:
+ case ZPA2326_CTRL_REG1_REG:
+ case ZPA2326_CTRL_REG2_REG:
+ case ZPA2326_CTRL_REG3_REG:
+ case ZPA2326_INT_SOURCE_REG:
+ case ZPA2326_THS_P_LOW_REG:
+ case ZPA2326_THS_P_HIGH_REG:
+ case ZPA2326_STATUS_REG:
+ case ZPA2326_PRESS_OUT_XL_REG:
+ case ZPA2326_PRESS_OUT_L_REG:
+ case ZPA2326_PRESS_OUT_H_REG:
+ case ZPA2326_TEMP_OUT_L_REG:
+ case ZPA2326_TEMP_OUT_H_REG:
+ return true;
+
+ default:
+ return false;
+ }
+}
+EXPORT_SYMBOL_GPL(zpa2326_isreg_readable);
+
+bool zpa2326_isreg_precious(struct device *dev, unsigned int reg)
+{
+ switch (reg) {
+ case ZPA2326_INT_SOURCE_REG:
+ case ZPA2326_PRESS_OUT_H_REG:
+ return true;
+
+ default:
+ return false;
+ }
+}
+EXPORT_SYMBOL_GPL(zpa2326_isreg_precious);
+
+/*
+ * Enable device, i.e. get out of low power mode.
+ *
+ * Required to access complete register space and to perform any sampling
+ * or control operations.
+ */
+static int zpa2326_enable_device(const struct iio_dev *indio_dev)
+{
+ int err = zpa2326_write_byte(indio_dev, ZPA2326_CTRL_REG0_REG,
+ ZPA2326_CTRL_REG0_ENABLE);
+ if (err) {
+ zpa2326_err(indio_dev, "failed to enable device (%d)", err);
+ return err;
+ }
+
+ zpa2326_dbg(indio_dev, "enabled");
+ return 0;
+}
+
+/*
+ * Disable device, i.e. switch to low power mode.
+ *
+ * Only ZPA2326_DEVICE_ID_REG and ZPA2326_CTRL_REG0_REG registers may be
+ * accessed once device is in the disabled state.
+ */
+static int zpa2326_disable_device(const struct iio_dev *indio_dev)
+{
+ int err = zpa2326_write_byte(indio_dev, ZPA2326_CTRL_REG0_REG, 0);
+
+ if (err) {
+ zpa2326_err(indio_dev, "failed to disable device (%d)", err);
+ return err;
+ }
+
+ zpa2326_dbg(indio_dev, "disabled");
+ return 0;
+}
+
+/*
+ * Reset device to default hardware state.
+ *
+ * Disable sampling and empty hardware fifo.
+ * Device must be enabled before reset, i.e. not in low power mode.
+ */
+static int zpa2326_reset_device(const struct iio_dev *indio_dev)
+{
+ int err = zpa2326_write_byte(indio_dev, ZPA2326_CTRL_REG2_REG,
+ ZPA2326_CTRL_REG2_SWRESET);
+ if (err) {
+ zpa2326_err(indio_dev, "failed to reset device (%d)", err);
+ return err;
+ }
+
+ usleep_range(ZPA2326_TPUP_USEC_MIN, ZPA2326_TPUP_USEC_MAX);
+
+ zpa2326_dbg(indio_dev, "reset");
+ return 0;
+}
+
+/*
+ * Start a single sampling cycle, i.e. in one shot mode.
+ *
+ * Device must have been previously enabled and configured for one shot mode.
+ * Device will automatically switch back to low power mode at end of cycle.
+ */
+static int zpa2326_start_oneshot(const struct iio_dev *indio_dev)
+{
+ int err = zpa2326_write_byte(indio_dev, ZPA2326_CTRL_REG0_REG,
+ ZPA2326_CTRL_REG0_ENABLE |
+ ZPA2326_CTRL_REG0_ONE_SHOT);
+ if (err) {
+ zpa2326_err(indio_dev, "failed to start one shot cycle (%d)",
+ err);
+ return err;
+ }
+
+ zpa2326_dbg(indio_dev, "one shot cycle started");
+ return 0;
+}
+
+/*
+ * Power on device to allow subsequent configuration.
+ *
+ * Sampling will be disabled, preventing strange things from happening in our
+ * back. Hardware fifo content will be cleared.
+ * When successful, device will be left in the enabled state to allow further
+ * configuration.
+ */
+static int zpa2326_power_on(const struct iio_dev *indio_dev,
+ const struct zpa2326_private *private)
+{
+ int err = regulator_enable(private->vref);
+
+ if (err)
+ return err;
+
+ err = regulator_enable(private->vdd);
+ if (err)
+ goto vref;
+
+ zpa2326_dbg(indio_dev, "powered on");
+
+ err = zpa2326_enable_device(indio_dev);
+ if (err)
+ goto vdd;
+
+ err = zpa2326_reset_device(indio_dev);
+ if (err)
+ goto disable;
+
+ return 0;
+
+disable:
+ zpa2326_disable_device(indio_dev);
+vdd:
+ regulator_disable(private->vdd);
+vref:
+ regulator_disable(private->vref);
+
+ zpa2326_dbg(indio_dev, "powered off");
+ return err;
+}
+
+/* Power off device, i.e. disable attached power regulators. */
+static void zpa2326_power_off(const struct iio_dev *indio_dev,
+ const struct zpa2326_private *private)
+{
+ regulator_disable(private->vdd);
+ regulator_disable(private->vref);
+
+ zpa2326_dbg(indio_dev, "powered off");
+}
+
+/*
+ * Setup device for one shot / on demand mode.
+ *
+ * Output Data Rate is configured for the highest possible rate so that
+ * conversion time and power consumption are reduced to a minimum.
+ * Note that hardware internal averaging machinery (not implemented in this
+ * driver) is not applicable in this mode.
+ *
+ * Device must have been previously enabled before calling
+ * zpa2326_config_oneshot().
+ */
+static int zpa2326_config_oneshot(const struct iio_dev *indio_dev,
+ int irq)
+{
+ const struct zpa2326_frequency *freq = zpa2326_highest_frequency();
+ int err;
+
+ /* Setup highest available Output Data Rate for one shot mode. */
+ err = zpa2326_write_byte(indio_dev, ZPA2326_CTRL_REG3_REG, freq->odr);
+ if (err)
+ return err;
+
+ if (irq > 0)
+ /* Request interrupt when new sample is available. */
+ err = zpa2326_write_byte(indio_dev, ZPA2326_CTRL_REG1_REG,
+ (u8)~ZPA2326_CTRL_REG1_MASK_DATA_RDY);
+ if (err) {
+ dev_err(zpa2326_iio2slave(indio_dev),
+ "failed to setup one shot mode (%d)", err);
+ return err;
+ }
+
+ dev_dbg(zpa2326_iio2slave(indio_dev), "one shot mode setup @%sHz",
+ freq->hz);
+ return 0;
+}
+
+/*
+ * Clear remaining entries in hardware fifo.
+ *
+ * min_count argument is a hint corresponding to the known minimum number of
+ * samples currently living in the fifo. This allows to reduce the number of bus
+ * accesses by skipping status register read operation as long as we know for
+ * sure there are still entries left.
+ */
+static int zpa2326_clear_fifo(const struct iio_dev *indio_dev,
+ unsigned int min_count)
+{
+ int err;
+
+ if (!min_count) {
+ /*
+ * No hint: read status register to determine whether fifo is
+ * empty or not.
+ */
+ err = zpa2326_read_byte(indio_dev, ZPA2326_STATUS_REG);
+ if (err < 0)
+ goto err;
+
+ if (err & ZPA2326_STATUS_FIFO_E)
+ /* Fifo is empty: nothing to trash. */
+ return 0;
+ }
+
+ /* Clear fifo. */
+ do {
+ /*
+ * A single fetch from pressure MSB register is enough to pop
+ * values out of fifo.
+ */
+ err = zpa2326_read_byte(indio_dev, ZPA2326_PRESS_OUT_H_REG);
+ if (err < 0)
+ goto err;
+
+ if (min_count) {
+ /*
+ * We know for sure there are at least min_count entries
+ * left in fifo. Skip status register read.
+ */
+ min_count--;
+ continue;
+ }
+
+ err = zpa2326_read_byte(indio_dev, ZPA2326_STATUS_REG);
+ if (err < 0)
+ goto err;
+
+ } while (!(err & ZPA2326_STATUS_FIFO_E));
+
+ zpa2326_dbg(indio_dev, "fifo cleared");
+ return 0;
+
+err:
+ zpa2326_err(indio_dev, "failed to clear fifo (%d)", err);
+ return err;
+}
+
+/*
+ * Retrieve the most recent pressure sample from hardware fifo.
+ *
+ * Note that ZPA2326 hardware fifo stores pressure samples only.
+ */
+static int zpa2326_dequeue_pressure(const struct iio_dev *indio_dev,
+ u32 *pressure)
+{
+ int err = zpa2326_read_byte(indio_dev, ZPA2326_STATUS_REG);
+ int cleared = -1;
+
+ if (err < 0)
+ return err;
+
+ *pressure = 0;
+
+ if (err & ZPA2326_STATUS_P_OR) {
+ /*
+ * Fifo overrun : first sample dequeued from fifo is the
+ * newest.
+ */
+ zpa2326_warn(indio_dev, "fifo overflow");
+
+ err = zpa2326_read_block(indio_dev, ZPA2326_PRESS_OUT_XL_REG, 3,
+ (u8 *)pressure);
+ if (err)
+ return err;
+
+#define ZPA2326_FIFO_DEPTH (16U)
+ /* Hardware fifo may hold no more than 16 pressure samples. */
+ return zpa2326_clear_fifo(indio_dev, ZPA2326_FIFO_DEPTH - 1);
+ }
+
+ /*
+ * Fifo has not overflown : retrieve newest sample. We need to pop
+ * values out until fifo is empty : last fetched pressure is the newest.
+ * In nominal cases, we should find a single queued sample only.
+ */
+ do {
+ err = zpa2326_read_block(indio_dev, ZPA2326_PRESS_OUT_XL_REG, 3,
+ (u8 *)pressure);
+ if (err)
+ return err;
+
+ err = zpa2326_read_byte(indio_dev, ZPA2326_STATUS_REG);
+ if (err < 0)
+ return err;
+
+ cleared++;
+ } while (!(err & ZPA2326_STATUS_FIFO_E));
+
+ if (cleared)
+ /*
+ * Samples were pushed by hardware during previous rounds but we
+ * didn't consume them fast enough: inform user.
+ */
+ zpa2326_warn(indio_dev, "cleared %d fifo entries", cleared);
+
+ return 0;
+}
+
+/* Enqueue new channel samples to IIO buffer. */
+static int zpa2326_fill_sample_buffer(struct iio_dev *indio_dev,
+ const struct zpa2326_private *private)
+{
+ struct {
+ u32 pressure;
+ u16 temperature;
+ u64 timestamp;
+ } sample;
+ int err;
+
+ if (test_bit(0, indio_dev->active_scan_mask)) {
+ /* Get current pressure from hardware fifo. */
+ err = zpa2326_dequeue_pressure(indio_dev, &sample.pressure);
+ if (err) {
+ zpa2326_warn(indio_dev, "failed to fetch pressure (%d)",
+ err);
+ return err;
+ }
+ }
+
+ if (test_bit(1, indio_dev->active_scan_mask)) {
+ /* Get current temperature. */
+ err = zpa2326_read_block(indio_dev, ZPA2326_TEMP_OUT_L_REG, 2,
+ (u8 *)&sample.temperature);
+ if (err) {
+ zpa2326_warn(indio_dev,
+ "failed to fetch temperature (%d)", err);
+ return err;
+ }
+ }
+
+ /*
+ * Now push samples using timestamp stored either :
+ * - by hardware interrupt handler if interrupt is available: see
+ * zpa2326_handle_irq(),
+ * - or oneshot completion polling machinery : see
+ * zpa2326_trigger_oneshot().
+ */
+ iio_push_to_buffers_with_timestamp(indio_dev, &sample,
+ private->timestamp);
+ return 0;
+}
+
+#ifdef CONFIG_PM
+static int zpa2326_runtime_suspend(struct device *slave)
+{
+ const struct iio_dev *indio_dev = dev_get_drvdata(slave);
+
+ if (pm_runtime_autosuspend_expiration(slave))
+ /* Userspace changed autosuspend delay. */
+ return -EAGAIN;
+
+ zpa2326_power_off(indio_dev, zpa2326_iio2priv(indio_dev));
+
+ return 0;
+}
+
+static int zpa2326_runtime_resume(struct device *slave)
+{
+ const struct iio_dev *indio_dev = dev_get_drvdata(slave);
+
+ return zpa2326_power_on(indio_dev, zpa2326_iio2priv(indio_dev));
+}
+
+const struct dev_pm_ops zpa2326_pm_ops = {
+ SET_SYSTEM_SLEEP_PM_OPS(pm_runtime_force_suspend,
+ pm_runtime_force_resume)
+ SET_RUNTIME_PM_OPS(zpa2326_runtime_suspend, zpa2326_runtime_resume,
+ NULL)
+};
+EXPORT_SYMBOL_GPL(zpa2326_pm_ops);
+
+/* Request the PM layer to power supply the device. */
+static int zpa2326_resume(const struct iio_dev *indio_dev)
+{
+ struct device *slave = zpa2326_iio2slave(indio_dev);
+ int err = pm_runtime_get_sync(slave);
+
+ if (err < 0)
+ return err;
+
+ if (err > 0) {
+ /*
+ * Device was already power supplied: get it out of low power
+ * mode and inform caller.
+ */
+ zpa2326_enable_device(indio_dev);
+ return 1;
+ }
+
+ /* Inform caller device has just been brought back to life. */
+ return 0;
+}
+
+/*
+ * Schedule a power down using autosuspend feature of PM layer.
+ *
+ * Device is switched to low power mode at first to save power even when
+ * attached regulator is a "dummy" one.
+ */
+static void zpa2326_suspend(struct iio_dev *indio_dev)
+{
+ struct device *slave = zpa2326_iio2slave(indio_dev);
+
+ zpa2326_disable_device(indio_dev);
+
+ pm_runtime_mark_last_busy(slave);
+ pm_runtime_put_autosuspend(slave);
+}
+
+/* Setup runtime power management with autosuspend support. */
+static void zpa2326_init_runtime(struct device *slave)
+{
+ pm_runtime_get_noresume(slave);
+ pm_runtime_set_active(slave);
+ pm_runtime_enable(slave);
+ pm_runtime_set_autosuspend_delay(slave, 1000);
+ pm_runtime_use_autosuspend(slave);
+ pm_runtime_mark_last_busy(slave);
+ pm_runtime_put_autosuspend(slave);
+}
+
+static void zpa2326_fini_runtime(struct device *slave)
+{
+ pm_runtime_disable(slave);
+ pm_runtime_set_suspended(slave);
+}
+#else /* !CONFIG_PM */
+static int zpa2326_resume(const struct iio_dev *indio_dev)
+{
+ zpa2326_enable_device(indio_dev);
+ return 0;
+}
+
+static void zpa2326_suspend(struct iio_dev *indio_dev)
+{
+ zpa2326_disable_device(indio_dev);
+}
+
+#define zpa2326_init_runtime(_slave)
+#define zpa2326_fini_runtime(_slave)
+#endif /* !CONFIG_PM */
+
+/*
+ * Hardware interrupt handler.
+ *
+ * Timestamp buffered samples as soon as possible then schedule threaded bottom
+ * half.
+ */
+static irqreturn_t zpa2326_handle_irq(int irq, void *data)
+{
+ struct iio_dev *indio_dev = (struct iio_dev *)data;
+
+ if (iio_buffer_enabled(indio_dev)) {
+ struct zpa2326_private *priv = zpa2326_iio2priv(indio_dev);
+
+ /* Timestamping needed for buffered sampling only. */
+ priv->timestamp = iio_get_time_ns(indio_dev);
+ }
+
+ return IRQ_WAKE_THREAD;
+}
+
+/*
+ * Interrupt bottom-half handler.
+ *
+ * Mainly ensures interrupt is caused by a real "new sample available"
+ * condition. This relies upon the ability to perform blocking / sleeping bus
+ * accesses to slave's registers. This is why zpa2326_handle_threaded_irq() is
+ * called from within a thread, i.e. not called from hard interrupt context.
+ */
+static irqreturn_t zpa2326_handle_threaded_irq(int irq, void *data)
+{
+ struct iio_dev *indio_dev = (struct iio_dev *)data;
+ struct zpa2326_private *priv = zpa2326_iio2priv(indio_dev);
+ irqreturn_t ret = IRQ_NONE;
+
+ /*
+ * Device works according to a level interrupt scheme: reading interrupt
+ * status de-asserts interrupt line.
+ */
+ priv->result = zpa2326_read_byte(indio_dev, ZPA2326_INT_SOURCE_REG);
+ if (priv->result < 0) {
+ if (indio_dev->currentmode == INDIO_BUFFER_SOFTWARE)
+ return IRQ_NONE;
+
+ goto complete;
+ }
+
+ /* Data ready is the only interrupt source we requested. */
+ if (!(priv->result & ZPA2326_INT_SOURCE_DATA_RDY)) {
+ /*
+ * Interrupt happened but no new sample available: likely caused
+ * by spurious interrupts, in which case, returning IRQ_NONE
+ * allows to benefit from the generic spurious interrupts
+ * handling.
+ */
+ zpa2326_warn(indio_dev, "unexpected interrupt status %02x",
+ priv->result);
+
+ if (indio_dev->currentmode == INDIO_BUFFER_SOFTWARE)
+ return IRQ_NONE;
+
+ priv->result = -ENODATA;
+ goto complete;
+ }
+
+ /* New sample available: dispatch internal trigger consumers. */
+ iio_trigger_poll_chained(priv->trigger);
+
+ if (indio_dev->currentmode == INDIO_BUFFER_SOFTWARE) {
+ /* Continuous sampling enabled: populate IIO buffer. */
+ zpa2326_fill_sample_buffer(indio_dev, priv);
+ return IRQ_HANDLED;
+ }
+
+ if (indio_dev->currentmode == INDIO_BUFFER_TRIGGERED)
+ priv->result = zpa2326_fill_sample_buffer(indio_dev, priv);
+ else
+ priv->result = 0;
+
+ ret = IRQ_HANDLED;
+
+complete:
+ /*
+ * Wake up direct or triggered buffer mode waiters: see
+ * zpa2326_sample_oneshot() and zpa2326_trigger_oneshot().
+ */
+ complete(&priv->data_ready);
+ return ret;
+}
+
+/* Wait for oneshot data ready interrupt. */
+static int zpa2326_wait_oneshot_completion(const struct iio_dev *indio_dev,
+ struct zpa2326_private *private)
+{
+ int err = wait_for_completion_interruptible_timeout(
+ &private->data_ready, ZPA2326_CONVERSION_JIFFIES);
+ if (err > 0)
+ /*
+ * Interrupt handler completed before timeout: return operation
+ * status.
+ */
+ return private->result;
+
+ /* Clear all interrupts just to be sure. */
+ zpa2326_read_byte(indio_dev, ZPA2326_INT_SOURCE_REG);
+
+ if (!err)
+ /* Timed out. */
+ err = -ETIME;
+
+ if (err != -ERESTARTSYS)
+ zpa2326_warn(indio_dev, "no one shot interrupt occurred (%d)",
+ err);
+
+ return err;
+}
+
+/* Setup interrupt handling. */
+static int zpa2326_init_irq(struct device *slave,
+ struct iio_dev *indio_dev,
+ struct zpa2326_private *private,
+ int irq)
+{
+ int err;
+
+ if (irq <= 0) {
+ /*
+ * Platform declared no interrupt line: device will be polled
+ * for data availability.
+ */
+ private->irq = -1;
+ dev_info(slave, "no interrupt found, running in polling mode");
+ return 0;
+ }
+
+ private->irq = irq;
+ init_completion(&private->data_ready);
+
+ /* Request handler to be scheduled into threaded interrupt context. */
+ err = devm_request_threaded_irq(slave, irq, zpa2326_handle_irq,
+ zpa2326_handle_threaded_irq,
+ IRQF_TRIGGER_RISING | IRQF_ONESHOT,
+ dev_name(slave), indio_dev);
+ if (err) {
+ dev_err(slave, "failed to request interrupt %d (%d)", irq, err);
+ return err;
+ }
+
+ dev_info(slave, "using interrupt %d", irq);
+ return 0;
+}
+
+/*
+ * Ensure exclusive access to prevent from modifying device setup in the back of
+ * ongoing operations.
+ * In the mean time, power on device to get access to the complete register map.
+ */
+static int zpa2326_claim_direct_mode(struct iio_dev *indio_dev)
+{
+ /* Gain exclusive access for userspace usage. */
+ int err = iio_device_claim_direct_mode(indio_dev);
+
+ if (err)
+ return err;
+
+ /* Bring device back to life. */
+ err = zpa2326_resume(indio_dev);
+ if (err < 0)
+ goto release;
+
+ return err;
+
+release:
+ /* Relinquish exclusive access. */
+ iio_device_release_direct_mode(indio_dev);
+ return err;
+}
+
+/* Release exclusive acces to device. Device is also powered down. */
+static void zpa2326_release_direct_mode(struct iio_dev *indio_dev)
+{
+ zpa2326_suspend(indio_dev);
+ iio_device_release_direct_mode(indio_dev);
+}
+
+/* Actively poll for one shot data ready. */
+static int zpa2326_poll_oneshot_completion(const struct iio_dev *indio_dev)
+{
+ unsigned long tmout = jiffies + ZPA2326_CONVERSION_JIFFIES;
+ int err;
+
+ /*
+ * At least, 100 ms is needed for the device to complete its one-shot
+ * cycle.
+ */
+ if (msleep_interruptible(100))
+ return -ERESTARTSYS;
+
+ /* Poll for conversion completion in hardware. */
+ while (true) {
+ err = zpa2326_read_byte(indio_dev, ZPA2326_CTRL_REG0_REG);
+ if (err < 0)
+ goto err;
+
+ if (!(err & ZPA2326_CTRL_REG0_ONE_SHOT))
+ /* One-shot bit self clears at conversion end. */
+ break;
+
+ if (time_after(jiffies, tmout)) {
+ /* Prevent from waiting forever : let's time out. */
+ err = -ETIME;
+ goto err;
+ }
+
+ usleep_range(10000, 20000);
+ }
+
+ /*
+ * In oneshot mode, pressure sample availability guarantees that
+ * temperature conversion has also completed : just check pressure
+ * status bit to keep things simple.
+ */
+ err = zpa2326_read_byte(indio_dev, ZPA2326_STATUS_REG);
+ if (err < 0)
+ goto err;
+
+ if (!(err & ZPA2326_STATUS_P_DA)) {
+ /* No sample available. */
+ err = -ENODATA;
+ goto err;
+ }
+
+ return 0;
+
+err:
+ zpa2326_warn(indio_dev, "failed to poll one shot completion (%d)", err);
+ return err;
+}
+
+/* Retrieve a raw sample and convert it to CPU endianness. */
+static int zpa2326_fetch_raw_sample(const struct iio_dev *indio_dev,
+ enum iio_chan_type type,
+ int *value)
+{
+ int err;
+
+ switch (type) {
+ case IIO_PRESSURE:
+ zpa2326_dbg(indio_dev, "fetching raw pressure sample");
+
+ err = zpa2326_read_block(indio_dev, ZPA2326_PRESS_OUT_XL_REG, 3,
+ (u8 *)value);
+ if (err) {
+ zpa2326_warn(indio_dev, "failed to fetch pressure (%d)",
+ err);
+ return err;
+ }
+
+ /* Pressure is a 24 bits wide little-endian unsigned int. */
+ *value = (((u8 *)value)[2] << 16) | (((u8 *)value)[1] << 8) |
+ ((u8 *)value)[0];
+
+ return IIO_VAL_INT;
+
+ case IIO_TEMP:
+ zpa2326_dbg(indio_dev, "fetching raw temperature sample");
+
+ err = zpa2326_read_block(indio_dev, ZPA2326_TEMP_OUT_L_REG, 2,
+ (u8 *)value);
+ if (err) {
+ zpa2326_warn(indio_dev,
+ "failed to fetch temperature (%d)", err);
+ return err;
+ }
+
+ /* Temperature is a 16 bits wide little-endian signed int. */
+ *value = (int)le16_to_cpup((__le16 *)value);
+
+ return IIO_VAL_INT;
+
+ default:
+ return -EINVAL;
+ }
+}
+
+/* Perform a complete one shot sampling cycle. */
+static int zpa2326_sample_oneshot(struct iio_dev *indio_dev,
+ enum iio_chan_type type,
+ int *value)
+{
+ /* Gain exclusive access to device. */
+ int err = zpa2326_claim_direct_mode(indio_dev);
+ struct zpa2326_private *priv = zpa2326_iio2priv(indio_dev);
+
+ if (err < 0)
+ return err;
+
+ if (err > 0) {
+ /*
+ * We were already power supplied. Just clear hardware fifo to
+ * get rid of samples acquired during previous rounds (if any).
+ * Sampling operation always generates both temperature and
+ * pressure samples. The latter are always enqueued into
+ * hardware fifo. This may lead to situations were pressure
+ * samples still sit into fifo when previous cycle(s) fetched
+ * temperature data only.
+ * Hence, we need to clear hardware fifo content to prevent from
+ * getting outdated values at the end of current cycle.
+ */
+ if (type == IIO_PRESSURE) {
+ err = zpa2326_clear_fifo(indio_dev, 0);
+ if (err)
+ goto release;
+ }
+ } else {
+ /*
+ * We have just been power supplied, i.e. device is in default
+ * "out of reset" state, meaning we need to reconfigure it
+ * entirely.
+ */
+ err = zpa2326_config_oneshot(indio_dev, priv->irq);
+ if (err)
+ goto release;
+ }
+
+ /* Start a sampling cycle in oneshot mode. */
+ err = zpa2326_start_oneshot(indio_dev);
+ if (err)
+ goto release;
+
+ /* Wait for sampling cycle to complete. */
+ zpa2326_dbg(indio_dev, "waiting for one shot completion");
+
+ if (priv->irq > 0)
+ err = zpa2326_wait_oneshot_completion(indio_dev, priv);
+ else
+ err = zpa2326_poll_oneshot_completion(indio_dev);
+
+ if (err)
+ goto release;
+
+ /* Retrieve raw sample value and convert it to CPU endianness. */
+ err = zpa2326_fetch_raw_sample(indio_dev, type, value);
+
+release:
+ zpa2326_release_direct_mode(indio_dev);
+ return err;
+}
+
+/*
+ * Reject external trigger attachment if setting up continuous hardware sampling
+ * mode.
+ */
+static int zpa2326_validate_trigger(struct iio_dev *indio_dev,
+ struct iio_trigger *trigger)
+{
+ int ret = 0;
+
+ mutex_lock(&indio_dev->mlock);
+
+ if (indio_dev->currentmode == INDIO_BUFFER_SOFTWARE)
+ ret = -EBUSY;
+
+ mutex_unlock(&indio_dev->mlock);
+
+ return ret;
+}
+
+/*
+ * Perform an IIO buffered sampling round in one shot mode.
+ *
+ * Handler called by the IIO trigger currently attached to device. Allows to
+ * synchronize this device buffered sampling with external events (such as timer
+ * expiration, external device sample ready, etc...).
+ * Basically run the same sequence of operations as for zpa2326_sample_oneshot()
+ * with the following exceptions:
+ * - hardware fifo is not cleared since already done at buffering enable time
+ * and samples dequeueing always retrieves the most recent value,
+ * - samples endianness is not processed since delegated to userspace in
+ * buffered mode.
+ */
+static irqreturn_t zpa2326_trigger_oneshot(int irq, void *data)
+{
+ struct iio_poll_func *pf = data;
+ struct iio_dev *indio_dev = pf->indio_dev;
+ struct zpa2326_private *priv = zpa2326_iio2priv(indio_dev);
+
+ /* Start a one shot sampling cycle. */
+ if (zpa2326_start_oneshot(indio_dev))
+ goto err;
+
+ if (priv->irq <= 0) {
+ /* No interrupt available: poll for sampling completion. */
+ if (zpa2326_poll_oneshot_completion(indio_dev))
+ goto disable;
+
+ /* Only timestamp sample once it is ready. */
+ priv->timestamp = iio_get_time_ns(indio_dev);
+
+ /* Enqueue to IIO buffer / userspace. */
+ zpa2326_fill_sample_buffer(indio_dev, priv);
+ } else {
+ /* Interrupt handlers will timestamp and get samples for us. */
+ if (zpa2326_wait_oneshot_completion(indio_dev, priv))
+ goto disable;
+ }
+
+ /* Inform attached trigger we are done. */
+ iio_trigger_notify_done(indio_dev->trig);
+ return IRQ_HANDLED;
+
+disable:
+ zpa2326_disable_device(indio_dev);
+err:
+ iio_trigger_notify_done(indio_dev->trig);
+ return IRQ_NONE;
+}
+
+/*
+ * Get device ready for configuration of continuous / triggered sampling modes.
+ *
+ * Called with IIO device's lock held.
+ */
+static int zpa2326_preenable_ring(struct iio_dev *indio_dev)
+{
+ int err = zpa2326_resume(indio_dev);
+
+ if (err < 0)
+ return err;
+
+ /* Tell zpa2326_postenable_ring() if we have just been powered on. */
+ zpa2326_iio2priv(indio_dev)->waken = !err;
+
+ return 0;
+}
+
+/*
+ * Configure and start continuous / triggered sampling.
+ *
+ * Called with IIO device's lock held.
+ * If an error is returned, IIO layer will call our postdisable hook for us,
+ * i.e. no need to explicitly power device off here.
+ */
+static int zpa2326_postenable_ring(struct iio_dev *indio_dev)
+{
+ const struct zpa2326_private *priv = zpa2326_iio2priv(indio_dev);
+ int err;
+
+ if (!priv->waken) {
+ /*
+ * We were already power supplied. Just clear hardware fifo to
+ * get rid of samples acquired during previous rounds (if any).
+ */
+ err = zpa2326_clear_fifo(indio_dev, 0);
+ if (err)
+ return err;
+ }
+
+ if (indio_dev->currentmode == INDIO_BUFFER_TRIGGERED) {
+ /* Prepare device for triggered buffering usage. */
+
+ if (priv->waken) {
+ /*
+ * We have just been power supplied, i.e. device is in
+ * default "out of reset" state, meaning we need to
+ * properly reconfigure it for on trigger demand usage.
+ */
+ err = zpa2326_config_oneshot(indio_dev, priv->irq);
+ if (err)
+ return err;
+ }
+
+ /* Plug our own trigger event handler. */
+ err = iio_triggered_buffer_postenable(indio_dev);
+ if (err)
+ return err;
+
+ /*
+ * Switch back to low power. A complete one shot sampling cycle
+ * will be started upon trigger notification.
+ */
+ return zpa2326_disable_device(indio_dev);
+ }
+
+ /* Prepare then start continuous sampling session. */
+
+ if (priv->waken) {
+ /*
+ * We have just been power supplied, i.e. device is in default
+ * "out of reset" state, meaning we need to unmask data ready
+ * interrupt to process new samples.
+ */
+ err = zpa2326_write_byte(indio_dev, ZPA2326_CTRL_REG1_REG,
+ (u8)~ZPA2326_CTRL_REG1_MASK_DATA_RDY);
+ if (err)
+ goto err;
+ }
+
+ /* Start continuous sampling at required frequency. */
+ err = zpa2326_write_byte(indio_dev, ZPA2326_CTRL_REG3_REG,
+ ZPA2326_CTRL_REG3_ENABLE_MEAS |
+ priv->frequency->odr);
+ if (err)
+ goto err;
+
+ zpa2326_dbg(indio_dev, "continuous mode setup @%sHz",
+ priv->frequency->hz);
+ return 0;
+
+err:
+ zpa2326_err(indio_dev, "failed to setup continuous mode (%d)", err);
+ return err;
+}
+
+/*
+ * Stop continuous / triggered sampling.
+ *
+ * Called with IIO device's lock held.
+ */
+static int zpa2326_predisable_ring(struct iio_dev *indio_dev)
+{
+ if (indio_dev->currentmode == INDIO_BUFFER_SOFTWARE) {
+ int irq = zpa2326_iio2priv(indio_dev)->irq;
+ int err;
+
+ /*
+ * As device is working in continuous mode, handlers may be
+ * accessing resources we are currently freeing...
+ * Prevent this by disabling interrupt handlers and ensure
+ * the device will generate no more interrupts unless explicitly
+ * required to, i.e. by restoring back to default one shot mode.
+ */
+ disable_irq(irq);
+
+ /*
+ * Disable continuous sampling mode to restore settings for
+ * one shot / direct sampling operations.
+ */
+ err = zpa2326_write_byte(indio_dev, ZPA2326_CTRL_REG3_REG,
+ zpa2326_highest_frequency()->odr);
+ if (err)
+ goto err;
+
+ /*
+ * Now that device won't generate interrupts on its own,
+ * acknowledge any currently active interrupts (may happen on
+ * rare occasions while stopping continuous mode).
+ */
+ err = zpa2326_read_byte(indio_dev, ZPA2326_INT_SOURCE_REG);
+ if (err < 0)
+ goto err;
+
+ /*
+ * Re-enable interrupts only if we can guarantee the device will
+ * generate no more interrupts to prevent handlers from
+ * accessing released resources.
+ */
+ enable_irq(irq);
+ return 0;
+
+err:
+ zpa2326_err(indio_dev,
+ "failed to stop buffering, leaving interrupt disabled... (%d)",
+ err);
+ return err;
+ }
+
+ /* Triggered buffering: unplug our own trigger event handler. */
+ return iio_triggered_buffer_predisable(indio_dev);
+}
+
+static int zpa2326_postdisable_ring(struct iio_dev *indio_dev)
+{
+ zpa2326_suspend(indio_dev);
+ return 0;
+}
+
+static const struct iio_buffer_setup_ops zpa2326_ring_setup_ops = {
+ .preenable = zpa2326_preenable_ring,
+ .postenable = zpa2326_postenable_ring,
+ .predisable = zpa2326_predisable_ring,
+ .postdisable = zpa2326_postdisable_ring
+};
+
+/* Setup buffered sampling. */
+static int zpa2326_init_ring(struct device *slave,
+ struct iio_dev *indio_dev,
+ int irq)
+{
+ int err = devm_iio_triggered_buffer_setup(slave, indio_dev, NULL,
+ zpa2326_trigger_oneshot,
+ &zpa2326_ring_setup_ops);
+ if (err)
+ return err;
+
+ if (irq > 0)
+ /* Allow continuous hardware sampling if interrupt available. */
+ indio_dev->modes |= INDIO_BUFFER_SOFTWARE;
+
+ return 0;
+}
+
+/*
+ * Using our own interrupt driven trigger to start a triggered buffer sampling
+ * round is pure nonsense: use continuous hardware sampling mode instead, i.e.
+ * INDIO_BUFFER_SOFTWARE. Or synchronize against an external trigger / device.
+ */
+static int zpa2326_validate_device(struct iio_trigger *trigger,
+ struct iio_dev *indio_dev)
+{
+ if (indio_dev == (struct iio_dev *)iio_trigger_get_drvdata(trigger))
+ return -EINVAL;
+
+ return 0;
+}
+
+static const struct iio_trigger_ops zpa2326_trigger_ops = {
+ .owner = THIS_MODULE,
+ .validate_device = zpa2326_validate_device,
+};
+
+/*
+ * Create an interrupt driven / hardware trigger allowing to notify external
+ * devices a new sample is ready.
+ */
+static int zpa2326_init_trigger(struct device *slave,
+ struct iio_dev *indio_dev,
+ struct zpa2326_private *private,
+ int irq)
+{
+ struct iio_trigger *trigger;
+ int err;
+
+ if (irq <= 0)
+ return 0;
+
+ trigger = devm_iio_trigger_alloc(slave, "%s-sample-ready-dev%d",
+ indio_dev->name, indio_dev->id);
+ if (!trigger)
+ return -ENOMEM;
+
+ /* Basic setup. */
+ trigger->dev.parent = slave;
+ trigger->ops = &zpa2326_trigger_ops;
+ iio_trigger_set_drvdata(trigger, indio_dev);
+
+ private->trigger = trigger;
+
+ /* Register to triggers space. */
+ err = devm_iio_trigger_register(slave, trigger);
+ if (err)
+ dev_err(slave, "failed to register hardware trigger (%d)", err);
+
+ return err;
+}
+
+/*
+ * Debugfs stuff
+ *
+ * Note this is mostly unusable with power management enabled systems since
+ * registers content is lost during power off -> on transitions.
+ */
+#if defined(CONFIG_DEBUG_FS)
+#define zpa2326_debugfs_ptr(_ptr) _ptr
+
+static int zpa2326_debugfs_read(struct iio_dev *indio_dev,
+ u8 reg,
+ unsigned int *value)
+{
+ int err = zpa2326_claim_direct_mode(indio_dev);
+
+ if (err < 0)
+ return err;
+
+ err = zpa2326_read_byte(indio_dev, reg);
+ if (err < 0)
+ goto release;
+
+ *value = err;
+ err = 0;
+
+release:
+ zpa2326_release_direct_mode(indio_dev);
+ return err;
+}
+
+static int zpa2326_debugfs_write(struct iio_dev *indio_dev,
+ u8 reg,
+ unsigned int value)
+{
+ int err = zpa2326_claim_direct_mode(indio_dev);
+
+ if (err < 0)
+ return err;
+
+ err = zpa2326_write_byte(indio_dev, reg, value);
+
+ zpa2326_release_direct_mode(indio_dev);
+ return err;
+}
+
+static int zpa2326_debugfs_reg_access(struct iio_dev *indio_dev,
+ unsigned int reg,
+ unsigned int writeval,
+ unsigned int *readval)
+{
+ if (readval)
+ return zpa2326_debugfs_read(indio_dev, reg, readval);
+
+ return zpa2326_debugfs_write(indio_dev, reg, writeval);
+}
+#else /* !CONFIG_DEBUG_FS */
+#define zpa2326_debugfs_ptr(_ptr) NULL
+#endif /* !CONFIG_DEBUG_FS */
+
+static int zpa2326_read_raw(struct iio_dev *indio_dev,
+ struct iio_chan_spec const *chan,
+ int *val,
+ int *val2,
+ long mask)
+{
+ switch (mask) {
+ case IIO_CHAN_INFO_RAW:
+ return zpa2326_sample_oneshot(indio_dev, chan->type, val);
+
+ case IIO_CHAN_INFO_SCALE:
+ switch (chan->type) {
+ case IIO_PRESSURE:
+ /*
+ * Pressure resolution is 1/64 Pascal. Scale to kPascal
+ * as required by IIO ABI.
+ */
+ *val = 0;
+ *val2 = 1000000 / 64;
+ return IIO_VAL_INT_PLUS_NANO;
+
+ case IIO_TEMP:
+ /*
+ * Temperature follows the equation:
+ * Temp[degC] = Tempcode * 0.00649 - 176.83
+ * where:
+ * Tempcode is composed the raw sampled 16 bits.
+ *
+ * Hence, to produce a temperature in milli-degrees
+ * Celsius according to IIO ABI, we need to apply the
+ * following equation to raw samples:
+ * Temp[milli degC] = (Tempcode + Offset) * Scale
+ * where:
+ * Offset = -176.83 / 0.00649
+ * Scale = 0.00649 * 1000
+ */
+ *val = 6;
+ *val2 = 490000;
+ return IIO_VAL_INT_PLUS_MICRO;
+
+ default:
+ return -EINVAL;
+ }
+
+ case IIO_CHAN_INFO_OFFSET:
+ switch (chan->type) {
+ case IIO_TEMP:
+ *val = -17683000 / 649;
+ *val2 = ((17683000 % 649) * 1000000000ULL) / 649ULL;
+ return IIO_VAL_INT_PLUS_NANO;
+
+ default:
+ return -EINVAL;
+ }
+
+ default:
+ return -EINVAL;
+ }
+}
+
+static ssize_t zpa2326_show_frequency(struct device *dev,
+ struct device_attribute *attr,
+ char *buf)
+{
+ struct iio_dev *indio_dev = dev_to_iio_dev(dev);
+ const struct zpa2326_frequency *freq;
+
+ mutex_lock(&indio_dev->mlock);
+ freq = zpa2326_iio2priv(indio_dev)->frequency;
+ mutex_unlock(&indio_dev->mlock);
+
+ return sprintf(buf, "%s\n", freq->hz);
+}
+
+static ssize_t zpa2326_store_frequency(struct device *dev,
+ struct device_attribute *attr,
+ const char *buf,
+ size_t len)
+{
+ struct iio_dev *indio_dev = dev_to_iio_dev(dev);
+ struct zpa2326_private *priv = zpa2326_iio2priv(indio_dev);
+ int freq, err;
+
+ /* Check if requested frequency is supported. */
+ for (freq = 0; freq < ARRAY_SIZE(zpa2326_sampling_frequencies); freq++)
+ if (sysfs_streq(zpa2326_sampling_frequencies[freq].hz, buf))
+ break;
+ if (freq == ARRAY_SIZE(zpa2326_sampling_frequencies))
+ return -EINVAL;
+
+ /* Don't allow changing frequency if buffered sampling is ongoing. */
+ err = iio_device_claim_direct_mode(indio_dev);
+ if (err)
+ return err;
+
+ priv->frequency = &zpa2326_sampling_frequencies[freq];
+
+ iio_device_release_direct_mode(indio_dev);
+ return len;
+}
+
+static IIO_DEV_ATTR_SAMP_FREQ(S_IWUSR | S_IRUGO,
+ zpa2326_show_frequency,
+ zpa2326_store_frequency);
+
+/* Expose supported hardware sampling frequencies (Hz) through sysfs. */
+static IIO_CONST_ATTR_SAMP_FREQ_AVAIL("1 5 11 23");
+
+static struct attribute *zpa2326_attributes[] = {
+ &iio_dev_attr_sampling_frequency.dev_attr.attr,
+ &iio_const_attr_sampling_frequency_available.dev_attr.attr,
+ NULL
+};
+
+static const struct attribute_group zpa2326_attribute_group = {
+ .attrs = zpa2326_attributes,
+};
+
+static const struct iio_chan_spec zpa2326_channels[] = {
+ [0] = {
+ .type = IIO_PRESSURE,
+ .scan_index = 0,
+ .scan_type = {
+ .sign = 'u',
+ .realbits = 24,
+ .storagebits = 32,
+ .endianness = IIO_LE,
+ },
+ .info_mask_separate = BIT(IIO_CHAN_INFO_RAW) |
+ BIT(IIO_CHAN_INFO_SCALE),
+ },
+ [1] = {
+ .type = IIO_TEMP,
+ .scan_index = 1,
+ .scan_type = {
+ .sign = 's',
+ .realbits = 16,
+ .storagebits = 16,
+ .endianness = IIO_LE,
+ },
+ .info_mask_separate = BIT(IIO_CHAN_INFO_RAW) |
+ BIT(IIO_CHAN_INFO_SCALE) |
+ BIT(IIO_CHAN_INFO_OFFSET),
+ },
+ [2] = IIO_CHAN_SOFT_TIMESTAMP(2),
+};
+
+static const struct iio_info zpa2326_info = {
+ .driver_module = THIS_MODULE,
+ .attrs = &zpa2326_attribute_group,
+ .read_raw = zpa2326_read_raw,
+ .debugfs_reg_access = zpa2326_debugfs_ptr(zpa2326_debugfs_reg_access),
+ .validate_trigger = zpa2326_validate_trigger,
+};
+
+/* Allocate and initialize a basic ZPA IIO device. */
+static struct iio_dev *zpa2326_create_iio(struct device *device,
+ const char *name,
+ struct regmap *regmap)
+{
+ struct iio_dev *indio_dev;
+
+ /* Allocate space to hold IIO device internal state. */
+ indio_dev = devm_iio_device_alloc(device,
+ sizeof(struct zpa2326_private));
+ if (!indio_dev)
+ return NULL;
+
+ /* Setup for userspace synchronous on demand sampling. */
+ indio_dev->modes = INDIO_DIRECT_MODE;
+ indio_dev->dev.parent = device;
+ indio_dev->channels = zpa2326_channels;
+ indio_dev->num_channels = ARRAY_SIZE(zpa2326_channels);
+ indio_dev->name = name;
+ indio_dev->info = &zpa2326_info;
+
+ /* Plug device's underlying bus abstraction. */
+ zpa2326_iio2priv(indio_dev)->regmap = regmap;
+
+ /* Init hardware sampling frequency to highest rate supported. */
+ zpa2326_iio2priv(indio_dev)->frequency = zpa2326_highest_frequency();
+
+ return indio_dev;
+}
+
+static int zpa2326_detect_hardware(struct device *slave,
+ struct zpa2326_private *private,
+ unsigned int hwid,
+ struct regmap *regmap)
+{
+ int err;
+ unsigned int id;
+
+ private->vref = devm_regulator_get(slave, "vref");
+ if (IS_ERR(private->vref))
+ return PTR_ERR(private->vref);
+
+ private->vdd = devm_regulator_get(slave, "vdd");
+ if (IS_ERR(private->vdd))
+ return PTR_ERR(private->vdd);
+
+ err = regulator_enable(private->vref);
+ if (err)
+ return err;
+
+ err = regulator_enable(private->vdd);
+ if (err)
+ goto vref;
+
+ usleep_range(ZPA2326_TPUP_USEC_MIN, ZPA2326_TPUP_USEC_MAX);
+
+ /*
+ * Read identification register to check we are talking to the right
+ * slave.
+ */
+ err = regmap_read(regmap, ZPA2326_DEVICE_ID_REG, &id);
+ if (err)
+ goto vdd;
+
+ if (id != hwid) {
+ dev_err(slave, "found device with unexpected id %02x", id);
+ err = -ENODEV;
+ goto vdd;
+ }
+
+ return 0;
+
+vdd:
+ regulator_disable(private->vdd);
+vref:
+ regulator_disable(private->vref);
+ return err;
+}
+
+int zpa2326_probe(struct device *slave,
+ const char *name,
+ int irq,
+ unsigned int hwid,
+ struct regmap *regmap)
+{
+ struct iio_dev *indio_dev = zpa2326_create_iio(slave, name,
+ regmap);
+ struct zpa2326_private *priv;
+ int err;
+
+ if (!indio_dev)
+ return -ENOMEM;
+
+ priv = zpa2326_iio2priv(indio_dev);
+
+ err = zpa2326_detect_hardware(slave, priv, hwid, regmap);
+ if (err)
+ return err;
+
+ err = zpa2326_init_ring(slave, indio_dev, irq);
+ if (err)
+ goto disable;
+
+ err = zpa2326_init_trigger(slave, indio_dev, priv, irq);
+ if (err)
+ goto disable;
+
+ err = zpa2326_init_irq(slave, indio_dev, priv, irq);
+ if (err)
+ goto disable;
+
+ err = zpa2326_config_oneshot(indio_dev, irq);
+ if (err)
+ goto disable;
+
+ err = zpa2326_disable_device(indio_dev);
+ if (err)
+ goto off;
+
+ dev_set_drvdata(slave, indio_dev);
+
+ zpa2326_init_runtime(slave);
+
+ err = iio_device_register(indio_dev);
+ if (err) {
+ zpa2326_fini_runtime(slave);
+ goto off;
+ }
+
+ dev_dbg(slave, "%s barometer ready", name);
+ return 0;
+
+disable:
+ zpa2326_disable_device(indio_dev);
+off:
+ zpa2326_power_off(indio_dev, priv);
+ return err;
+}
+EXPORT_SYMBOL_GPL(zpa2326_probe);
+
+int zpa2326_remove(const struct device *slave)
+{
+ struct iio_dev *indio_dev = dev_get_drvdata(slave);
+
+ iio_device_unregister(indio_dev);
+ zpa2326_fini_runtime(zpa2326_iio2slave(indio_dev));
+ zpa2326_disable_device(indio_dev);
+ zpa2326_power_off(indio_dev, zpa2326_iio2priv(indio_dev));
+ return 0;
+}
+EXPORT_SYMBOL_GPL(zpa2326_remove);
+
+MODULE_AUTHOR("Gregor Boirie <gregor.boirie@parrot.com>");
+MODULE_DESCRIPTION("Core driver for Murata ZPA2326 pressure sensor");
+MODULE_LICENSE("GPL v2");
diff --git a/drivers/iio/pressure/zpa2326.h b/drivers/iio/pressure/zpa2326.h
new file mode 100644
index 0000000..19a7cfa
--- /dev/null
+++ b/drivers/iio/pressure/zpa2326.h
@@ -0,0 +1,93 @@
+/*
+ * Murata ZPA2326 pressure and temperature sensor IIO driver
+ *
+ * Copyright (c) 2016 Parrot S.A.
+ *
+ * Author: Gregor Boirie <gregor.boirie@parrot.com>
+ *
+ * This program is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License version 2 as published by
+ * the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
+ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for
+ * more details.
+ */
+
+#ifndef _ZPA2326_H
+#define _ZPA2326_H
+
+#include <linux/iio/iio.h>
+
+/* Register map. */
+#define ZPA2326_REF_P_XL_REG (0x8)
+#define ZPA2326_REF_P_L_REG (0x9)
+#define ZPA2326_REF_P_H_REG (0xa)
+#define ZPA2326_DEVICE_ID_REG (0xf)
+#define ZPA2326_DEVICE_ID (0xb9)
+#define ZPA2326_RES_CONF_REG (0x10)
+#define ZPA2326_CTRL_REG0_REG (0x20)
+#define ZPA2326_CTRL_REG0_ONE_SHOT BIT(0)
+#define ZPA2326_CTRL_REG0_ENABLE BIT(1)
+#define ZPA2326_CTRL_REG1_REG (0x21)
+#define ZPA2326_CTRL_REG1_MASK_DATA_RDY BIT(2)
+#define ZPA2326_CTRL_REG2_REG (0x22)
+#define ZPA2326_CTRL_REG2_SWRESET BIT(2)
+#define ZPA2326_CTRL_REG3_REG (0x23)
+#define ZPA2326_CTRL_REG3_ODR_SHIFT (4)
+#define ZPA2326_CTRL_REG3_ENABLE_MEAS BIT(7)
+#define ZPA2326_INT_SOURCE_REG (0x24)
+#define ZPA2326_INT_SOURCE_DATA_RDY BIT(2)
+#define ZPA2326_THS_P_LOW_REG (0x25)
+#define ZPA2326_THS_P_HIGH_REG (0x26)
+#define ZPA2326_STATUS_REG (0x27)
+#define ZPA2326_STATUS_P_DA BIT(1)
+#define ZPA2326_STATUS_FIFO_E BIT(2)
+#define ZPA2326_STATUS_P_OR BIT(5)
+#define ZPA2326_PRESS_OUT_XL_REG (0x28)
+#define ZPA2326_PRESS_OUT_L_REG (0x29)
+#define ZPA2326_PRESS_OUT_H_REG (0x2a)
+#define ZPA2326_TEMP_OUT_L_REG (0x2b)
+#define ZPA2326_TEMP_OUT_H_REG (0x2c)
+
+bool zpa2326_isreg_writeable(struct device *dev, unsigned int reg);
+bool zpa2326_isreg_readable(struct device *dev, unsigned int reg);
+bool zpa2326_isreg_precious(struct device *dev, unsigned int reg);
+
+struct regmap;
+
+/*
+ * zpa2326_probe() - instantiate and register core zpa2326 IIO device
+ * @slave: device the IIO device will be a child of
+ * @name: arbitrary name identify the device
+ * @irq: interrupt line, negative if none
+ * @hwid: expected device hardware id
+ * @regmap: registers map used to abstract underlying bus accesses
+ */
+int zpa2326_probe(struct device *slave,
+ const char *name,
+ int irq,
+ unsigned int hwid,
+ struct regmap *regmap);
+
+/*
+ * zpa2326_remove() - unregister and destroy core zpa2326 IIO device
+ * @slave: device the IIO device to remove is a child of
+ */
+int zpa2326_remove(const struct device *slave);
+
+static inline struct device *zpa2326_iio2slave(const struct iio_dev *indio_dev)
+{
+ return indio_dev->dev.parent;
+}
+
+#ifdef CONFIG_PM
+#include <linux/pm.h>
+extern const struct dev_pm_ops zpa2326_pm_ops;
+#define ZPA2326_PM_OPS (&zpa2326_pm_ops)
+#else
+#define ZPA2326_PM_OPS (NULL)
+#endif
+
+#endif
diff --git a/drivers/iio/pressure/zpa2326_i2c.c b/drivers/iio/pressure/zpa2326_i2c.c
new file mode 100644
index 0000000..0a6b0a8
--- /dev/null
+++ b/drivers/iio/pressure/zpa2326_i2c.c
@@ -0,0 +1,97 @@
+/*
+ * Murata ZPA2326 I2C pressure and temperature sensor driver
+ *
+ * Copyright (c) 2016 Parrot S.A.
+ *
+ * Author: Gregor Boirie <gregor.boirie@parrot.com>
+ *
+ * This program is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License version 2 as published by
+ * the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
+ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for
+ * more details.
+ */
+
+#include <linux/module.h>
+#include <linux/regmap.h>
+#include <linux/i2c.h>
+#include <linux/of_device.h>
+#include "zpa2326.h"
+
+/*
+ * read_flag_mask:
+ * - address bit 7 must be set to request a register read operation
+ */
+static const struct regmap_config zpa2326_regmap_i2c_config = {
+ .reg_bits = 8,
+ .val_bits = 8,
+ .writeable_reg = zpa2326_isreg_writeable,
+ .readable_reg = zpa2326_isreg_readable,
+ .precious_reg = zpa2326_isreg_precious,
+ .max_register = ZPA2326_TEMP_OUT_H_REG,
+ .read_flag_mask = BIT(7),
+ .cache_type = REGCACHE_NONE,
+};
+
+static unsigned int zpa2326_i2c_hwid(const struct i2c_client *client)
+{
+ /* Identification register bit 1 mirrors device address bit 0. */
+#define ZPA2326_SA0(_addr) (_addr & BIT(0))
+#define ZPA2326_DEVICE_ID_SA0_SHIFT (1)
+
+ return (ZPA2326_DEVICE_ID |
+ (ZPA2326_SA0(client->addr) << ZPA2326_DEVICE_ID_SA0_SHIFT));
+}
+
+static int zpa2326_probe_i2c(struct i2c_client *client,
+ const struct i2c_device_id *i2c_id)
+{
+ struct regmap *regmap;
+
+ regmap = devm_regmap_init_i2c(client, &zpa2326_regmap_i2c_config);
+ if (IS_ERR(regmap)) {
+ dev_err(&client->dev, "failed to init registers map");
+ return PTR_ERR(regmap);
+ }
+
+ return zpa2326_probe(&client->dev, i2c_id->name, client->irq,
+ zpa2326_i2c_hwid(client), regmap);
+}
+
+static int zpa2326_remove_i2c(struct i2c_client *client)
+{
+ return zpa2326_remove(&client->dev);
+}
+
+static const struct i2c_device_id zpa2326_i2c_ids[] = {
+ { "zpa2326", 0 },
+ { },
+};
+MODULE_DEVICE_TABLE(i2c, zpa2326_i2c_ids);
+
+#if defined(CONFIG_OF)
+static const struct of_device_id zpa2326_i2c_matches[] = {
+ { .compatible = "murata,zpa2326" },
+ { }
+};
+MODULE_DEVICE_TABLE(of, zpa2326_i2c_matches);
+#endif
+
+static struct i2c_driver zpa2326_i2c_driver = {
+ .driver = {
+ .name = "zpa2326-i2c",
+ .of_match_table = of_match_ptr(zpa2326_i2c_matches),
+ .pm = ZPA2326_PM_OPS,
+ },
+ .probe = zpa2326_probe_i2c,
+ .remove = zpa2326_remove_i2c,
+ .id_table = zpa2326_i2c_ids,
+};
+module_i2c_driver(zpa2326_i2c_driver);
+
+MODULE_AUTHOR("Gregor Boirie <gregor.boirie@parrot.com>");
+MODULE_DESCRIPTION("I2C driver for Murata ZPA2326 pressure sensor");
+MODULE_LICENSE("GPL v2");
diff --git a/drivers/iio/pressure/zpa2326_spi.c b/drivers/iio/pressure/zpa2326_spi.c
new file mode 100644
index 0000000..d3c8862
--- /dev/null
+++ b/drivers/iio/pressure/zpa2326_spi.c
@@ -0,0 +1,101 @@
+/*
+ * Murata ZPA2326 SPI pressure and temperature sensor driver
+ *
+ * Copyright (c) 2016 Parrot S.A.
+ *
+ * Author: Gregor Boirie <gregor.boirie@parrot.com>
+ *
+ * This program is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License version 2 as published by
+ * the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
+ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for
+ * more details.
+ */
+
+#include <linux/module.h>
+#include <linux/regmap.h>
+#include <linux/spi/spi.h>
+#include <linux/of_device.h>
+#include "zpa2326.h"
+
+/*
+ * read_flag_mask:
+ * - address bit 7 must be set to request a register read operation
+ * - address bit 6 must be set to request register address auto increment
+ */
+static const struct regmap_config zpa2326_regmap_spi_config = {
+ .reg_bits = 8,
+ .val_bits = 8,
+ .writeable_reg = zpa2326_isreg_writeable,
+ .readable_reg = zpa2326_isreg_readable,
+ .precious_reg = zpa2326_isreg_precious,
+ .max_register = ZPA2326_TEMP_OUT_H_REG,
+ .read_flag_mask = BIT(7) | BIT(6),
+ .cache_type = REGCACHE_NONE,
+};
+
+static int zpa2326_probe_spi(struct spi_device *slave)
+{
+ struct regmap *regmap;
+ int err;
+
+ regmap = devm_regmap_init_spi(slave, &zpa2326_regmap_spi_config);
+ if (IS_ERR(regmap)) {
+ dev_err(&slave->dev, "failed to init registers map");
+ return PTR_ERR(regmap);
+ }
+
+ /*
+ * Enforce SPI slave settings to prevent from DT misconfiguration.
+ *
+ * Clock is idle high. Sampling happens on trailing edge, i.e., rising
+ * edge. Maximum bus frequency is 1 mHz. Registers are 8 bits wide.
+ */
+ slave->mode = SPI_MODE_3;
+ slave->max_speed_hz = min(slave->max_speed_hz, 1000U * 1000U);
+ slave->bits_per_word = 8;
+ err = spi_setup(slave);
+ if (err < 0)
+ return err;
+
+ return zpa2326_probe(&slave->dev, spi_get_device_id(slave)->name,
+ slave->irq, ZPA2326_DEVICE_ID, regmap);
+}
+
+static int zpa2326_remove_spi(struct spi_device *slave)
+{
+ return zpa2326_remove(&slave->dev);
+}
+
+static const struct spi_device_id zpa2326_spi_ids[] = {
+ { "zpa2326", 0 },
+ { },
+};
+MODULE_DEVICE_TABLE(spi, zpa2326_spi_ids);
+
+#if defined(CONFIG_OF)
+static const struct of_device_id zpa2326_spi_matches[] = {
+ { .compatible = "murata,zpa2326" },
+ { }
+};
+MODULE_DEVICE_TABLE(of, zpa2326_spi_matches);
+#endif
+
+static struct spi_driver zpa2326_spi_driver = {
+ .driver = {
+ .name = "zpa2326-spi",
+ .of_match_table = of_match_ptr(zpa2326_spi_matches),
+ .pm = ZPA2326_PM_OPS,
+ },
+ .probe = zpa2326_probe_spi,
+ .remove = zpa2326_remove_spi,
+ .id_table = zpa2326_spi_ids,
+};
+module_spi_driver(zpa2326_spi_driver);
+
+MODULE_AUTHOR("Gregor Boirie <gregor.boirie@parrot.com>");
+MODULE_DESCRIPTION("SPI driver for Murata ZPA2326 pressure sensor");
+MODULE_LICENSE("GPL v2");
--
2.1.4
^ permalink raw reply related [flat|nested] 6+ messages in thread