devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Théo Lebrun" <theo.lebrun@bootlin.com>
To: "Gregory CLEMENT" <gregory.clement@bootlin.com>,
	"Michael Turquette" <mturquette@baylibre.com>,
	"Stephen Boyd" <sboyd@kernel.org>,
	"Rob Herring" <robh+dt@kernel.org>,
	"Krzysztof Kozlowski" <krzysztof.kozlowski+dt@linaro.org>,
	"Conor Dooley" <conor+dt@kernel.org>,
	"Thomas Bogendoerfer" <tsbogend@alpha.franken.de>,
	"Linus Walleij" <linus.walleij@linaro.org>,
	"Rafał Miłecki" <rafal@milecki.pl>,
	"Philipp Zabel" <p.zabel@pengutronix.de>
Cc: "Vladimir Kondratiev" <vladimir.kondratiev@mobileye.com>,
	linux-mips@vger.kernel.org, linux-clk@vger.kernel.org,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
	"Thomas Petazzoni" <thomas.petazzoni@bootlin.com>,
	"Tawfik Bayouk" <tawfik.bayouk@mobileye.com>,
	linux-gpio@vger.kernel.org,
	"Théo Lebrun" <theo.lebrun@bootlin.com>
Subject: [PATCH v4 03/18] dt-bindings: pinctrl: allow pin controller device without unit address
Date: Wed, 31 Jan 2024 17:26:16 +0100	[thread overview]
Message-ID: <20240131-mbly-clk-v4-3-bcd00510d6a0@bootlin.com> (raw)
In-Reply-To: <20240131-mbly-clk-v4-0-bcd00510d6a0@bootlin.com>

Allow a pin controller device to have no address, therefore no unit
address.

The previous $nodename was enforcing a unit address, but
scripts/dtc/checks.c enforced that names with unit addresses have reg
or ranges:

   Warning (unit_address_vs_reg): .../pinctrl@0: node has a unit
   name, but no reg or ranges property

Fix pinctrl.yaml to adopt a (pinctrl|pinmux)(-[a-z]+)? node name when
neither reg nor ranges are required. Use [a-z]+ to avoid conflicts with
pinctrl-consumer.yaml.

Signed-off-by: Théo Lebrun <theo.lebrun@bootlin.com>
---
 Documentation/devicetree/bindings/pinctrl/pinctrl.yaml | 18 +++++++++++++++---
 1 file changed, 15 insertions(+), 3 deletions(-)

diff --git a/Documentation/devicetree/bindings/pinctrl/pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/pinctrl.yaml
index d471563119a9..3d977b54e260 100644
--- a/Documentation/devicetree/bindings/pinctrl/pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/pinctrl.yaml
@@ -26,9 +26,6 @@ description: |
   controller device.
 
 properties:
-  $nodename:
-    pattern: "^(pinctrl|pinmux)(@[0-9a-f]+)?$"
-
   "#pinctrl-cells":
     description: >
       Number of pin control cells in addition to the index within the pin
@@ -42,4 +39,19 @@ properties:
       This property can be set either globally for the pin controller or in
       child nodes for individual pin group control.
 
+if:
+  anyOf:
+    - required:
+        - reg
+    - required:
+        - ranges
+then:
+  properties:
+    $nodename:
+      pattern: "^(pinctrl|pinmux)(@[0-9a-f]+)?$"
+else:
+  properties:
+    $nodename:
+      pattern: "^(pinctrl|pinmux)(-[a-z]+)?$"
+
 additionalProperties: true

-- 
2.43.0


  parent reply	other threads:[~2024-01-31 16:27 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-01-31 16:26 [PATCH v4 00/18] Add support for Mobileye EyeQ5 system controller Théo Lebrun
2024-01-31 16:26 ` [PATCH v4 01/18] clk: fixed-factor: add optional accuracy support Théo Lebrun
2024-01-31 16:26 ` [PATCH v4 02/18] clk: fixed-factor: add fwname-based constructor functions Théo Lebrun
2024-01-31 16:26 ` Théo Lebrun [this message]
2024-02-02 19:52   ` [PATCH v4 03/18] dt-bindings: pinctrl: allow pin controller device without unit address Rob Herring
2024-01-31 16:26 ` [PATCH v4 04/18] dt-bindings: clock: mobileye,eyeq5-clk: add bindings Théo Lebrun
2024-02-01  8:58   ` Krzysztof Kozlowski
2024-02-01 10:38     ` Théo Lebrun
2024-02-01 11:00       ` Krzysztof Kozlowski
2024-02-06 10:13         ` Théo Lebrun
2024-01-31 16:26 ` [PATCH v4 05/18] dt-bindings: reset: mobileye,eyeq5-reset: " Théo Lebrun
2024-02-01  9:00   ` Krzysztof Kozlowski
2024-01-31 16:26 ` [PATCH v4 06/18] dt-bindings: pinctrl: mobileye,eyeq5-pinctrl: " Théo Lebrun
2024-02-05 17:19   ` Rob Herring
2024-01-31 16:26 ` [PATCH v4 07/18] dt-bindings: soc: mobileye: add EyeQ5 OLB system controller Théo Lebrun
2024-02-01  9:36   ` Krzysztof Kozlowski
2024-02-01 10:45     ` Théo Lebrun
2024-01-31 16:26 ` [PATCH v4 08/18] clk: eyeq5: add platform driver, and init routine at of_clk_init() Théo Lebrun
2024-01-31 16:26 ` [PATCH v4 09/18] reset: eyeq5: add platform driver Théo Lebrun
2024-01-31 16:26 ` [PATCH v4 10/18] pinctrl: " Théo Lebrun
2024-01-31 20:55   ` Linus Walleij
2024-02-01 10:23     ` Théo Lebrun
2024-02-01 16:47       ` Linus Walleij
2024-01-31 16:26 ` [PATCH v4 11/18] MIPS: mobileye: eyeq5: rename olb@e00000 to system-controller@e00000 Théo Lebrun
2024-02-01  9:10   ` Krzysztof Kozlowski
2024-02-01  9:12     ` Krzysztof Kozlowski
2024-01-31 16:26 ` [PATCH v4 12/18] MIPS: mobileye: eyeq5: remove reg-io-width property from OLB syscon Théo Lebrun
2024-01-31 16:26 ` [PATCH v4 13/18] MIPS: mobileye: eyeq5: add memory translation inside " Théo Lebrun
2024-01-31 16:26 ` [PATCH v4 14/18] MIPS: mobileye: eyeq5: use OLB clocks controller Théo Lebrun
2024-01-31 16:26 ` [PATCH v4 15/18] MIPS: mobileye: eyeq5: add OLB reset controller node Théo Lebrun
2024-01-31 16:26 ` [PATCH v4 16/18] MIPS: mobileye: eyeq5: add reset properties to UARTs Théo Lebrun
2024-02-01  9:13   ` Krzysztof Kozlowski
2024-02-01 10:32     ` Théo Lebrun
2024-01-31 16:26 ` [PATCH v4 17/18] MIPS: mobileye: eyeq5: add pinctrl nodes & pinmux function nodes Théo Lebrun
2024-01-31 16:26 ` [PATCH v4 18/18] MIPS: mobileye: eyeq5: add pinctrl properties to UART nodes Théo Lebrun
2024-01-31 20:44 ` [PATCH v4 00/18] Add support for Mobileye EyeQ5 system controller Linus Walleij
2024-02-01 10:30   ` Théo Lebrun

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20240131-mbly-clk-v4-3-bcd00510d6a0@bootlin.com \
    --to=theo.lebrun@bootlin.com \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=gregory.clement@bootlin.com \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=linus.walleij@linaro.org \
    --cc=linux-clk@vger.kernel.org \
    --cc=linux-gpio@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mips@vger.kernel.org \
    --cc=mturquette@baylibre.com \
    --cc=p.zabel@pengutronix.de \
    --cc=rafal@milecki.pl \
    --cc=robh+dt@kernel.org \
    --cc=sboyd@kernel.org \
    --cc=tawfik.bayouk@mobileye.com \
    --cc=thomas.petazzoni@bootlin.com \
    --cc=tsbogend@alpha.franken.de \
    --cc=vladimir.kondratiev@mobileye.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).