mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH] dt-bindings: pinctrl: Fix MT6797 pin nodes
@ 2026-09-22 22:24 Rob Herring (Arm)
  2026-09-23  8:41 ` AngeloGioacchino Del Regno
  0 siblings, 1 reply; 2+ messages in thread
From: Rob Herring (Arm) @ 2026-09-22 22:24 UTC (permalink / raw)
  To: Sean Wang, Linus Walleij, Krzysztof Kozlowski, Conor Dooley,
	Matthias Brugger, AngeloGioacchino Del Regno, Andy Teng
  Cc: linux-mediatek, linux-gpio, devicetree, linux-kernel, linux-arm-kernel

Allow MT6797 pinctrl state nodes named *-pins directly under the
pinctrl controller. The existing mt6797.dtsi uses nodes such as
i2c0-pins and uart0-pins, each with pins-* child nodes, while the
conversion only allowed numbered state nodes such as foo-0.

Assisted-by: LLM
Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
---
 .../devicetree/bindings/pinctrl/mediatek,mt6779-pinctrl.yaml  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/pinctrl/mediatek,mt6779-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/mediatek,mt6779-pinctrl.yaml
index f4bab7a132d3..dba2dec15477 100644
--- a/Documentation/devicetree/bindings/pinctrl/mediatek,mt6779-pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/mediatek,mt6779-pinctrl.yaml
@@ -111,12 +111,12 @@ allOf:
         - "#interrupt-cells"
 
 patternProperties:
-  '-[0-9]*$':
+  '(-pins|-[0-9]+)$':
     type: object
     additionalProperties: false
 
     patternProperties:
-      '-pins*$':
+      '.*pins.*':
         type: object
         description:
           A pinctrl node should contain at least one subnodes representing the
-- 
2.53.0


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

* Re: [PATCH] dt-bindings: pinctrl: Fix MT6797 pin nodes
  2026-09-22 22:24 [PATCH] dt-bindings: pinctrl: Fix MT6797 pin nodes Rob Herring (Arm)
@ 2026-09-23  8:41 ` AngeloGioacchino Del Regno
  0 siblings, 0 replies; 2+ messages in thread
From: AngeloGioacchino Del Regno @ 2026-09-23  8:41 UTC (permalink / raw)
  To: Rob Herring (Arm),
	Sean Wang, Linus Walleij, Krzysztof Kozlowski, Conor Dooley,
	Matthias Brugger, Andy Teng
  Cc: linux-mediatek, linux-gpio, devicetree, linux-kernel, linux-arm-kernel

On 9/23/26 00:24, Rob Herring (Arm) wrote:
> Allow MT6797 pinctrl state nodes named *-pins directly under the
> pinctrl controller. The existing mt6797.dtsi uses nodes such as
> i2c0-pins and uart0-pins, each with pins-* child nodes, while the
> conversion only allowed numbered state nodes such as foo-0.
> 
> Assisted-by: LLM
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>

Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>


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

end of thread, other threads:[~2026-09-23  8:41 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-09-22 22:24 [PATCH] dt-bindings: pinctrl: Fix MT6797 pin nodes Rob Herring (Arm)
2026-09-23  8:41 ` AngeloGioacchino Del Regno

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

all inboxes | Powered by JetHome®