[PATCH v2 1/2] dt-bindings: PCI: fsl,imx6q-pcie: Explicitly deprecate reset-gpio

From: Krzysztof Kozlowski

Date: Thu Mar 12 2026 - 14:45:52 EST


The binding references snps,dw-pcie-common.yaml which already deprecates
"reset-gpio", however mentioning this property here lacks this mark thus
code is confusing and suggests property is not deprecated.

OTOH, the property cannot be removed from this schema, because it is an
ABI implemented by Linux kernel in gpiolib-of.c in backwards-compatible
map between compatibles, "reset-gpio" and "reset-gpio-active-high"
properties.

This creates code and binding which is not obvious to follow, so rewrite
it and be more explicit about:
1. Deprecation of "reset-gpio",
2. Relation that "reset-gpio-active-high" is used only for that property
and not proper "reset-gpios".

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxxxxxxxx>

---

Cc: Sherry Sun <sherry.sun@xxxxxxx>

Changes in v2:
1. Rewrite the commit completely after Sherry Sun comments - keep the
property, but add "not" clause to forbid usavge of
"reset-gpio-active-high" and "reset-gpios".
2. Because of above not adding Richard's ack.

v1:
https://lore.kernel.org/r/20260311154955.145120-3-krzysztof.kozlowski@xxxxxxxxxxxxxxxx
---
Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml | 8 ++++++++
1 file changed, 8 insertions(+)

diff --git a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml
index 12a01f7a5744..12cd1b9f78d0 100644
--- a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml
+++ b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml
@@ -62,6 +62,9 @@ properties:
description: Should specify the GPIO for controlling the PCI bus device
reset signal. It's not polarity aware and defaults to active-low reset
sequence (L=reset state, H=operation state) (optional required).
+ Deprecated, use reset-gpios which respects the polarity expressed in
+ phandle argument.
+ deprecated: true

reset-gpio-active-high:
description: If present then the reset sequence using the GPIO
@@ -224,6 +227,11 @@ allOf:
- const: ref
- const: extref # Optional

+ - not:
+ required:
+ - reset-gpios
+ - reset-gpio-active-high
+
unevaluatedProperties: false

examples:
--
2.51.0