2023-10-12 14:21:47

by Martin Kepplinger

[permalink] [raw]
Subject: [PATCH v2] dt-bindings: iio: imu: st,lsm6dsx: add mount-matrix property

Add the mount-matrix optional property to the binding since it's supported
and very useful when using the chip on a board.

Signed-off-by: Martin Kepplinger <[email protected]>
---

revision history
----------------
v2: (thanks Jonathan)
* remove the addition to the example to avoid unrelated discussions :)

v1:
https://lore.kernel.org/linux-devicetree/[email protected]/


Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml | 3 +++
1 file changed, 3 insertions(+)

diff --git a/Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml b/Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml
index ee8724ad33ab..28b667a9cb76 100644
--- a/Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml
+++ b/Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml
@@ -93,6 +93,9 @@ properties:
wakeup-source:
$ref: /schemas/types.yaml#/definitions/flag

+ mount-matrix:
+ description: an optional 3x3 mounting rotation matrix
+
required:
- compatible
- reg
--
2.39.2


2023-10-13 15:21:45

by Conor Dooley

[permalink] [raw]
Subject: Re: [PATCH v2] dt-bindings: iio: imu: st,lsm6dsx: add mount-matrix property

On Thu, Oct 12, 2023 at 02:20:41PM +0000, Martin Kepplinger wrote:
> Add the mount-matrix optional property to the binding since it's supported
> and very useful when using the chip on a board.
>
> Signed-off-by: Martin Kepplinger <[email protected]>

Acked-by: Conor Dooley <[email protected]>

Thanks,
Cnor.

> ---
>
> revision history
> ----------------
> v2: (thanks Jonathan)
> * remove the addition to the example to avoid unrelated discussions :)
>
> v1:
> https://lore.kernel.org/linux-devicetree/[email protected]/
>
>
> Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml b/Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml
> index ee8724ad33ab..28b667a9cb76 100644
> --- a/Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml
> +++ b/Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml
> @@ -93,6 +93,9 @@ properties:
> wakeup-source:
> $ref: /schemas/types.yaml#/definitions/flag
>
> + mount-matrix:
> + description: an optional 3x3 mounting rotation matrix
> +
> required:
> - compatible
> - reg
> --
> 2.39.2
>


Attachments:
(No filename) (1.24 kB)
signature.asc (235.00 B)
Download all attachments

2023-10-14 12:56:07

by Jonathan Cameron

[permalink] [raw]
Subject: Re: [PATCH v2] dt-bindings: iio: imu: st,lsm6dsx: add mount-matrix property

On Fri, 13 Oct 2023 16:19:54 +0100
Conor Dooley <[email protected]> wrote:

> On Thu, Oct 12, 2023 at 02:20:41PM +0000, Martin Kepplinger wrote:
> > Add the mount-matrix optional property to the binding since it's supported
> > and very useful when using the chip on a board.
> >
> > Signed-off-by: Martin Kepplinger <[email protected]>
>
> Acked-by: Conor Dooley <[email protected]>
>
> Thanks,
> Cnor.
>
> > ---
> >
> > revision history
> > ----------------
> > v2: (thanks Jonathan)
> > * remove the addition to the example to avoid unrelated discussions :)
:)

Applied

> >
> > v1:
> > https://lore.kernel.org/linux-devicetree/[email protected]/
> >
> >
> > Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml | 3 +++
> > 1 file changed, 3 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml b/Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml
> > index ee8724ad33ab..28b667a9cb76 100644
> > --- a/Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml
> > +++ b/Documentation/devicetree/bindings/iio/imu/st,lsm6dsx.yaml
> > @@ -93,6 +93,9 @@ properties:
> > wakeup-source:
> > $ref: /schemas/types.yaml#/definitions/flag
> >
> > + mount-matrix:
> > + description: an optional 3x3 mounting rotation matrix
> > +
> > required:
> > - compatible
> > - reg
> > --
> > 2.39.2
> >