[PATCH 3/3] dt-bindings: PCI: fsl,imx6q-pcie: Remove the deprecated "reset-gpio-active-high" property

[Sherry Sun]

> -----Original Message-----
> From: Krzysztof Kozlowski <krzk at kernel.org>
> Sent: Thursday, January 15, 2026 5:08 PM
> To: Sherry Sun <sherry.sun at nxp.com>; robh at kernel.org;
> krzk+dt at kernel.org; conor+dt at kernel.org; shawnguo at kernel.org;
> s.hauer at pengutronix.de; festevam at gmail.com; kwilczynski at kernel.org;
> mani at kernel.org; andriy.shevchenko at linux.intel.com;
> bhelgaas at google.com; Hongxing Zhu <hongxing.zhu at nxp.com>; Frank Li
> <frank.li at nxp.com>; l.stach at pengutronix.de; lpieralisi at kernel.org
> Cc: kernel at pengutronix.de; devicetree at vger.kernel.org; imx at lists.linux.dev;
> linux-arm-kernel at lists.infradead.org; linux-kernel at vger.kernel.org
> Subject: Re: [PATCH 3/3] dt-bindings: PCI: fsl,imx6q-pcie: Remove the
> deprecated "reset-gpio-active-high" property
> 
> On 15/01/2026 08:18, Sherry Sun wrote:
> > Commit 2e81122d681c ("PCI: imx6: Convert to use agnostic GPIO API")
> > deleted the support of "reset-gpio-active-high" property in imx6 PCIe
> > driver. This deprecated preoperty is no longer used in any DTS file,
> > let's also remove this in dt-binding.
> >
> > Signed-off-by: Sherry Sun <sherry.sun at nxp.com>
> > ---
> >  Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml | 6 ------
> >  1 file changed, 6 deletions(-)
> >
> > diff --git a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml
> > b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml
> > index 12a01f7a5744..6b18264afecf 100644
> > --- a/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml
> > +++ b/Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.yaml
> > @@ -63,12 +63,6 @@ properties:
> >        reset signal. It's not polarity aware and defaults to active-low reset
> >        sequence (L=reset state, H=operation state) (optional required).
> >
> > -  reset-gpio-active-high:
> > -    description: If present then the reset sequence using the GPIO
> > -      specified in the "reset-gpio" property is reversed (H=reset state,
> > -      L=operation state) (optional required).
> 
> You had one command to run before preparing such patch:
> 
> git grep reset-gpio-active-high
> 
> so now you made this ABI undocumented.
> 
Hi Krzysztof,
Yes, Francesco also reminded me for this, thanks for the catch, will be more careful next time.

Best Regards
Sherry