dt-bindings: i2c: opencores: Add missing type for "regstep"

[PATCH] dt-bindings: i2c: opencores: Add missing type for "regstep"

Posted by Rob Herring 2 years, 7 months ago

"regstep" may be deprecated, but it still needs a type.

Signed-off-by: Rob Herring <robh@kernel.org>
---
 Documentation/devicetree/bindings/i2c/opencores,i2c-ocores.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/i2c/opencores,i2c-ocores.yaml b/Documentation/devicetree/bindings/i2c/opencores,i2c-ocores.yaml
index 85d9efb743ee..d9ef86729011 100644
--- a/Documentation/devicetree/bindings/i2c/opencores,i2c-ocores.yaml
+++ b/Documentation/devicetree/bindings/i2c/opencores,i2c-ocores.yaml
@@ -60,6 +60,7 @@ properties:
     default: 0
 
   regstep:
+    $ref: /schemas/types.yaml#/definitions/uint32
     description: |
       deprecated, use reg-shift above
     deprecated: true
-- 
2.39.2

Re: [PATCH] dt-bindings: i2c: opencores: Add missing type for "regstep"

Posted by Wolfram Sang 2 years, 7 months ago

On Tue, Jun 13, 2023 at 02:11:04PM -0600, Rob Herring wrote:
> "regstep" may be deprecated, but it still needs a type.
> 
> Signed-off-by: Rob Herring <robh@kernel.org>

Added fixes tag and applied to for-current, thanks!

Re: [PATCH] dt-bindings: i2c: opencores: Add missing type for "regstep"

Posted by Andi Shyti 2 years, 7 months ago

Hi Rob,

On Tue, Jun 13, 2023 at 02:11:04PM -0600, Rob Herring wrote:
> "regstep" may be deprecated, but it still needs a type.
> 
> Signed-off-by: Rob Herring <robh@kernel.org>

Acked-by: Andi Shyti <andi.shyti@kernel.org> 

Thanks,
Andi

Re: [PATCH] dt-bindings: i2c: opencores: Add missing type for "regstep"

Posted by Conor Dooley 2 years, 7 months ago

On Tue, Jun 13, 2023 at 02:11:04PM -0600, Rob Herring wrote:
> "regstep" may be deprecated, but it still needs a type.

Mea culpa!
Reviewed-by: Conor Dooley <conor.dooley@microchip.com>

> 
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
>  Documentation/devicetree/bindings/i2c/opencores,i2c-ocores.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/i2c/opencores,i2c-ocores.yaml b/Documentation/devicetree/bindings/i2c/opencores,i2c-ocores.yaml
> index 85d9efb743ee..d9ef86729011 100644
> --- a/Documentation/devicetree/bindings/i2c/opencores,i2c-ocores.yaml
> +++ b/Documentation/devicetree/bindings/i2c/opencores,i2c-ocores.yaml
> @@ -60,6 +60,7 @@ properties:
>      default: 0
>  
>    regstep:
> +    $ref: /schemas/types.yaml#/definitions/uint32
>      description: |
>        deprecated, use reg-shift above
>      deprecated: true
> -- 
> 2.39.2
>

Re: [PATCH] dt-bindings: i2c: opencores: Add missing type for "regstep"

Posted by Peter Korsgaard 2 years, 7 months ago

>>>>> "Rob" == Rob Herring <robh@kernel.org> writes:

 > "regstep" may be deprecated, but it still needs a type.
 > Signed-off-by: Rob Herring <robh@kernel.org>
 > ---
 >  Documentation/devicetree/bindings/i2c/opencores,i2c-ocores.yaml | 1 +
 >  1 file changed, 1 insertion(+)

 > diff --git a/Documentation/devicetree/bindings/i2c/opencores,i2c-ocores.yaml b/Documentation/devicetree/bindings/i2c/opencores,i2c-ocores.yaml
 > index 85d9efb743ee..d9ef86729011 100644
 > --- a/Documentation/devicetree/bindings/i2c/opencores,i2c-ocores.yaml
 > +++ b/Documentation/devicetree/bindings/i2c/opencores,i2c-ocores.yaml
 > @@ -60,6 +60,7 @@ properties:
 >      default: 0
 
 >    regstep:
 > +    $ref: /schemas/types.yaml#/definitions/uint32

Reviewed-by: Peter Korsgaard <peter@korsgaard.com>

-- 
Bye, Peter Korsgaard

Re: [PATCH] dt-bindings: i2c: opencores: Add missing type for "regstep"

Posted by Andrew Lunn 2 years, 7 months ago

On Tue, Jun 13, 2023 at 02:11:04PM -0600, Rob Herring wrote:
> "regstep" may be deprecated, but it still needs a type.
> 
> Signed-off-by: Rob Herring <robh@kernel.org>

Reviewed-by: Andrew Lunn <andrew@lunn.ch>

    Andrew