Re: [PATCH V2 5/8] iio: accel: add support for IIS328DQ variant

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Jonathan,

Am 18.03.23 um 16:58 schrieb Jonathan Cameron:
On Fri, 17 Mar 2023 11:33:20 +0100
Stefan Wahren <stefan.wahren@xxxxxxxxxxxxxx> wrote:

Since the ST IIS328DQ accelerometer is compatible to the ST LIS331DL,
just add the new compatible to the st_accel framework.

Link: https://www.st.com/resource/en/datasheet/iis328dq.pdf
Signed-off-by: Stefan Wahren <stefan.wahren@xxxxxxxxxxxxxx>
Hi Stefan,

Assuming the DT binding is fine with the DT maintainers, I plan to
pick the pair of IIO fixes up through my tree.  Is there any strong
reason they should go via another route?

i'm a little bit confused about consider this as "fixes", but i'm fine with them going your tree.

Best regards


It's common for the dts files to go via ARM soc in cases like this
and we tend not to worry that they are 'broken' until the binding docs
are visible via other trees. Other than that I can't see an obvious
reason not to split these up.

Thanks,

Jonathan

---
  drivers/iio/accel/st_accel.h      | 1 +
  drivers/iio/accel/st_accel_core.c | 1 +
  drivers/iio/accel/st_accel_i2c.c  | 5 +++++
  drivers/iio/accel/st_accel_spi.c  | 5 +++++
  4 files changed, 12 insertions(+)

diff --git a/drivers/iio/accel/st_accel.h b/drivers/iio/accel/st_accel.h
index 56ed0c776d4a..e7525615712b 100644
--- a/drivers/iio/accel/st_accel.h
+++ b/drivers/iio/accel/st_accel.h
@@ -39,6 +39,7 @@
  #define LIS302DL_ACCEL_DEV_NAME		"lis302dl"
  #define LSM303C_ACCEL_DEV_NAME		"lsm303c_accel"
  #define SC7A20_ACCEL_DEV_NAME		"sc7a20"
+#define IIS328DQ_ACCEL_DEV_NAME		"iis328dq"
#ifdef CONFIG_IIO_BUFFER
diff --git a/drivers/iio/accel/st_accel_core.c b/drivers/iio/accel/st_accel_core.c
index 6b8562f684d5..5f7d81b44b1d 100644
--- a/drivers/iio/accel/st_accel_core.c
+++ b/drivers/iio/accel/st_accel_core.c
@@ -517,6 +517,7 @@ static const struct st_sensor_settings st_accel_sensors_settings[] = {
  		.wai_addr = ST_SENSORS_DEFAULT_WAI_ADDRESS,
  		.sensors_supported = {
  			[0] = H3LIS331DL_ACCEL_DEV_NAME,
+			[1] = IIS328DQ_ACCEL_DEV_NAME,
  		},
  		.ch = (struct iio_chan_spec *)st_accel_12bit_channels,
  		.odr = {
diff --git a/drivers/iio/accel/st_accel_i2c.c b/drivers/iio/accel/st_accel_i2c.c
index 3f02fd5d5946..fb9e2d6f4210 100644
--- a/drivers/iio/accel/st_accel_i2c.c
+++ b/drivers/iio/accel/st_accel_i2c.c
@@ -119,6 +119,10 @@ static const struct of_device_id st_accel_of_match[] = {
  		.compatible = "silan,sc7a20",
  		.data = SC7A20_ACCEL_DEV_NAME,
  	},
+	{
+		.compatible = "st,iis328dq",
+		.data = IIS328DQ_ACCEL_DEV_NAME,
+	},
  	{},
  };
  MODULE_DEVICE_TABLE(of, st_accel_of_match);
@@ -157,6 +161,7 @@ static const struct i2c_device_id st_accel_id_table[] = {
  	{ LIS302DL_ACCEL_DEV_NAME },
  	{ LSM303C_ACCEL_DEV_NAME },
  	{ SC7A20_ACCEL_DEV_NAME },
+	{ IIS328DQ_ACCEL_DEV_NAME },
  	{},
  };
  MODULE_DEVICE_TABLE(i2c, st_accel_id_table);
diff --git a/drivers/iio/accel/st_accel_spi.c b/drivers/iio/accel/st_accel_spi.c
index 5740dc1820bd..f72a24f45322 100644
--- a/drivers/iio/accel/st_accel_spi.c
+++ b/drivers/iio/accel/st_accel_spi.c
@@ -100,6 +100,10 @@ static const struct of_device_id st_accel_of_match[] = {
  		.compatible = "st,lsm303c-accel",
  		.data = LSM303C_ACCEL_DEV_NAME,
  	},
+	{
+		.compatible = "st,iis328dq",
+		.data = IIS328DQ_ACCEL_DEV_NAME,
+	},
  	{}
  };
  MODULE_DEVICE_TABLE(of, st_accel_of_match);
@@ -157,6 +161,7 @@ static const struct spi_device_id st_accel_id_table[] = {
  	{ LIS3DE_ACCEL_DEV_NAME },
  	{ LIS302DL_ACCEL_DEV_NAME },
  	{ LSM303C_ACCEL_DEV_NAME },
+	{ IIS328DQ_ACCEL_DEV_NAME },
  	{},
  };
  MODULE_DEVICE_TABLE(spi, st_accel_id_table);

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux