From: mario tesi <[email protected]>
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 <[email protected]>
---
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
Acked-by: Denis Ciocca <[email protected]>
-----Original Message-----
From: [email protected] <[email protected]> On Behalf Of Mario Tesi
Sent: Monday, January 14, 2019 9:24 AM
To: [email protected]
Cc: [email protected]; [email protected]; Mario TESI <[email protected]>
Subject: [PATCH] iio:st_pressure:initial lps22hh sensor support
From: mario tesi <[email protected]>
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 <[email protected]>
---
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
On Tue, 15 Jan 2019 00:18:58 +0000
Denis CIOCCA <[email protected]> wrote:
> Acked-by: Denis Ciocca <[email protected]>
>
>
> -----Original Message-----
> From: [email protected] <[email protected]> On Behalf Of Mario Tesi
> Sent: Monday, January 14, 2019 9:24 AM
> To: [email protected]
> Cc: [email protected]; [email protected]; Mario TESI <[email protected]>
> Subject: [PATCH] iio:st_pressure:initial lps22hh sensor support
>
> From: mario tesi <[email protected]>
>
> 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 <[email protected]>
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
>
On Sat, 19 Jan 2019 17:33:30 +0000
Jonathan Cameron <[email protected]> wrote:
> On Tue, 15 Jan 2019 00:18:58 +0000
> Denis CIOCCA <[email protected]> wrote:
>
> > Acked-by: Denis Ciocca <[email protected]>
Denis,
One random question inline. Not about this patch as such...
> >
> >
> > -----Original Message-----
> > From: [email protected] <[email protected]> On Behalf Of Mario Tesi
> > Sent: Monday, January 14, 2019 9:24 AM
> > To: [email protected]
> > Cc: [email protected]; [email protected]; Mario TESI <[email protected]>
> > Subject: [PATCH] iio:st_pressure:initial lps22hh sensor support
> >
> > From: mario tesi <[email protected]>
> >
> > 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 <[email protected]>
> 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
> >
>