Received: by 2002:a05:6a10:16a7:0:0:0:0 with SMTP id gp39csp1146632pxb; Wed, 4 Nov 2020 00:56:20 -0800 (PST) X-Google-Smtp-Source: ABdhPJyFK+4oa0/fKwug/RZHbhdxMkkMeRLO98EuAs33/w0+plZGXwb3QcX2u2aBEGr/99YNDE5S X-Received: by 2002:a17:906:7cc:: with SMTP id m12mr5174326ejc.365.1604480180074; Wed, 04 Nov 2020 00:56:20 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1604480180; cv=none; d=google.com; s=arc-20160816; b=JpES+kD0QQ9XHTMBAi00gwkr50wLn5uz2Rf4IBIkYZUufD9C+BaptYrlIy66SXo5XU TjL5YZC9j2kUPCgra5ddHrSdJ168VMef/oeoeSMgoRVxQASv7SJXdp6R1yBj20FcGxg3 Z3yRaXiBxP7EVVDIHEMfNt9V+PCaWnuM0tZumlAPIUDr66AsSJH7e1hUbYYCGhzzBBtc Sx1IULPPbzkbYPI9r2f1FYUN81iUXRhJztwZFFsPheYRuuM1fNdQw8X6YxR9SgAL/04U 7Y52GKIINR6TiH7eTEzerPlxikWoKwwslzqq4MHjXOos7qOdO7bOmfmmy/S98k9ZuaWJ GnBQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:dkim-signature:mime-version:references :in-reply-to:message-id:date:subject:cc:to:from; bh=n8XdfeCZduO7ogcyhJVAlYbsvTFqkbs1wknNZ3ixbf4=; b=cJQ+uLoqpWzZJJ8jk27ybnpBiTwZSBlGQA5AxZQk4l9BrS6Wf6NvoxC4UgoKd0G4VU lMVfnbkpMyWXxQ9CMQOiTysL0o2HqAaPHWeocEE9enMJDeNQT39QjOe8Eu2yXw4prYM2 k7stvcmBCmZM99DDT9lDfLcphSmj3sFk7g2PidmG+hxxK9QQjmnHxJLcePAEtgdAqQ3G iBxUiobXekDOAFj5XrbUr7bIIYDaxstxlngts1ZRKkR0hBLUHshMVBuyz353o2NTYOVx RgPnmpI7WL2Yq21LeO9eFF06TKetwFoop+SDv4UgtKfhdVchaHqIQg0TAJx06VEa2omu NLEQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@nvidia.com header.s=n1 header.b=pZrMuYJE; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=nvidia.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id x2si1264711ejy.195.2020.11.04.00.55.57; Wed, 04 Nov 2020 00:56:20 -0800 (PST) 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; Authentication-Results: mx.google.com; dkim=pass header.i=@nvidia.com header.s=n1 header.b=pZrMuYJE; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=nvidia.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728960AbgKDIwq (ORCPT + 99 others); Wed, 4 Nov 2020 03:52:46 -0500 Received: from hqnvemgate24.nvidia.com ([216.228.121.143]:2695 "EHLO hqnvemgate24.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728886AbgKDIwp (ORCPT ); Wed, 4 Nov 2020 03:52:45 -0500 Received: from hqmail.nvidia.com (Not Verified[216.228.121.13]) by hqnvemgate24.nvidia.com (using TLS: TLSv1.2, AES256-SHA) id ; Wed, 04 Nov 2020 00:52:47 -0800 Received: from HQMAIL105.nvidia.com (172.20.187.12) by HQMAIL105.nvidia.com (172.20.187.12) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Wed, 4 Nov 2020 08:52:42 +0000 Received: from vidyas-desktop.nvidia.com (10.124.1.5) by mail.nvidia.com (172.20.187.12) with Microsoft SMTP Server id 15.0.1473.3 via Frontend Transport; Wed, 4 Nov 2020 08:52:37 +0000 From: Vidya Sagar To: , , , , , , , CC: , , , , , , Subject: [PATCH V3 5/5] PCI: tegra: Check return value of tegra_pcie_init_controller() Date: Wed, 4 Nov 2020 14:20:18 +0530 Message-ID: <20201104085018.13021-6-vidyas@nvidia.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20201104085018.13021-1-vidyas@nvidia.com> References: <20201104085018.13021-1-vidyas@nvidia.com> X-NVConfidentiality: public MIME-Version: 1.0 Content-Type: text/plain DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nvidia.com; s=n1; t=1604479967; bh=n8XdfeCZduO7ogcyhJVAlYbsvTFqkbs1wknNZ3ixbf4=; h=From:To:CC:Subject:Date:Message-ID:X-Mailer:In-Reply-To: References:X-NVConfidentiality:MIME-Version:Content-Type; b=pZrMuYJElTBvismIA6kjwVHxaFE7FOuOChV1MmIorPgM+RhqDiJy8iiv9KlR5oVNW QylfQbhVrOJU/rAGAJ1+EfK4wzHIz1SnGUDFHuRQOnEd1rgytRVYZv0U5pkTh1fqGO YqSCoHqVq6vEGSApK/hxHN434Qb2o0SCkKN/iN8L7/1lU8ThikxdcfB4mV/J8b+cIM 9pYpoI4Ti1QYh6P3JOfOb1IH75RfqD6Q3CdviNkTzHPrXkimf2zwZpynPowkKl0jaS 35nqtU/5psp54DQ6LwFO0K/aJSXJg2rylUVTcU0BoRn2Lrb+BbG0di9y1FUBPJhE/t pK41dEitL4G3Q== Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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. Signed-off-by: Vidya Sagar --- V3: * New patch in this series drivers/pci/controller/dwc/pcie-tegra194.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/pci/controller/dwc/pcie-tegra194.c b/drivers/pci/controller/dwc/pcie-tegra194.c index 9be10c8953df..8c08998b9ce1 100644 --- a/drivers/pci/controller/dwc/pcie-tegra194.c +++ b/drivers/pci/controller/dwc/pcie-tegra194.c @@ -1579,7 +1579,11 @@ static int tegra_pcie_config_rp(struct tegra_pcie_dw *pcie) goto fail_pm_get_sync; } - tegra_pcie_init_controller(pcie); + ret = tegra_pcie_init_controller(pcie); + if (ret < 0) { + dev_err(dev, "Failed to initialize controller: %d\n", ret); + goto fail_pm_get_sync; + } pcie->link_state = tegra_pcie_dw_link_up(&pcie->pci); if (!pcie->link_state) { -- 2.17.1