* [PATCH v1 1/3] dt-bindings: iio: imu: bmi160: add boolean type for drive-open-drain
2024-12-19 23:47 [PATCH v1 0/3] dt-bindings: iio: imu: add missing property type Vasileios Amoiridis
@ 2024-12-19 23:47 ` Vasileios Amoiridis
2024-12-19 23:47 ` [PATCH v1 2/3] dt-bindings: iio: imu: bmi270: " Vasileios Amoiridis
` (2 subsequent siblings)
3 siblings, 0 replies; 10+ messages in thread
From: Vasileios Amoiridis @ 2024-12-19 23:47 UTC (permalink / raw)
To: jic23, lars, robh, krzk+dt, conor+dt
Cc: lanzano.alex, jagathjog1996, vassilisamir, trabarni, danila,
linux-iio, devicetree, linux-kernel
Add missing type description "boolean" for the drive-open-drain property.
Signed-off-by: Vasileios Amoiridis <vassilisamir@gmail.com>
---
Documentation/devicetree/bindings/iio/imu/bosch,bmi160.yaml | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/iio/imu/bosch,bmi160.yaml b/Documentation/devicetree/bindings/iio/imu/bosch,bmi160.yaml
index 3b0a2d8b2e91..2cf8a0c7eb4c 100644
--- a/Documentation/devicetree/bindings/iio/imu/bosch,bmi160.yaml
+++ b/Documentation/devicetree/bindings/iio/imu/bosch,bmi160.yaml
@@ -37,6 +37,7 @@ properties:
to "INT2" if INT2 pin should be used instead
drive-open-drain:
+ type: boolean
description: |
set if the specified interrupt pin should be configured as
open drain. If not set, defaults to push-pull.
--
2.43.0
^ permalink raw reply related [flat|nested] 10+ messages in thread* [PATCH v1 2/3] dt-bindings: iio: imu: bmi270: add boolean type for drive-open-drain
2024-12-19 23:47 [PATCH v1 0/3] dt-bindings: iio: imu: add missing property type Vasileios Amoiridis
2024-12-19 23:47 ` [PATCH v1 1/3] dt-bindings: iio: imu: bmi160: add boolean type for drive-open-drain Vasileios Amoiridis
@ 2024-12-19 23:47 ` Vasileios Amoiridis
2024-12-23 19:50 ` Alex Lanzano
2024-12-26 19:08 ` Alex Lanzano
2024-12-19 23:47 ` [PATCH v1 3/3] dt-bindings: iio: imu: bmi323: " Vasileios Amoiridis
2024-12-25 14:23 ` [PATCH v1 0/3] dt-bindings: iio: imu: add missing property type Conor Dooley
3 siblings, 2 replies; 10+ messages in thread
From: Vasileios Amoiridis @ 2024-12-19 23:47 UTC (permalink / raw)
To: jic23, lars, robh, krzk+dt, conor+dt
Cc: lanzano.alex, jagathjog1996, vassilisamir, trabarni, danila,
linux-iio, devicetree, linux-kernel
Add missing type description "boolean" for the drive-open-drain property.
Signed-off-by: Vasileios Amoiridis <vassilisamir@gmail.com>
---
Documentation/devicetree/bindings/iio/imu/bosch,bmi270.yaml | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/iio/imu/bosch,bmi270.yaml b/Documentation/devicetree/bindings/iio/imu/bosch,bmi270.yaml
index 7b0cde1c9b0a..860a6c1fea3c 100644
--- a/Documentation/devicetree/bindings/iio/imu/bosch,bmi270.yaml
+++ b/Documentation/devicetree/bindings/iio/imu/bosch,bmi270.yaml
@@ -41,6 +41,7 @@ properties:
- INT2
drive-open-drain:
+ type: boolean
description:
set if the specified interrupt pins should be configured as
open drain. If not set, defaults to push-pull.
--
2.43.0
^ permalink raw reply related [flat|nested] 10+ messages in thread* Re: [PATCH v1 2/3] dt-bindings: iio: imu: bmi270: add boolean type for drive-open-drain
2024-12-19 23:47 ` [PATCH v1 2/3] dt-bindings: iio: imu: bmi270: " Vasileios Amoiridis
@ 2024-12-23 19:50 ` Alex Lanzano
2024-12-25 14:21 ` Conor Dooley
2024-12-26 19:08 ` Alex Lanzano
1 sibling, 1 reply; 10+ messages in thread
From: Alex Lanzano @ 2024-12-23 19:50 UTC (permalink / raw)
To: Vasileios Amoiridis
Cc: jic23, lars, robh, krzk+dt, conor+dt, jagathjog1996, trabarni,
danila, linux-iio, devicetree, linux-kernel
On Fri, Dec 20, 2024 at 12:47:44AM +0100, Vasileios Amoiridis wrote:
> Add missing type description "boolean" for the drive-open-drain property.
>
> Signed-off-by: Vasileios Amoiridis <vassilisamir@gmail.com>
> ---
> Documentation/devicetree/bindings/iio/imu/bosch,bmi270.yaml | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/Documentation/devicetree/bindings/iio/imu/bosch,bmi270.yaml b/Documentation/devicetree/bindings/iio/imu/bosch,bmi270.yaml
> index 7b0cde1c9b0a..860a6c1fea3c 100644
> --- a/Documentation/devicetree/bindings/iio/imu/bosch,bmi270.yaml
> +++ b/Documentation/devicetree/bindings/iio/imu/bosch,bmi270.yaml
> @@ -41,6 +41,7 @@ properties:
> - INT2
>
> drive-open-drain:
> + type: boolean
> description:
> set if the specified interrupt pins should be configured as
> open drain. If not set, defaults to push-pull.
Signed-off-by: Alex Lanzano <lanzano.alex@gmail.com>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v1 2/3] dt-bindings: iio: imu: bmi270: add boolean type for drive-open-drain
2024-12-23 19:50 ` Alex Lanzano
@ 2024-12-25 14:21 ` Conor Dooley
2024-12-26 19:06 ` Alex Lanzano
0 siblings, 1 reply; 10+ messages in thread
From: Conor Dooley @ 2024-12-25 14:21 UTC (permalink / raw)
To: Alex Lanzano
Cc: Vasileios Amoiridis, jic23, lars, robh, krzk+dt, conor+dt,
jagathjog1996, trabarni, danila, linux-iio, devicetree,
linux-kernel
[-- Attachment #1: Type: text/plain, Size: 1246 bytes --]
On Mon, Dec 23, 2024 at 02:50:53PM -0500, Alex Lanzano wrote:
> On Fri, Dec 20, 2024 at 12:47:44AM +0100, Vasileios Amoiridis wrote:
> > Add missing type description "boolean" for the drive-open-drain property.
> >
> > Signed-off-by: Vasileios Amoiridis <vassilisamir@gmail.com>
> > ---
> > Documentation/devicetree/bindings/iio/imu/bosch,bmi270.yaml | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/Documentation/devicetree/bindings/iio/imu/bosch,bmi270.yaml b/Documentation/devicetree/bindings/iio/imu/bosch,bmi270.yaml
> > index 7b0cde1c9b0a..860a6c1fea3c 100644
> > --- a/Documentation/devicetree/bindings/iio/imu/bosch,bmi270.yaml
> > +++ b/Documentation/devicetree/bindings/iio/imu/bosch,bmi270.yaml
> > @@ -41,6 +41,7 @@ properties:
> > - INT2
> >
> > drive-open-drain:
> > + type: boolean
> > description:
> > set if the specified interrupt pins should be configured as
> > open drain. If not set, defaults to push-pull.
>
> Signed-off-by: Alex Lanzano <lanzano.alex@gmail.com>
What do you mean with this tag? A sign-off means you're part of the
chain of custody of the patch, but you're neither author nor submitter.
Did you mean Reviewed-by or Acked-by?
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v1 2/3] dt-bindings: iio: imu: bmi270: add boolean type for drive-open-drain
2024-12-25 14:21 ` Conor Dooley
@ 2024-12-26 19:06 ` Alex Lanzano
0 siblings, 0 replies; 10+ messages in thread
From: Alex Lanzano @ 2024-12-26 19:06 UTC (permalink / raw)
To: Conor Dooley
Cc: Vasileios Amoiridis, jic23, lars, robh, krzk+dt, conor+dt,
jagathjog1996, trabarni, danila, linux-iio, devicetree,
linux-kernel
On Wed, Dec 25, 2024 at 02:21:42PM +0000, Conor Dooley wrote:
> On Mon, Dec 23, 2024 at 02:50:53PM -0500, Alex Lanzano wrote:
> > On Fri, Dec 20, 2024 at 12:47:44AM +0100, Vasileios Amoiridis wrote:
> > > Add missing type description "boolean" for the drive-open-drain property.
> > >
> > > Signed-off-by: Vasileios Amoiridis <vassilisamir@gmail.com>
> > > ---
> > > Documentation/devicetree/bindings/iio/imu/bosch,bmi270.yaml | 1 +
> > > 1 file changed, 1 insertion(+)
> > >
> > > diff --git a/Documentation/devicetree/bindings/iio/imu/bosch,bmi270.yaml b/Documentation/devicetree/bindings/iio/imu/bosch,bmi270.yaml
> > > index 7b0cde1c9b0a..860a6c1fea3c 100644
> > > --- a/Documentation/devicetree/bindings/iio/imu/bosch,bmi270.yaml
> > > +++ b/Documentation/devicetree/bindings/iio/imu/bosch,bmi270.yaml
> > > @@ -41,6 +41,7 @@ properties:
> > > - INT2
> > >
> > > drive-open-drain:
> > > + type: boolean
> > > description:
> > > set if the specified interrupt pins should be configured as
> > > open drain. If not set, defaults to push-pull.
> >
> > Signed-off-by: Alex Lanzano <lanzano.alex@gmail.com>
>
> What do you mean with this tag? A sign-off means you're part of the
> chain of custody of the patch, but you're neither author nor submitter.
> Did you mean Reviewed-by or Acked-by?
Whoops! I ment to use the reviewed-by tag
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v1 2/3] dt-bindings: iio: imu: bmi270: add boolean type for drive-open-drain
2024-12-19 23:47 ` [PATCH v1 2/3] dt-bindings: iio: imu: bmi270: " Vasileios Amoiridis
2024-12-23 19:50 ` Alex Lanzano
@ 2024-12-26 19:08 ` Alex Lanzano
1 sibling, 0 replies; 10+ messages in thread
From: Alex Lanzano @ 2024-12-26 19:08 UTC (permalink / raw)
To: Vasileios Amoiridis
Cc: jic23, lars, robh, krzk+dt, conor+dt, jagathjog1996, trabarni,
danila, linux-iio, devicetree, linux-kernel
On Fri, Dec 20, 2024 at 12:47:44AM +0100, Vasileios Amoiridis wrote:
> Add missing type description "boolean" for the drive-open-drain property.
>
> Signed-off-by: Vasileios Amoiridis <vassilisamir@gmail.com>
> ---
> Documentation/devicetree/bindings/iio/imu/bosch,bmi270.yaml | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/Documentation/devicetree/bindings/iio/imu/bosch,bmi270.yaml b/Documentation/devicetree/bindings/iio/imu/bosch,bmi270.yaml
> index 7b0cde1c9b0a..860a6c1fea3c 100644
> --- a/Documentation/devicetree/bindings/iio/imu/bosch,bmi270.yaml
> +++ b/Documentation/devicetree/bindings/iio/imu/bosch,bmi270.yaml
> @@ -41,6 +41,7 @@ properties:
> - INT2
>
> drive-open-drain:
> + type: boolean
> description:
> set if the specified interrupt pins should be configured as
> open drain. If not set, defaults to push-pull.
> --
> 2.43.0
>
Ignore previous tag I sent in.
Reviewed-by: Alex Lanzano <lanzano.alex@gmail.com>
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v1 3/3] dt-bindings: iio: imu: bmi323: add boolean type for drive-open-drain
2024-12-19 23:47 [PATCH v1 0/3] dt-bindings: iio: imu: add missing property type Vasileios Amoiridis
2024-12-19 23:47 ` [PATCH v1 1/3] dt-bindings: iio: imu: bmi160: add boolean type for drive-open-drain Vasileios Amoiridis
2024-12-19 23:47 ` [PATCH v1 2/3] dt-bindings: iio: imu: bmi270: " Vasileios Amoiridis
@ 2024-12-19 23:47 ` Vasileios Amoiridis
2024-12-25 14:23 ` [PATCH v1 0/3] dt-bindings: iio: imu: add missing property type Conor Dooley
3 siblings, 0 replies; 10+ messages in thread
From: Vasileios Amoiridis @ 2024-12-19 23:47 UTC (permalink / raw)
To: jic23, lars, robh, krzk+dt, conor+dt
Cc: lanzano.alex, jagathjog1996, vassilisamir, trabarni, danila,
linux-iio, devicetree, linux-kernel
Add missing type description "boolean" for the drive-open-drain property.
Signed-off-by: Vasileios Amoiridis <vassilisamir@gmail.com>
---
Documentation/devicetree/bindings/iio/imu/bosch,bmi323.yaml | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/iio/imu/bosch,bmi323.yaml b/Documentation/devicetree/bindings/iio/imu/bosch,bmi323.yaml
index 64ef26e19669..7bf8294a8f2e 100644
--- a/Documentation/devicetree/bindings/iio/imu/bosch,bmi323.yaml
+++ b/Documentation/devicetree/bindings/iio/imu/bosch,bmi323.yaml
@@ -38,6 +38,7 @@ properties:
- INT2
drive-open-drain:
+ type: boolean
description:
set if the specified interrupt pin should be configured as
open drain. If not set, defaults to push-pull.
--
2.43.0
^ permalink raw reply related [flat|nested] 10+ messages in thread* Re: [PATCH v1 0/3] dt-bindings: iio: imu: add missing property type
2024-12-19 23:47 [PATCH v1 0/3] dt-bindings: iio: imu: add missing property type Vasileios Amoiridis
` (2 preceding siblings ...)
2024-12-19 23:47 ` [PATCH v1 3/3] dt-bindings: iio: imu: bmi323: " Vasileios Amoiridis
@ 2024-12-25 14:23 ` Conor Dooley
2024-12-28 13:54 ` Jonathan Cameron
3 siblings, 1 reply; 10+ messages in thread
From: Conor Dooley @ 2024-12-25 14:23 UTC (permalink / raw)
To: Vasileios Amoiridis
Cc: jic23, lars, robh, krzk+dt, conor+dt, lanzano.alex, jagathjog1996,
trabarni, danila, linux-iio, devicetree, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 485 bytes --]
On Fri, Dec 20, 2024 at 12:47:42AM +0100, Vasileios Amoiridis wrote:
> It seems that some drivers miss the "type: boolean" under the
> drive-open-drain property, so add it.
>
> Vasileios Amoiridis (3):
> dt-bindings: iio: imu: bmi160: add boolean type for drive-open-drain
> dt-bindings: iio: imu: bmi270: add boolean type for drive-open-drain
> dt-bindings: iio: imu: bmi323: add boolean type for drive-open-drain
Acked-by: Conor Dooley <conor.dooley@microchip.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH v1 0/3] dt-bindings: iio: imu: add missing property type
2024-12-25 14:23 ` [PATCH v1 0/3] dt-bindings: iio: imu: add missing property type Conor Dooley
@ 2024-12-28 13:54 ` Jonathan Cameron
0 siblings, 0 replies; 10+ messages in thread
From: Jonathan Cameron @ 2024-12-28 13:54 UTC (permalink / raw)
To: Conor Dooley
Cc: Vasileios Amoiridis, lars, robh, krzk+dt, conor+dt, lanzano.alex,
jagathjog1996, trabarni, danila, linux-iio, devicetree,
linux-kernel
On Wed, 25 Dec 2024 14:23:34 +0000
Conor Dooley <conor@kernel.org> wrote:
> On Fri, Dec 20, 2024 at 12:47:42AM +0100, Vasileios Amoiridis wrote:
> > It seems that some drivers miss the "type: boolean" under the
> > drive-open-drain property, so add it.
> >
> > Vasileios Amoiridis (3):
> > dt-bindings: iio: imu: bmi160: add boolean type for drive-open-drain
> > dt-bindings: iio: imu: bmi270: add boolean type for drive-open-drain
> > dt-bindings: iio: imu: bmi323: add boolean type for drive-open-drain
>
> Acked-by: Conor Dooley <conor.dooley@microchip.com>
Applied to the togreg branch of iio.git and pushed out as testing
for 0-day to probably ignore these.
Thanks,
Jonathan
^ permalink raw reply [flat|nested] 10+ messages in thread