dt-bindings: pinctrl: pincfg-node: permit bias-high-impedance with other bias properties

[PATCH] dt-bindings: pinctrl: pincfg-node: permit bias-high-impedance with other bias properties

Posted by Conor Dooley 4 weeks ago

From: Conor Dooley <conor.dooley@microchip.com>

It is possible that devices tristate buffers may set the buffer to
the high-Z state in addition to setting pull-up or pull-down on a pin.
Remove this particular restriction to prevent warning on zynqmp systems
where this configuration seems to be valid.

Reported-by: Rob Herring (Arm) <robh@kernel.org>
Fixes: a901e8705f89f ("dt-bindings: pinctrl: pincfg-node: add restrictions on conflicting properties")
Signed-off-by: Conor Dooley <conor.dooley@microchip.com>
---
CC: Linus Walleij <linusw@kernel.org>
CC: Rob Herring <robh@kernel.org>
CC: Krzysztof Kozlowski <krzk+dt@kernel.org>
CC: Conor Dooley <conor+dt@kernel.org>
CC: linux-gpio@vger.kernel.org
CC: devicetree@vger.kernel.org
CC: linux-kernel@vger.kernel.org
---
 Documentation/devicetree/bindings/pinctrl/pincfg-node.yaml | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/Documentation/devicetree/bindings/pinctrl/pincfg-node.yaml b/Documentation/devicetree/bindings/pinctrl/pincfg-node.yaml
index fe936ab091040..981f45c2f56b0 100644
--- a/Documentation/devicetree/bindings/pinctrl/pincfg-node.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/pincfg-node.yaml
@@ -236,8 +236,6 @@ allOf:
       anyOf:
         - required:
             - bias-disable
-        - required:
-            - bias-high-impedance
         - required:
             - bias-bus-hold
         - required:
@@ -250,8 +248,6 @@ allOf:
       oneOf:
         - required:
             - bias-disable
-        - required:
-            - bias-high-impedance
         - required:
             - bias-bus-hold
         - required:
-- 
2.51.0

Re: [PATCH] dt-bindings: pinctrl: pincfg-node: permit bias-high-impedance with other bias properties

Posted by Linus Walleij 3 weeks, 6 days ago

On Tue, Mar 10, 2026 at 6:58 PM Conor Dooley <conor@kernel.org> wrote:

> From: Conor Dooley <conor.dooley@microchip.com>
>
> It is possible that devices tristate buffers may set the buffer to
> the high-Z state in addition to setting pull-up or pull-down on a pin.
> Remove this particular restriction to prevent warning on zynqmp systems
> where this configuration seems to be valid.
>
> Reported-by: Rob Herring (Arm) <robh@kernel.org>
> Fixes: a901e8705f89f ("dt-bindings: pinctrl: pincfg-node: add restrictions on conflicting properties")
> Signed-off-by: Conor Dooley <conor.dooley@microchip.com>

Applied to devel (next) so we rid the warnings, thanks!

Yours,
Linus Walleij