Skip to contentSkip to conversation

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

PATCH Applied Conor Dooley 2 participants 2 messages Mar 10 – Mar 11

Subsystems (3)
lore
Share & tools
lore
#0 lore raw

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

From: Conor Dooley <[email protected]>

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.

Fixes a901e8705f89f ("dt-bindings: pinctrl: pincfg-node: add restrictions on conflicting properties")

---
CC: Linus Walleij <[email protected]>
CC: Rob Herring <[email protected]>
CC: Krzysztof Kozlowski <[email protected]>
CC: Conor Dooley <[email protected]>
CC: [email protected]
CC: [email protected]
CC: [email protected]

---
 Documentation/devicetree/bindings/pinctrl/pincfg-node.yaml | 4 ----
 1 file changed, 4 deletions(-)
Documentation/devicetree/bindings/pinctrl/pincfg-node.yaml +0 -4
index fe936ab091040..981f45c2f56b0 100644
@@ -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:
Signature · 1 line
2.51.0
maintainer Applied
#1 lore raw
Quoted 10 lines · Conor Dooley

From: Conor Dooley <[email protected]>

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.

Fixes a901e8705f89f ("dt-bindings: pinctrl: pincfg-node: add restrictions on conflicting properties")

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

Yours,
Linus Walleij