[PATCH v4] dt-bindings: gpio: sifive: add gpio-line-names

Conor Dooley posted 1 patch 3 years, 8 months ago
Documentation/devicetree/bindings/gpio/sifive,gpio.yaml | 4 ++++
1 file changed, 4 insertions(+)
[PATCH v4] dt-bindings: gpio: sifive: add gpio-line-names
Posted by Conor Dooley 3 years, 8 months ago
From: Atul Khare <atulkhare@rivosinc.com>

Fix device tree schema validation messages like 'gpio-line-names'
does not match any of the regexes: 'pinctrl-[0-9]+' From schema: ...
sifive,gpio.yaml'.

The bindings were missing the gpio-line-names element, which was
causing the dt-schema checker to trip-up.

Acked-by: Rob Herring <robh@kernel.org>
Signed-off-by: Atul Khare <atulkhare@rivosinc.com>
Signed-off-by: Conor Dooley <conor.dooley@microchip.com>
---
Changes since v3[0]:
- Dropped patch 1 & the now unneeded cover letter
- Added Rob's Ack
- Changed the patch title to include the subsystem

0 - https://lore.kernel.org/all/20220726170725.3245278-3-mail@conchuod.ie/
---
 Documentation/devicetree/bindings/gpio/sifive,gpio.yaml | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/Documentation/devicetree/bindings/gpio/sifive,gpio.yaml b/Documentation/devicetree/bindings/gpio/sifive,gpio.yaml
index 939e31c48081..fc095646adea 100644
--- a/Documentation/devicetree/bindings/gpio/sifive,gpio.yaml
+++ b/Documentation/devicetree/bindings/gpio/sifive,gpio.yaml
@@ -46,6 +46,10 @@ properties:
     maximum: 32
     default: 16
 
+  gpio-line-names:
+    minItems: 1
+    maxItems: 32
+
   gpio-controller: true
 
 required:
-- 
2.37.1
Re: [PATCH v4] dt-bindings: gpio: sifive: add gpio-line-names
Posted by Palmer Dabbelt 3 years, 8 months ago
On Wed, 03 Aug 2022 08:55:40 PDT (-0700), mail@conchuod.ie wrote:
> From: Atul Khare <atulkhare@rivosinc.com>
>
> Fix device tree schema validation messages like 'gpio-line-names'
> does not match any of the regexes: 'pinctrl-[0-9]+' From schema: ...
> sifive,gpio.yaml'.
>
> The bindings were missing the gpio-line-names element, which was
> causing the dt-schema checker to trip-up.
>
> Acked-by: Rob Herring <robh@kernel.org>
> Signed-off-by: Atul Khare <atulkhare@rivosinc.com>
> Signed-off-by: Conor Dooley <conor.dooley@microchip.com>
> ---
> Changes since v3[0]:
> - Dropped patch 1 & the now unneeded cover letter
> - Added Rob's Ack
> - Changed the patch title to include the subsystem
>
> 0 - https://lore.kernel.org/all/20220726170725.3245278-3-mail@conchuod.ie/
> ---
>  Documentation/devicetree/bindings/gpio/sifive,gpio.yaml | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/gpio/sifive,gpio.yaml b/Documentation/devicetree/bindings/gpio/sifive,gpio.yaml
> index 939e31c48081..fc095646adea 100644
> --- a/Documentation/devicetree/bindings/gpio/sifive,gpio.yaml
> +++ b/Documentation/devicetree/bindings/gpio/sifive,gpio.yaml
> @@ -46,6 +46,10 @@ properties:
>      maximum: 32
>      default: 16
>
> +  gpio-line-names:
> +    minItems: 1
> +    maxItems: 32
> +
>    gpio-controller: true
>
>  required:

Thanks, this is on for-next.