RE: [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@xxxxxxxxxx>
> Sent: Thursday, January 15, 2026 5:08 PM
> To: Sherry Sun <sherry.sun@xxxxxxx>; robh@xxxxxxxxxx;
> krzk+dt@xxxxxxxxxx; conor+dt@xxxxxxxxxx; shawnguo@xxxxxxxxxx;
> s.hauer@xxxxxxxxxxxxxx; festevam@xxxxxxxxx; kwilczynski@xxxxxxxxxx;
> mani@xxxxxxxxxx; andriy.shevchenko@xxxxxxxxxxxxxxx;
> bhelgaas@xxxxxxxxxx; Hongxing Zhu <hongxing.zhu@xxxxxxx>; Frank Li
> <frank.li@xxxxxxx>; l.stach@xxxxxxxxxxxxxx; lpieralisi@xxxxxxxxxx
> Cc: kernel@xxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; imx@xxxxxxxxxxxxxxx;
> linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx
> 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@xxxxxxx>
> > ---
> >  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