2024-01-15 18:22:25

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH] dt-bindings: mfd: iqs62x: Do not override firmware-name $ref

dtschema package defines firmware-name as string-array, so individual
bindings should not make it a string but instead just narrow the number
of expected firmware file names.

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
Documentation/devicetree/bindings/mfd/iqs62x.yaml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/mfd/iqs62x.yaml b/Documentation/devicetree/bindings/mfd/iqs62x.yaml
index 044cd7542c2b..f438c2374966 100644
--- a/Documentation/devicetree/bindings/mfd/iqs62x.yaml
+++ b/Documentation/devicetree/bindings/mfd/iqs62x.yaml
@@ -31,7 +31,7 @@ properties:
maxItems: 1

firmware-name:
- $ref: /schemas/types.yaml#/definitions/string
+ maxItems: 1
description:
Specifies the name of the calibration and configuration file selected by
the driver. If this property is omitted, the name is chosen based on the
--
2.34.1



2024-01-15 22:33:06

by Jeff LaBundy

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: mfd: iqs62x: Do not override firmware-name $ref

On Mon, Jan 15, 2024 at 07:20:42PM +0100, Krzysztof Kozlowski wrote:
> dtschema package defines firmware-name as string-array, so individual
> bindings should not make it a string but instead just narrow the number
> of expected firmware file names.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>

Acked-by: Jeff LaBundy <[email protected]>

> ---
> Documentation/devicetree/bindings/mfd/iqs62x.yaml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/mfd/iqs62x.yaml b/Documentation/devicetree/bindings/mfd/iqs62x.yaml
> index 044cd7542c2b..f438c2374966 100644
> --- a/Documentation/devicetree/bindings/mfd/iqs62x.yaml
> +++ b/Documentation/devicetree/bindings/mfd/iqs62x.yaml
> @@ -31,7 +31,7 @@ properties:
> maxItems: 1
>
> firmware-name:
> - $ref: /schemas/types.yaml#/definitions/string
> + maxItems: 1
> description:
> Specifies the name of the calibration and configuration file selected by
> the driver. If this property is omitted, the name is chosen based on the
> --
> 2.34.1
>

2024-01-16 17:36:33

by Conor Dooley

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: mfd: iqs62x: Do not override firmware-name $ref

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

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

2024-01-18 14:50:56

by Lee Jones

[permalink] [raw]
Subject: Re: (subset) [PATCH] dt-bindings: mfd: iqs62x: Do not override firmware-name $ref

On Mon, 15 Jan 2024 19:20:42 +0100, Krzysztof Kozlowski wrote:
> dtschema package defines firmware-name as string-array, so individual
> bindings should not make it a string but instead just narrow the number
> of expected firmware file names.
>
>

Applied, thanks!

[1/1] dt-bindings: mfd: iqs62x: Do not override firmware-name $ref
commit: da4326075cff7ee6416309a9c3641d7eca2a53c7

--
Lee Jones [李琼斯]