Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Thu, 26 Nov 2020 12:34:47 +0100
From:   Thierry Reding <thierry.reding@gmail.com>
To:     Vidya Sagar <vidyas@nvidia.com>
Cc:     lorenzo.pieralisi@arm.com, robh+dt@kernel.org, bhelgaas@google.com,
        jonathanh@nvidia.com, amanharitsh123@gmail.com,
        dinghao.liu@zju.edu.cn, kw@linux.com, linux-pci@vger.kernel.org,
        linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org,
        kthota@nvidia.com, mmaddireddy@nvidia.com, sagar.tv@gmail.com
Subject: Re: [PATCH V4 5/6] PCI: tegra: Check return value of
 tegra_pcie_init_controller()
Message-ID: <X7+S1wSqUCKW+/x6@ulmo>
References: <20201109171937.28326-1-vidyas@nvidia.com>
 <20201109171937.28326-6-vidyas@nvidia.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
        protocol="application/pgp-signature"; boundary="85sL3q+8YeXPZSg7"
Content-Disposition: inline
In-Reply-To: <20201109171937.28326-6-vidyas@nvidia.com>
User-Agent: Mutt/2.0.2 (d9268908) (2020-11-20)
Precedence: bulk


--85sL3q+8YeXPZSg7
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Mon, Nov 09, 2020 at 10:49:36PM +0530, Vidya Sagar wrote:
> The return value of tegra_pcie_init_controller() must be checked before
> PCIe link up check and registering debugfs entries subsequently as it
> doesn't make sense to do these when the controller initialization itself
> has failed.
>=20
> Signed-off-by: Vidya Sagar <vidyas@nvidia.com>
> ---
> V4:
> * None
>=20
> V3:
> * New patch in this series
>=20
>  drivers/pci/controller/dwc/pcie-tegra194.c | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)

Tested-by: Thierry Reding <treding@nvidia.com>
Acked-by: Thierry Reding <treding@nvidia.com>

--85sL3q+8YeXPZSg7
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIzBAABCAAdFiEEiOrDCAFJzPfAjcif3SOs138+s6EFAl+/ktcACgkQ3SOs138+
s6ELhBAAj7OWOMN2zAgZ9IQL58MGpBP2DKECtOGo0ec01e+OGIZ8KmEEQpmS0FPS
atnvTy9HKTY2fhwG5hrElFgolhDMIevSrw14GQry0xi0DHKsKa2jV0u4wCEq5nja
lNUuE7SSeHu/Tph2mtDLWxHJgsDzwwQbqQPsKh3mWgd4bmRXBN72mmq3MP7u7Fk1
w8CIh83Unc/BAYXBYqNhiixtQzYxUfYZYbXEHsA44L3DRF+QaaqPrt21G+w/kUbV
B1/LvCClBcK6H0KZelzDEDbxCzW+aHXlmaLfsSoPblHYu7czjcACoBIms+/AaWSu
+AQek4cJ25nUWkrEFz7vWUf93t4+7oSffMKEglHT9A+arT/bKhiiZnKCRsTBCxv9
mai6U+ZPiMoSufiSNAeFsiWpSaXpwObuXeNVJKSaXZbtdsrthCSxhpiGpzUPYT1B
ycmAHu2pnvI0C7rA2FtPObwaP6Pmc16KFt3vtYH6AV2pFdoMz4ZvhoxT7GUye4+P
StBKwBrRI+V552peX3dRrWGSOMATCVqGKVa+3N6X42CaWNzBl/AqP0IUWAzshZnh
k36GtJmszAHdVfv5+2T2q/fQrw8HD76MkRrMVKeOKoeQJESthDjitEbLGcS8xEhc
+eUzFrLYTGNgMBoefax5NpzuNpZBQrpdnZTlyIiymC375uE5SG0=
=D9Rv
-----END PGP SIGNATURE-----

--85sL3q+8YeXPZSg7--