[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20190119173330.29e49c63@archlinux>
Date: Sat, 19 Jan 2019 17:33:30 +0000
From: Jonathan Cameron <jic23@...nel.org>
To: Denis CIOCCA <denis.ciocca@...com>
Cc: Mario Tesi <martepisa@...il.com>,
"linux-iio@...r.kernel.org" <linux-iio@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
Mario TESI <mario.tesi@...com>
Subject: Re: [PATCH] iio:st_pressure:initial lps22hh sensor support
On Tue, 15 Jan 2019 00:18:58 +0000
Denis CIOCCA <denis.ciocca@...com> wrote:
> Acked-by: Denis Ciocca <denis.ciocca@...com>
>
>
> -----Original Message-----
> From: linux-iio-owner@...r.kernel.org <linux-iio-owner@...r.kernel.org> On Behalf Of Mario Tesi
> Sent: Monday, January 14, 2019 9:24 AM
> To: jic23@...nel.org
> Cc: linux-iio@...r.kernel.org; linux-kernel@...r.kernel.org; Mario TESI <mario.tesi@...com>
> Subject: [PATCH] iio:st_pressure:initial lps22hh sensor support
>
> From: mario tesi <mario.tesi@...com>
>
> Initial support for ST LPS22HH pressure sensor. Datasheet:
> http://www2.st.com/resource/en/datasheet/lps22hh.pdf
>
> Features:
> * pressure, temperature data and timestamping channels
> * sampling frequency selection [1..200] Hz
> * interrupt based trigger
> * over I2C or SPI interface
>
> Signed-off-by: mario tesi <mario.tesi@...com>
Applied to the togreg branch of iio.git and pushed out as testing for
the autobuilders to play with it.
Thanks,
Jonathan
> ---
> drivers/iio/pressure/Kconfig | 2 +-
> drivers/iio/pressure/st_pressure.h | 2 +
> drivers/iio/pressure/st_pressure_core.c | 69 +++++++++++++++++++++++++++++++++ drivers/iio/pressure/st_pressure_i2c.c | 5 +++ drivers/iio/pressure/st_pressure_spi.c | 5 +++
> 5 files changed, 82 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/iio/pressure/Kconfig b/drivers/iio/pressure/Kconfig index eaa7cfc..efeb89f 100644
> --- a/drivers/iio/pressure/Kconfig
> +++ b/drivers/iio/pressure/Kconfig
> @@ -165,7 +165,7 @@ config IIO_ST_PRESS
> select IIO_TRIGGERED_BUFFER if (IIO_BUFFER)
> help
> Say yes here to build support for STMicroelectronics pressure
> - sensors: LPS001WP, LPS25H, LPS331AP, LPS22HB.
> + sensors: LPS001WP, LPS25H, LPS331AP, LPS22HB, LPS22HH.
>
> This driver can also be built as a module. If so, these modules
> will be created:
> diff --git a/drivers/iio/pressure/st_pressure.h b/drivers/iio/pressure/st_pressure.h
> index e67eb0d..5794660 100644
> --- a/drivers/iio/pressure/st_pressure.h
> +++ b/drivers/iio/pressure/st_pressure.h
> @@ -21,6 +21,7 @@ enum st_press_type {
> LPS22HB,
> LPS33HW,
> LPS35HW,
> + LPS22HH,
> ST_PRESS_MAX,
> };
>
> @@ -30,6 +31,7 @@ enum st_press_type {
> #define LPS22HB_PRESS_DEV_NAME "lps22hb"
> #define LPS33HW_PRESS_DEV_NAME "lps33hw"
> #define LPS35HW_PRESS_DEV_NAME "lps35hw"
> +#define LPS22HH_PRESS_DEV_NAME "lps22hh"
>
> /**
> * struct st_sensors_platform_data - default press platform data diff --git a/drivers/iio/pressure/st_pressure_core.c b/drivers/iio/pressure/st_pressure_core.c
> index 4ddb6cf..38dcdb7 100644
> --- a/drivers/iio/pressure/st_pressure_core.c
> +++ b/drivers/iio/pressure/st_pressure_core.c
> @@ -492,6 +492,75 @@
> .multi_read_bit = false,
> .bootime = 2,
> },
> + {
> + /*
> + * CUSTOM VALUES FOR LPS22HH SENSOR
> + * See LPS22HH datasheet:
> + * http://www2.st.com/resource/en/datasheet/lps22hh.pdf
> + */
> + .wai = 0xb3,
> + .wai_addr = ST_SENSORS_DEFAULT_WAI_ADDRESS,
> + .sensors_supported = {
> + [0] = LPS22HH_PRESS_DEV_NAME,
> + },
> + .ch = (struct iio_chan_spec *)st_press_lps22hb_channels,
> + .num_ch = ARRAY_SIZE(st_press_lps22hb_channels),
> + .odr = {
> + .addr = 0x10,
> + .mask = 0x70,
> + .odr_avl = {
> + { .hz = 1, .value = 0x01 },
> + { .hz = 10, .value = 0x02 },
> + { .hz = 25, .value = 0x03 },
> + { .hz = 50, .value = 0x04 },
> + { .hz = 75, .value = 0x05 },
> + { .hz = 100, .value = 0x06 },
> + { .hz = 200, .value = 0x07 },
> + },
> + },
> + .pw = {
> + .addr = 0x10,
> + .mask = 0x70,
> + .value_off = ST_SENSORS_DEFAULT_POWER_OFF_VALUE,
> + },
> + .fs = {
> + .fs_avl = {
> + /*
> + * Pressure and temperature sensitivity values
> + * as defined in table 3 of LPS22HH datasheet.
> + */
> + [0] = {
> + .num = ST_PRESS_FS_AVL_1260MB,
> + .gain = ST_PRESS_KPASCAL_NANO_SCALE,
> + .gain2 = ST_PRESS_LPS22HB_LSB_PER_CELSIUS,
> + },
> + },
> + },
> + .bdu = {
> + .addr = 0x10,
> + .mask = BIT(1),
> + },
> + .drdy_irq = {
> + .int1 = {
> + .addr = 0x12,
> + .mask = BIT(2),
> + .addr_od = 0x11,
> + .mask_od = BIT(5),
> + },
> + .addr_ihl = 0x11,
> + .mask_ihl = BIT(6),
> + .stat_drdy = {
> + .addr = ST_SENSORS_DEFAULT_STAT_ADDR,
> + .mask = 0x03,
> + },
> + },
> + .sim = {
> + .addr = 0x10,
> + .value = BIT(0),
> + },
> + .multi_read_bit = false,
> + .bootime = 2,
> + },
> };
>
> static int st_press_write_raw(struct iio_dev *indio_dev, diff --git a/drivers/iio/pressure/st_pressure_i2c.c b/drivers/iio/pressure/st_pressure_i2c.c
> index 2026a10..a60849d 100644
> --- a/drivers/iio/pressure/st_pressure_i2c.c
> +++ b/drivers/iio/pressure/st_pressure_i2c.c
> @@ -45,6 +45,10 @@
> .compatible = "st,lps35hw",
> .data = LPS35HW_PRESS_DEV_NAME,
> },
> + {
> + .compatible = "st,lps22hh",
> + .data = LPS22HH_PRESS_DEV_NAME,
> + },
> {},
> };
> MODULE_DEVICE_TABLE(of, st_press_of_match); @@ -69,6 +73,7 @@
> { LPS22HB_PRESS_DEV_NAME, LPS22HB },
> { LPS33HW_PRESS_DEV_NAME, LPS33HW },
> { LPS35HW_PRESS_DEV_NAME, LPS35HW },
> + { LPS22HH_PRESS_DEV_NAME, LPS22HH },
> {},
> };
> MODULE_DEVICE_TABLE(i2c, st_press_id_table); diff --git a/drivers/iio/pressure/st_pressure_spi.c b/drivers/iio/pressure/st_pressure_spi.c
> index 9a3441b..79a12ed 100644
> --- a/drivers/iio/pressure/st_pressure_spi.c
> +++ b/drivers/iio/pressure/st_pressure_spi.c
> @@ -49,6 +49,10 @@
> .compatible = "st,lps35hw",
> .data = LPS35HW_PRESS_DEV_NAME,
> },
> + {
> + .compatible = "st,lps22hh",
> + .data = LPS22HH_PRESS_DEV_NAME,
> + },
> {},
> };
> MODULE_DEVICE_TABLE(of, st_press_of_match); @@ -93,6 +97,7 @@ static int st_press_spi_remove(struct spi_device *spi)
> { LPS22HB_PRESS_DEV_NAME },
> { LPS33HW_PRESS_DEV_NAME },
> { LPS35HW_PRESS_DEV_NAME },
> + { LPS22HH_PRESS_DEV_NAME },
> {},
> };
> MODULE_DEVICE_TABLE(spi, st_press_id_table);
> --
> 1.9.1
>
Powered by blists - more mailing lists