* [PATCH 0/3] iio: Silan SC7A20 accelerometer support
@ 2022-01-30 3:44 Samuel Holland
2022-01-30 3:44 ` [PATCH 1/3] dt-bindings: vendor-prefixes: Add silan vendor prefix Samuel Holland
` (2 more replies)
0 siblings, 3 replies; 7+ messages in thread
From: Samuel Holland @ 2022-01-30 3:44 UTC (permalink / raw)
To: Jonathan Cameron, Lars-Peter Clausen, Denis Ciocca, Linus Walleij,
Rob Herring, devicetree
Cc: linux-kernel, linux-iio, Samuel Holland
This chip is a clone of an ST part, so it works without any issues when
added to the existing driver.
Samuel Holland (3):
dt-bindings: vendor-prefixes: Add silan vendor prefix
dt-bindings: iio: st: Add Silan SC7A20 accelerometer
iio: accel: st_accel: Add support for Silan SC7A20
.../bindings/iio/st,st-sensors.yaml | 1 +
.../devicetree/bindings/vendor-prefixes.yaml | 2 +
drivers/iio/accel/st_accel.h | 2 +
drivers/iio/accel/st_accel_core.c | 79 +++++++++++++++++++
drivers/iio/accel/st_accel_i2c.c | 5 ++
5 files changed, 89 insertions(+)
--
2.33.1
^ permalink raw reply [flat|nested] 7+ messages in thread* [PATCH 1/3] dt-bindings: vendor-prefixes: Add silan vendor prefix 2022-01-30 3:44 [PATCH 0/3] iio: Silan SC7A20 accelerometer support Samuel Holland @ 2022-01-30 3:44 ` Samuel Holland 2022-01-30 3:44 ` [PATCH 2/3] dt-bindings: iio: st: Add Silan SC7A20 accelerometer Samuel Holland 2022-01-30 3:44 ` [PATCH 3/3] iio: accel: st_accel: Add support for Silan SC7A20 Samuel Holland 2 siblings, 0 replies; 7+ messages in thread From: Samuel Holland @ 2022-01-30 3:44 UTC (permalink / raw) To: Jonathan Cameron, Lars-Peter Clausen, Denis Ciocca, Linus Walleij, Rob Herring, devicetree Cc: linux-kernel, linux-iio, Samuel Holland Hangzhou Silan Microelectronics Co., Ltd. (http://www.silan.com.cn/) is a manufacturer of ICs, including MEMS sensors. Signed-off-by: Samuel Holland <samuel@sholland.org> --- Documentation/devicetree/bindings/vendor-prefixes.yaml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml b/Documentation/devicetree/bindings/vendor-prefixes.yaml index 294093d45a23..b923be2c8172 100644 --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml @@ -1082,6 +1082,8 @@ patternProperties: description: Silicon Image "^silabs,.*": description: Silicon Laboratories + "^silan,.*": + description: Hangzhou Silan Microelectronics Co., Ltd. "^silead,.*": description: Silead Inc. "^silergy,.*": -- 2.33.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 2/3] dt-bindings: iio: st: Add Silan SC7A20 accelerometer 2022-01-30 3:44 [PATCH 0/3] iio: Silan SC7A20 accelerometer support Samuel Holland 2022-01-30 3:44 ` [PATCH 1/3] dt-bindings: vendor-prefixes: Add silan vendor prefix Samuel Holland @ 2022-01-30 3:44 ` Samuel Holland 2022-01-30 14:12 ` Jonathan Cameron 2022-01-30 3:44 ` [PATCH 3/3] iio: accel: st_accel: Add support for Silan SC7A20 Samuel Holland 2 siblings, 1 reply; 7+ messages in thread From: Samuel Holland @ 2022-01-30 3:44 UTC (permalink / raw) To: Jonathan Cameron, Lars-Peter Clausen, Denis Ciocca, Linus Walleij, Rob Herring, devicetree Cc: linux-kernel, linux-iio, Samuel Holland This chip appears to be a clone of the LIS2DH, as the register bit definitions match exactly. Signed-off-by: Samuel Holland <samuel@sholland.org> --- Documentation/devicetree/bindings/iio/st,st-sensors.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/iio/st,st-sensors.yaml b/Documentation/devicetree/bindings/iio/st,st-sensors.yaml index 71de5631ebae..70f755041d8a 100644 --- a/Documentation/devicetree/bindings/iio/st,st-sensors.yaml +++ b/Documentation/devicetree/bindings/iio/st,st-sensors.yaml @@ -24,6 +24,7 @@ properties: oneOf: - description: STMicroelectronics Accelerometers enum: + - silan,sc7a20 - st,h3lis331dl-accel - st,lis2de12 - st,lis2dw12 -- 2.33.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 2/3] dt-bindings: iio: st: Add Silan SC7A20 accelerometer 2022-01-30 3:44 ` [PATCH 2/3] dt-bindings: iio: st: Add Silan SC7A20 accelerometer Samuel Holland @ 2022-01-30 14:12 ` Jonathan Cameron 0 siblings, 0 replies; 7+ messages in thread From: Jonathan Cameron @ 2022-01-30 14:12 UTC (permalink / raw) To: Samuel Holland Cc: Lars-Peter Clausen, Denis Ciocca, Linus Walleij, Rob Herring, devicetree, linux-kernel, linux-iio On Sat, 29 Jan 2022 21:44:40 -0600 Samuel Holland <samuel@sholland.org> wrote: > This chip appears to be a clone of the LIS2DH, as the register bit > definitions match exactly. > > Signed-off-by: Samuel Holland <samuel@sholland.org> > --- > > Documentation/devicetree/bindings/iio/st,st-sensors.yaml | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Documentation/devicetree/bindings/iio/st,st-sensors.yaml b/Documentation/devicetree/bindings/iio/st,st-sensors.yaml > index 71de5631ebae..70f755041d8a 100644 > --- a/Documentation/devicetree/bindings/iio/st,st-sensors.yaml > +++ b/Documentation/devicetree/bindings/iio/st,st-sensors.yaml > @@ -24,6 +24,7 @@ properties: > oneOf: > - description: STMicroelectronics Accelerometers It's not an STMicroelectronics part, so add a new block for silan ones. Thanks, Jonathan > enum: > + - silan,sc7a20 > - st,h3lis331dl-accel > - st,lis2de12 > - st,lis2dw12 ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 3/3] iio: accel: st_accel: Add support for Silan SC7A20 2022-01-30 3:44 [PATCH 0/3] iio: Silan SC7A20 accelerometer support Samuel Holland 2022-01-30 3:44 ` [PATCH 1/3] dt-bindings: vendor-prefixes: Add silan vendor prefix Samuel Holland 2022-01-30 3:44 ` [PATCH 2/3] dt-bindings: iio: st: Add Silan SC7A20 accelerometer Samuel Holland @ 2022-01-30 3:44 ` Samuel Holland 2022-01-30 14:10 ` Jonathan Cameron 2 siblings, 1 reply; 7+ messages in thread From: Samuel Holland @ 2022-01-30 3:44 UTC (permalink / raw) To: Jonathan Cameron, Lars-Peter Clausen, Denis Ciocca, Linus Walleij, Rob Herring, devicetree Cc: linux-kernel, linux-iio, Samuel Holland This chip appears to be a clone of the LIS2DH. The new description is a copy of the LIS2DH's description with a different WAI value. Signed-off-by: Samuel Holland <samuel@sholland.org> --- drivers/iio/accel/st_accel.h | 2 + drivers/iio/accel/st_accel_core.c | 79 +++++++++++++++++++++++++++++++ drivers/iio/accel/st_accel_i2c.c | 5 ++ 3 files changed, 86 insertions(+) diff --git a/drivers/iio/accel/st_accel.h b/drivers/iio/accel/st_accel.h index 8750dea56fcb..00e056c21bfc 100644 --- a/drivers/iio/accel/st_accel.h +++ b/drivers/iio/accel/st_accel.h @@ -36,6 +36,7 @@ enum st_accel_type { LIS3DHH, LIS2DE12, LIS2HH12, + SC7A20, ST_ACCEL_MAX, }; @@ -61,6 +62,7 @@ enum st_accel_type { #define LIS3DE_ACCEL_DEV_NAME "lis3de" #define LIS2DE12_ACCEL_DEV_NAME "lis2de12" #define LIS2HH12_ACCEL_DEV_NAME "lis2hh12" +#define SC7A20_ACCEL_DEV_NAME "sc7a20" #ifdef CONFIG_IIO_BUFFER int st_accel_allocate_ring(struct iio_dev *indio_dev); diff --git a/drivers/iio/accel/st_accel_core.c b/drivers/iio/accel/st_accel_core.c index 31ea19d0ba71..d9aa0ff1922b 100644 --- a/drivers/iio/accel/st_accel_core.c +++ b/drivers/iio/accel/st_accel_core.c @@ -1087,6 +1087,85 @@ static const struct st_sensor_settings st_accel_sensors_settings[] = { .multi_read_bit = true, .bootime = 2, }, + { + .wai = 0x11, + .wai_addr = ST_SENSORS_DEFAULT_WAI_ADDRESS, + .sensors_supported = { + [0] = SC7A20_ACCEL_DEV_NAME, + }, + .ch = (struct iio_chan_spec *)st_accel_12bit_channels, + .odr = { + .addr = 0x20, + .mask = 0xf0, + .odr_avl = { + { .hz = 1, .value = 0x01, }, + { .hz = 10, .value = 0x02, }, + { .hz = 25, .value = 0x03, }, + { .hz = 50, .value = 0x04, }, + { .hz = 100, .value = 0x05, }, + { .hz = 200, .value = 0x06, }, + { .hz = 400, .value = 0x07, }, + { .hz = 1600, .value = 0x08, }, + }, + }, + .pw = { + .addr = 0x20, + .mask = 0xf0, + .value_off = ST_SENSORS_DEFAULT_POWER_OFF_VALUE, + }, + .enable_axis = { + .addr = ST_SENSORS_DEFAULT_AXIS_ADDR, + .mask = ST_SENSORS_DEFAULT_AXIS_MASK, + }, + .fs = { + .addr = 0x23, + .mask = 0x30, + .fs_avl = { + [0] = { + .num = ST_ACCEL_FS_AVL_2G, + .value = 0x00, + .gain = IIO_G_TO_M_S_2(1000), + }, + [1] = { + .num = ST_ACCEL_FS_AVL_4G, + .value = 0x01, + .gain = IIO_G_TO_M_S_2(2000), + }, + [2] = { + .num = ST_ACCEL_FS_AVL_8G, + .value = 0x02, + .gain = IIO_G_TO_M_S_2(4000), + }, + [3] = { + .num = ST_ACCEL_FS_AVL_16G, + .value = 0x03, + .gain = IIO_G_TO_M_S_2(12000), + }, + }, + }, + .bdu = { + .addr = 0x23, + .mask = 0x80, + }, + .drdy_irq = { + .int1 = { + .addr = 0x22, + .mask = 0x10, + }, + .addr_ihl = 0x25, + .mask_ihl = 0x02, + .stat_drdy = { + .addr = ST_SENSORS_DEFAULT_STAT_ADDR, + .mask = 0x07, + }, + }, + .sim = { + .addr = 0x23, + .value = BIT(0), + }, + .multi_read_bit = true, + .bootime = 2, + }, }; /* Default accel DRDY is available on INT1 pin */ diff --git a/drivers/iio/accel/st_accel_i2c.c b/drivers/iio/accel/st_accel_i2c.c index c0ce78eebad9..7f5888570e87 100644 --- a/drivers/iio/accel/st_accel_i2c.c +++ b/drivers/iio/accel/st_accel_i2c.c @@ -107,6 +107,10 @@ static const struct of_device_id st_accel_of_match[] = { .compatible = "st,lis2hh12", .data = LIS2HH12_ACCEL_DEV_NAME, }, + { + .compatible = "silan,sc7a20", + .data = SC7A20_ACCEL_DEV_NAME, + }, {}, }; MODULE_DEVICE_TABLE(of, st_accel_of_match); @@ -142,6 +146,7 @@ static const struct i2c_device_id st_accel_id_table[] = { { LIS3DE_ACCEL_DEV_NAME }, { LIS2DE12_ACCEL_DEV_NAME }, { LIS2HH12_ACCEL_DEV_NAME }, + { SC7A20_ACCEL_DEV_NAME }, {}, }; MODULE_DEVICE_TABLE(i2c, st_accel_id_table); -- 2.33.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 3/3] iio: accel: st_accel: Add support for Silan SC7A20 2022-01-30 3:44 ` [PATCH 3/3] iio: accel: st_accel: Add support for Silan SC7A20 Samuel Holland @ 2022-01-30 14:10 ` Jonathan Cameron 2022-01-31 11:23 ` Linus Walleij 0 siblings, 1 reply; 7+ messages in thread From: Jonathan Cameron @ 2022-01-30 14:10 UTC (permalink / raw) To: Samuel Holland Cc: Lars-Peter Clausen, Denis Ciocca, Linus Walleij, Rob Herring, devicetree, linux-kernel, linux-iio On Sat, 29 Jan 2022 21:44:41 -0600 Samuel Holland <samuel@sholland.org> wrote: > This chip appears to be a clone of the LIS2DH. The new description is a > copy of the LIS2DH's description with a different WAI value. > > Signed-off-by: Samuel Holland <samuel@sholland.org> Hi Samuel, This is nasty as 0x11 is a valid who am I for an LIS3DHH which is a very different device. One request inline for a bit more info in the code about what this device is etc. I've found a Chinese data sheet on one of the datasheet aggregator sites. If you know of a stable location to add a Datasheet: tag to this patch that would also be helpful. Thanks, Jonathan > --- > > drivers/iio/accel/st_accel.h | 2 + > drivers/iio/accel/st_accel_core.c | 79 +++++++++++++++++++++++++++++++ > drivers/iio/accel/st_accel_i2c.c | 5 ++ > 3 files changed, 86 insertions(+) > > diff --git a/drivers/iio/accel/st_accel.h b/drivers/iio/accel/st_accel.h > index 8750dea56fcb..00e056c21bfc 100644 > --- a/drivers/iio/accel/st_accel.h > +++ b/drivers/iio/accel/st_accel.h > @@ -36,6 +36,7 @@ enum st_accel_type { > LIS3DHH, > LIS2DE12, > LIS2HH12, > + SC7A20, > ST_ACCEL_MAX, > }; > > @@ -61,6 +62,7 @@ enum st_accel_type { > #define LIS3DE_ACCEL_DEV_NAME "lis3de" > #define LIS2DE12_ACCEL_DEV_NAME "lis2de12" > #define LIS2HH12_ACCEL_DEV_NAME "lis2hh12" > +#define SC7A20_ACCEL_DEV_NAME "sc7a20" > > #ifdef CONFIG_IIO_BUFFER > int st_accel_allocate_ring(struct iio_dev *indio_dev); > diff --git a/drivers/iio/accel/st_accel_core.c b/drivers/iio/accel/st_accel_core.c > index 31ea19d0ba71..d9aa0ff1922b 100644 > --- a/drivers/iio/accel/st_accel_core.c > +++ b/drivers/iio/accel/st_accel_core.c > @@ -1087,6 +1087,85 @@ static const struct st_sensor_settings st_accel_sensors_settings[] = { > .multi_read_bit = true, > .bootime = 2, > }, > + { Please add a comment here about the fact it seems to be a clone of the LIS2DH despite the wrong WAI. It's worth noting that a good part of the maintenance of this driver is done by ST employees who are probably less than happy with seeing a clone (with bugs) supported. So support is likely to be somewhat best effort / not going to deliberately break support for this part but no means to test it. Hopefully we'll remember to cc you to test anything new added to the driver. Clones aren't always perfect! Jonathan > + .wai = 0x11, > + .wai_addr = ST_SENSORS_DEFAULT_WAI_ADDRESS, > + .sensors_supported = { > + [0] = SC7A20_ACCEL_DEV_NAME, > + }, > + .ch = (struct iio_chan_spec *)st_accel_12bit_channels, > + .odr = { > + .addr = 0x20, > + .mask = 0xf0, > + .odr_avl = { > + { .hz = 1, .value = 0x01, }, > + { .hz = 10, .value = 0x02, }, > + { .hz = 25, .value = 0x03, }, > + { .hz = 50, .value = 0x04, }, > + { .hz = 100, .value = 0x05, }, > + { .hz = 200, .value = 0x06, }, > + { .hz = 400, .value = 0x07, }, > + { .hz = 1600, .value = 0x08, }, > + }, > + }, > + .pw = { > + .addr = 0x20, > + .mask = 0xf0, > + .value_off = ST_SENSORS_DEFAULT_POWER_OFF_VALUE, > + }, > + .enable_axis = { > + .addr = ST_SENSORS_DEFAULT_AXIS_ADDR, > + .mask = ST_SENSORS_DEFAULT_AXIS_MASK, > + }, > + .fs = { > + .addr = 0x23, > + .mask = 0x30, > + .fs_avl = { > + [0] = { > + .num = ST_ACCEL_FS_AVL_2G, > + .value = 0x00, > + .gain = IIO_G_TO_M_S_2(1000), > + }, > + [1] = { > + .num = ST_ACCEL_FS_AVL_4G, > + .value = 0x01, > + .gain = IIO_G_TO_M_S_2(2000), > + }, > + [2] = { > + .num = ST_ACCEL_FS_AVL_8G, > + .value = 0x02, > + .gain = IIO_G_TO_M_S_2(4000), > + }, > + [3] = { > + .num = ST_ACCEL_FS_AVL_16G, > + .value = 0x03, > + .gain = IIO_G_TO_M_S_2(12000), > + }, > + }, > + }, > + .bdu = { > + .addr = 0x23, > + .mask = 0x80, > + }, > + .drdy_irq = { > + .int1 = { > + .addr = 0x22, > + .mask = 0x10, > + }, > + .addr_ihl = 0x25, > + .mask_ihl = 0x02, > + .stat_drdy = { > + .addr = ST_SENSORS_DEFAULT_STAT_ADDR, > + .mask = 0x07, > + }, > + }, > + .sim = { > + .addr = 0x23, > + .value = BIT(0), > + }, > + .multi_read_bit = true, > + .bootime = 2, > + }, > }; > > /* Default accel DRDY is available on INT1 pin */ > diff --git a/drivers/iio/accel/st_accel_i2c.c b/drivers/iio/accel/st_accel_i2c.c > index c0ce78eebad9..7f5888570e87 100644 > --- a/drivers/iio/accel/st_accel_i2c.c > +++ b/drivers/iio/accel/st_accel_i2c.c > @@ -107,6 +107,10 @@ static const struct of_device_id st_accel_of_match[] = { > .compatible = "st,lis2hh12", > .data = LIS2HH12_ACCEL_DEV_NAME, > }, > + { > + .compatible = "silan,sc7a20", > + .data = SC7A20_ACCEL_DEV_NAME, > + }, > {}, > }; > MODULE_DEVICE_TABLE(of, st_accel_of_match); > @@ -142,6 +146,7 @@ static const struct i2c_device_id st_accel_id_table[] = { > { LIS3DE_ACCEL_DEV_NAME }, > { LIS2DE12_ACCEL_DEV_NAME }, > { LIS2HH12_ACCEL_DEV_NAME }, > + { SC7A20_ACCEL_DEV_NAME }, > {}, > }; > MODULE_DEVICE_TABLE(i2c, st_accel_id_table); ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 3/3] iio: accel: st_accel: Add support for Silan SC7A20 2022-01-30 14:10 ` Jonathan Cameron @ 2022-01-31 11:23 ` Linus Walleij 0 siblings, 0 replies; 7+ messages in thread From: Linus Walleij @ 2022-01-31 11:23 UTC (permalink / raw) To: Jonathan Cameron Cc: Samuel Holland, Lars-Peter Clausen, Denis Ciocca, Rob Herring, devicetree, linux-kernel, linux-iio On Sun, Jan 30, 2022 at 3:04 PM Jonathan Cameron <jic23@kernel.org> wrote: > This is nasty as 0x11 is a valid who am I for an LIS3DHH which is a very > different device. The WAI value is just a second line check anyway, the device is probed from the compatible, this is just checked to make sure the device is what it says it is. Annoying, but it will do the job. > It's worth noting that a good part of the maintenance of this driver is > done by ST employees who are probably less than happy with seeing > a clone (with bugs) supported. So support is likely to be somewhat best > effort / not going to deliberately break support for this part but no > means to test it. Hopefully we'll remember to cc you to test anything new added to > the driver. Clones aren't always perfect! Linux began with people running it on IBM PC clones... ;) I would assume good faith, especially with the silicon shortage plaguing us it is perfectly possible that this is a fully licensed copy. I think the two derivatives of the AK8974 magnetometer for example are legally transferred IP, we just don't know the details behind the curtain. Yours, Linus Walleij ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2022-01-31 11:36 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2022-01-30 3:44 [PATCH 0/3] iio: Silan SC7A20 accelerometer support Samuel Holland 2022-01-30 3:44 ` [PATCH 1/3] dt-bindings: vendor-prefixes: Add silan vendor prefix Samuel Holland 2022-01-30 3:44 ` [PATCH 2/3] dt-bindings: iio: st: Add Silan SC7A20 accelerometer Samuel Holland 2022-01-30 14:12 ` Jonathan Cameron 2022-01-30 3:44 ` [PATCH 3/3] iio: accel: st_accel: Add support for Silan SC7A20 Samuel Holland 2022-01-30 14:10 ` Jonathan Cameron 2022-01-31 11:23 ` Linus Walleij
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).