* [PATCH] iio:st_pressure:initial lps22hh sensor support
@ 2019-01-14 17:24 Mario Tesi
2019-01-15 0:18 ` Denis CIOCCA
0 siblings, 1 reply; 4+ messages in thread
From: Mario Tesi @ 2019-01-14 17:24 UTC (permalink / raw)
To: jic23; +Cc: linux-iio, linux-kernel, mario.tesi
From: mario tesi <mario.tesi@st.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@st.com>
---
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
^ permalink raw reply related [flat|nested] 4+ messages in thread
* RE: [PATCH] iio:st_pressure:initial lps22hh sensor support
2019-01-14 17:24 [PATCH] iio:st_pressure:initial lps22hh sensor support Mario Tesi
@ 2019-01-15 0:18 ` Denis CIOCCA
2019-01-19 17:33 ` Jonathan Cameron
0 siblings, 1 reply; 4+ messages in thread
From: Denis CIOCCA @ 2019-01-15 0:18 UTC (permalink / raw)
To: Mario Tesi, jic23; +Cc: linux-iio, linux-kernel, Mario TESI
Acked-by: Denis Ciocca <denis.ciocca@st.com>
-----Original Message-----
From: linux-iio-owner@vger.kernel.org <linux-iio-owner@vger.kernel.org> On Behalf Of Mario Tesi
Sent: Monday, January 14, 2019 9:24 AM
To: jic23@kernel.org
Cc: linux-iio@vger.kernel.org; linux-kernel@vger.kernel.org; Mario TESI <mario.tesi@st.com>
Subject: [PATCH] iio:st_pressure:initial lps22hh sensor support
From: mario tesi <mario.tesi@st.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@st.com>
---
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
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] iio:st_pressure:initial lps22hh sensor support
2019-01-15 0:18 ` Denis CIOCCA
@ 2019-01-19 17:33 ` Jonathan Cameron
2019-01-19 17:40 ` Jonathan Cameron
0 siblings, 1 reply; 4+ messages in thread
From: Jonathan Cameron @ 2019-01-19 17:33 UTC (permalink / raw)
To: Denis CIOCCA; +Cc: Mario Tesi, linux-iio, linux-kernel, Mario TESI
On Tue, 15 Jan 2019 00:18:58 +0000
Denis CIOCCA <denis.ciocca@st.com> wrote:
> Acked-by: Denis Ciocca <denis.ciocca@st.com>
>
>
> -----Original Message-----
> From: linux-iio-owner@vger.kernel.org <linux-iio-owner@vger.kernel.org> On Behalf Of Mario Tesi
> Sent: Monday, January 14, 2019 9:24 AM
> To: jic23@kernel.org
> Cc: linux-iio@vger.kernel.org; linux-kernel@vger.kernel.org; Mario TESI <mario.tesi@st.com>
> Subject: [PATCH] iio:st_pressure:initial lps22hh sensor support
>
> From: mario tesi <mario.tesi@st.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@st.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
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] iio:st_pressure:initial lps22hh sensor support
2019-01-19 17:33 ` Jonathan Cameron
@ 2019-01-19 17:40 ` Jonathan Cameron
0 siblings, 0 replies; 4+ messages in thread
From: Jonathan Cameron @ 2019-01-19 17:40 UTC (permalink / raw)
To: Denis CIOCCA; +Cc: Mario Tesi, linux-iio, linux-kernel, Mario TESI
On Sat, 19 Jan 2019 17:33:30 +0000
Jonathan Cameron <jic23@kernel.org> wrote:
> On Tue, 15 Jan 2019 00:18:58 +0000
> Denis CIOCCA <denis.ciocca@st.com> wrote:
>
> > Acked-by: Denis Ciocca <denis.ciocca@st.com>
Denis,
One random question inline. Not about this patch as such...
> >
> >
> > -----Original Message-----
> > From: linux-iio-owner@vger.kernel.org <linux-iio-owner@vger.kernel.org> On Behalf Of Mario Tesi
> > Sent: Monday, January 14, 2019 9:24 AM
> > To: jic23@kernel.org
> > Cc: linux-iio@vger.kernel.org; linux-kernel@vger.kernel.org; Mario TESI <mario.tesi@st.com>
> > Subject: [PATCH] iio:st_pressure:initial lps22hh sensor support
> >
> > From: mario tesi <mario.tesi@st.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@st.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,
Why is ch not
const stuct iio_chan_spec *
in the structure definition?
I tried it to see if it would throw complaints at me and it didn't
rather implying the code never assumes it can change the content of
channels via this pointer.
It dates right back to the start of this driver I think. If we
make it const, we can get rid of all the casts in locations like this
which would be a nice tidy up. Never good to cast away a const
without very very good reasons!
Jonathan
> > + .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
> >
>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-01-19 17:40 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-01-14 17:24 [PATCH] iio:st_pressure:initial lps22hh sensor support Mario Tesi
2019-01-15 0:18 ` Denis CIOCCA
2019-01-19 17:33 ` Jonathan Cameron
2019-01-19 17:40 ` Jonathan Cameron
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).