On Tue, Apr 23, 2019 at 02:58:23PM +0530, Manikanta Maddireddy wrote: > Document "reset-gpio" optional property which supports GPIO based PERST# > signal. > > Signed-off-by: Manikanta Maddireddy <mmaddireddy@xxxxxxxxxx> > --- > V2: Using standard "reset-gpio" property > > .../devicetree/bindings/pci/nvidia,tegra20-pcie.txt | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/Documentation/devicetree/bindings/pci/nvidia,tegra20-pcie.txt b/Documentation/devicetree/bindings/pci/nvidia,tegra20-pcie.txt > index 7939bca47861..4e75e017f660 100644 > --- a/Documentation/devicetree/bindings/pci/nvidia,tegra20-pcie.txt > +++ b/Documentation/devicetree/bindings/pci/nvidia,tegra20-pcie.txt > @@ -162,6 +162,10 @@ Required properties: > - Root port 0 uses 4 lanes, root port 1 is unused. > - Both root ports use 2 lanes. > > +Optional properties: > +- reset-gpio: If GPIO is used as PERST# signal instead of available > + SFIO, add this property with phandle to GPIO controller and GPIO number. 'reset-gpios' is the preferred form. This is already defined in several other drivers. Perhaps document in a common location and also parse the property in common code. Rob