Linux IIO development
 help / color / mirror / Atom feed
* [PATCH v1 0/3] dt-bindings: iio: imu: add missing property type
@ 2024-12-19 23:47 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
                   ` (3 more replies)
  0 siblings, 4 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

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

 Documentation/devicetree/bindings/iio/imu/bosch,bmi160.yaml | 1 +
 Documentation/devicetree/bindings/iio/imu/bosch,bmi270.yaml | 1 +
 Documentation/devicetree/bindings/iio/imu/bosch,bmi323.yaml | 1 +
 3 files changed, 3 insertions(+)


base-commit: fe2a04fbcbfe44694fcb36ff6212fce54b8f56cc
-- 
2.43.0


^ permalink raw reply	[flat|nested] 10+ messages in thread

* [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

* [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 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 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 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

* 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

end of thread, other threads:[~2024-12-28 13:54 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
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-23 19:50   ` Alex Lanzano
2024-12-25 14:21     ` Conor Dooley
2024-12-26 19:06       ` 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
2024-12-28 13:54   ` Jonathan Cameron

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox