[PATCH v2 0/4] iio: accel: fxls8962af: add fxls8967af and fxls8974cf

[PATCH v2 0/4] iio: accel: fxls8962af: add fxls8967af and fxls8974cf

[Frank Li]

fxls8967af and fxls8974cf are similar with fxls8962af, only change ID.

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
Changes in v2:
- make nxp,fxls8967af and nxp,fxls8974cf fallback to nxp,fxls8962af.
- Link to v1: https://lore.kernel.org/r/20241113-fxls-v1-0-5e48ff1b1fb8@nxp.com

---
Haibo Chen (1):
      iio: accel: fxls8962af: add fxls8974cf support

Han Xu (3):
      dt-bindings: iio: accel: fxls8962af: add compatible string 'nxp,fxls8967af'
      dt-bindings: iio: accel: fxls8962af: add compatible string 'nxp,fxls8974cf'
      iio: accel: fxls8962af: add fxls8967af support

 .../devicetree/bindings/iio/accel/nxp,fxls8962af.yaml      | 14 +++++++++++---
 drivers/iio/accel/fxls8962af-core.c                        | 14 ++++++++++++++
 drivers/iio/accel/fxls8962af-i2c.c                         |  2 ++
 drivers/iio/accel/fxls8962af.h                             |  2 ++
 4 files changed, 29 insertions(+), 3 deletions(-)
---
base-commit: bd05b9a700c10473c2f52bf12c5c5938c30e80b0
change-id: 20241113-fxls-d93a888889f7

Best regards,
---
Frank Li <Frank.Li@nxp.com>

[PATCH v2 1/4] dt-bindings: iio: accel: fxls8962af: add compatible string 'nxp,fxls8967af'

[Frank Li]

From: Han Xu <han.xu@nxp.com>

Add 'nxp,fxls8967af' compatible for the FXLS8967AF sensor, falling back to
'nxp,fxls8962af' as the only difference is the ID.

Signed-off-by: Han Xu <han.xu@nxp.com>
Reviewed-by: Sean Nyekjaer <sean@geanix.com>
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 .../devicetree/bindings/iio/accel/nxp,fxls8962af.yaml        | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/Documentation/devicetree/bindings/iio/accel/nxp,fxls8962af.yaml b/Documentation/devicetree/bindings/iio/accel/nxp,fxls8962af.yaml
index 783c7ddfcd90a..a5c882bd7d06e 100644
--- a/Documentation/devicetree/bindings/iio/accel/nxp,fxls8962af.yaml
+++ b/Documentation/devicetree/bindings/iio/accel/nxp,fxls8962af.yaml
@@ -14,12 +14,18 @@ description: |
   SPI and I2C interface.
     https://www.nxp.com/docs/en/data-sheet/FXLS8962AF.pdf
     https://www.nxp.com/docs/en/data-sheet/FXLS8964AF.pdf
+    https://www.nxp.com/docs/en/data-sheet/FXLS8967AF.pdf
 
 properties:
   compatible:
-    enum:
-      - nxp,fxls8962af
-      - nxp,fxls8964af
+    oneOf:
+      - enum:
+          - nxp,fxls8962af
+          - nxp,fxls8964af
+      - items:
+          - enum:
+              - nxp,fxls8967af
+          - const: nxp,fxls8962af
 
   reg:
     maxItems: 1

-- 
2.34.1

[PATCH v2 2/4] dt-bindings: iio: accel: fxls8962af: add compatible string 'nxp,fxls8974cf'

[Frank Li]

From: Han Xu <han.xu@nxp.com>

Add 'nxp,fxls8974cf' compatible for the FXLS8974CF sensor, falling back to
'nxp,fxls8962af' as the only difference is the ID.

Signed-off-by: Han Xu <han.xu@nxp.com>
Reviewed-by: Sean Nyekjaer <sean@geanix.com>
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 Documentation/devicetree/bindings/iio/accel/nxp,fxls8962af.yaml | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Documentation/devicetree/bindings/iio/accel/nxp,fxls8962af.yaml b/Documentation/devicetree/bindings/iio/accel/nxp,fxls8962af.yaml
index a5c882bd7d06e..2d99e3811da07 100644
--- a/Documentation/devicetree/bindings/iio/accel/nxp,fxls8962af.yaml
+++ b/Documentation/devicetree/bindings/iio/accel/nxp,fxls8962af.yaml
@@ -15,6 +15,7 @@ description: |
     https://www.nxp.com/docs/en/data-sheet/FXLS8962AF.pdf
     https://www.nxp.com/docs/en/data-sheet/FXLS8964AF.pdf
     https://www.nxp.com/docs/en/data-sheet/FXLS8967AF.pdf
+    https://www.nxp.com/docs/en/data-sheet/FXLS8974CF.pdf
 
 properties:
   compatible:
@@ -25,6 +26,7 @@ properties:
       - items:
           - enum:
               - nxp,fxls8967af
+              - nxp,fxls8974cf
           - const: nxp,fxls8962af
 
   reg:

-- 
2.34.1

[PATCH v2 3/4] iio: accel: fxls8962af: add fxls8974cf support

[Frank Li]

From: Haibo Chen <haibo.chen@nxp.com>

fxls8974cf is similar with fxls8962af, the only difference is the device id
change to 0x86.

Signed-off-by: Haibo Chen <haibo.chen@nxp.com>
Reviewed-by: Clark Wang <xiaoning.wang@nxp.com>
Reviewed-by: Sean Nyekjaer <sean@geanix.com>
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 drivers/iio/accel/fxls8962af-core.c | 7 +++++++
 drivers/iio/accel/fxls8962af-i2c.c  | 1 +
 drivers/iio/accel/fxls8962af.h      | 1 +
 3 files changed, 9 insertions(+)

diff --git a/drivers/iio/accel/fxls8962af-core.c b/drivers/iio/accel/fxls8962af-core.c
index f07fba17048e7..b5607e753a7db 100644
--- a/drivers/iio/accel/fxls8962af-core.c
+++ b/drivers/iio/accel/fxls8962af-core.c
@@ -129,6 +129,7 @@
 
 #define FXLS8962AF_DEVICE_ID			0x62
 #define FXLS8964AF_DEVICE_ID			0x84
+#define FXLS8974CF_DEVICE_ID			0x86
 
 /* Raw temp channel offset */
 #define FXLS8962AF_TEMP_CENTER_VAL		25
@@ -766,6 +767,12 @@ static const struct fxls8962af_chip_info fxls_chip_info_table[] = {
 		.channels = fxls8962af_channels,
 		.num_channels = ARRAY_SIZE(fxls8962af_channels),
 	},
+	[fxls8974cf] = {
+		.chip_id = FXLS8974CF_DEVICE_ID,
+		.name = "fxls8974cf",
+		.channels = fxls8962af_channels,
+		.num_channels = ARRAY_SIZE(fxls8962af_channels),
+	},
 };
 
 static const struct iio_info fxls8962af_info = {
diff --git a/drivers/iio/accel/fxls8962af-i2c.c b/drivers/iio/accel/fxls8962af-i2c.c
index 1601246733083..687eb49035111 100644
--- a/drivers/iio/accel/fxls8962af-i2c.c
+++ b/drivers/iio/accel/fxls8962af-i2c.c
@@ -30,6 +30,7 @@ static int fxls8962af_probe(struct i2c_client *client)
 static const struct i2c_device_id fxls8962af_id[] = {
 	{ "fxls8962af", fxls8962af },
 	{ "fxls8964af", fxls8964af },
+	{ "fxls8974cf", fxls8974cf },
 	{}
 };
 MODULE_DEVICE_TABLE(i2c, fxls8962af_id);
diff --git a/drivers/iio/accel/fxls8962af.h b/drivers/iio/accel/fxls8962af.h
index 6eaa2803b26f0..733b69e01e1cc 100644
--- a/drivers/iio/accel/fxls8962af.h
+++ b/drivers/iio/accel/fxls8962af.h
@@ -11,6 +11,7 @@ struct device;
 enum {
 	fxls8962af,
 	fxls8964af,
+	fxls8974cf,
 };
 
 int fxls8962af_core_probe(struct device *dev, struct regmap *regmap, int irq);

-- 
2.34.1

[PATCH v2 4/4] iio: accel: fxls8962af: add fxls8967af support

[Frank Li]

From: Han Xu <han.xu@nxp.com>

fxls8967af is similar with fxls8962af, the only difference is the device id
change to 0x87.

Signed-off-by: Han Xu <han.xu@nxp.com>
Reviewed-by: Sean Nyekjaer <sean@geanix.com>
Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 drivers/iio/accel/fxls8962af-core.c | 7 +++++++
 drivers/iio/accel/fxls8962af-i2c.c  | 1 +
 drivers/iio/accel/fxls8962af.h      | 1 +
 3 files changed, 9 insertions(+)

diff --git a/drivers/iio/accel/fxls8962af-core.c b/drivers/iio/accel/fxls8962af-core.c
index b5607e753a7db..fd9b461904c20 100644
--- a/drivers/iio/accel/fxls8962af-core.c
+++ b/drivers/iio/accel/fxls8962af-core.c
@@ -130,6 +130,7 @@
 #define FXLS8962AF_DEVICE_ID			0x62
 #define FXLS8964AF_DEVICE_ID			0x84
 #define FXLS8974CF_DEVICE_ID			0x86
+#define FXLS8967AF_DEVICE_ID			0x87
 
 /* Raw temp channel offset */
 #define FXLS8962AF_TEMP_CENTER_VAL		25
@@ -767,6 +768,12 @@ static const struct fxls8962af_chip_info fxls_chip_info_table[] = {
 		.channels = fxls8962af_channels,
 		.num_channels = ARRAY_SIZE(fxls8962af_channels),
 	},
+	[fxls8967af] = {
+		.chip_id = FXLS8967AF_DEVICE_ID,
+		.name = "fxls8967af",
+		.channels = fxls8962af_channels,
+		.num_channels = ARRAY_SIZE(fxls8962af_channels),
+	},
 	[fxls8974cf] = {
 		.chip_id = FXLS8974CF_DEVICE_ID,
 		.name = "fxls8974cf",
diff --git a/drivers/iio/accel/fxls8962af-i2c.c b/drivers/iio/accel/fxls8962af-i2c.c
index 687eb49035111..c66ba3ecf373a 100644
--- a/drivers/iio/accel/fxls8962af-i2c.c
+++ b/drivers/iio/accel/fxls8962af-i2c.c
@@ -30,6 +30,7 @@ static int fxls8962af_probe(struct i2c_client *client)
 static const struct i2c_device_id fxls8962af_id[] = {
 	{ "fxls8962af", fxls8962af },
 	{ "fxls8964af", fxls8964af },
+	{ "fxls8967af", fxls8967af },
 	{ "fxls8974cf", fxls8974cf },
 	{}
 };
diff --git a/drivers/iio/accel/fxls8962af.h b/drivers/iio/accel/fxls8962af.h
index 733b69e01e1cc..1c9adfc8c0dc1 100644
--- a/drivers/iio/accel/fxls8962af.h
+++ b/drivers/iio/accel/fxls8962af.h
@@ -11,6 +11,7 @@ struct device;
 enum {
 	fxls8962af,
 	fxls8964af,
+	fxls8967af,
 	fxls8974cf,
 };
 

-- 
2.34.1

Re: [PATCH v2 1/4] dt-bindings: iio: accel: fxls8962af: add compatible string 'nxp,fxls8967af'

[Rob Herring (Arm)]

On Fri, 15 Nov 2024 15:23:57 -0500, Frank Li wrote:
> From: Han Xu <han.xu@nxp.com>
> 
> Add 'nxp,fxls8967af' compatible for the FXLS8967AF sensor, falling back to
> 'nxp,fxls8962af' as the only difference is the ID.
> 
> Signed-off-by: Han Xu <han.xu@nxp.com>
> Reviewed-by: Sean Nyekjaer <sean@geanix.com>
> Signed-off-by: Frank Li <Frank.Li@nxp.com>
> ---
>  .../devicetree/bindings/iio/accel/nxp,fxls8962af.yaml        | 12 +++++++++---
>  1 file changed, 9 insertions(+), 3 deletions(-)
> 

Acked-by: Rob Herring (Arm) <robh@kernel.org>

Re: [PATCH v2 2/4] dt-bindings: iio: accel: fxls8962af: add compatible string 'nxp,fxls8974cf'

[Rob Herring (Arm)]

On Fri, 15 Nov 2024 15:23:58 -0500, Frank Li wrote:
> From: Han Xu <han.xu@nxp.com>
> 
> Add 'nxp,fxls8974cf' compatible for the FXLS8974CF sensor, falling back to
> 'nxp,fxls8962af' as the only difference is the ID.
> 
> Signed-off-by: Han Xu <han.xu@nxp.com>
> Reviewed-by: Sean Nyekjaer <sean@geanix.com>
> Signed-off-by: Frank Li <Frank.Li@nxp.com>
> ---
>  Documentation/devicetree/bindings/iio/accel/nxp,fxls8962af.yaml | 2 ++
>  1 file changed, 2 insertions(+)
> 

Acked-by: Rob Herring (Arm) <robh@kernel.org>

Re: [PATCH v2 0/4] iio: accel: fxls8962af: add fxls8967af and fxls8974cf

[Jonathan Cameron]

On Fri, 15 Nov 2024 15:23:56 -0500
Frank Li <Frank.Li@nxp.com> wrote:

> fxls8967af and fxls8974cf are similar with fxls8962af, only change ID.
> 
> Signed-off-by: Frank Li <Frank.Li@nxp.com>

Applied to the togreg branch of iio.git and initially pushed out as testing
for 0-day to take a look.  I'll rebase on rc1 once available.

Thanks,

Jonathan


> ---
> Changes in v2:
> - make nxp,fxls8967af and nxp,fxls8974cf fallback to nxp,fxls8962af.
> - Link to v1: https://lore.kernel.org/r/20241113-fxls-v1-0-5e48ff1b1fb8@nxp.com
> 
> ---
> Haibo Chen (1):
>       iio: accel: fxls8962af: add fxls8974cf support
> 
> Han Xu (3):
>       dt-bindings: iio: accel: fxls8962af: add compatible string 'nxp,fxls8967af'
>       dt-bindings: iio: accel: fxls8962af: add compatible string 'nxp,fxls8974cf'
>       iio: accel: fxls8962af: add fxls8967af support
> 
>  .../devicetree/bindings/iio/accel/nxp,fxls8962af.yaml      | 14 +++++++++++---
>  drivers/iio/accel/fxls8962af-core.c                        | 14 ++++++++++++++
>  drivers/iio/accel/fxls8962af-i2c.c                         |  2 ++
>  drivers/iio/accel/fxls8962af.h                             |  2 ++
>  4 files changed, 29 insertions(+), 3 deletions(-)
> ---
> base-commit: bd05b9a700c10473c2f52bf12c5c5938c30e80b0
> change-id: 20241113-fxls-d93a888889f7
> 
> Best regards,
> ---
> Frank Li <Frank.Li@nxp.com>
>