From nobody Sat May 10 06:44:46 2025 Delivered-To: importer2@patchew.org Received-SPF: pass (zohomail.com: domain of vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; envelope-from=linux-kernel-owner@vger.kernel.org; helo=vger.kernel.org; Authentication-Results: mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass(p=none dis=none) header.from=kernel.org ARC-Seal: i=1; a=rsa-sha256; t=1612336018; cv=none; d=zohomail.com; s=zohoarc; b=PSheS1u3Bgoti2beqzpewv82fkjUzCjAJF6+a2H76GEyEE7Dqe4qA11meQUh6gkkflCVlb/NFDrvIYhvUnd8PiKwv/KAq9pbGce3vUVvjwQrB5/HvsDKFBjSX43XgvqFUpFK9hfU1mEW/h1G3/5knPqpLduNXD3u4tEvZsYEhe8= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1612336018; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Id:MIME-Version:Message-ID:References:Sender:Subject:To; bh=s++Lce4tBDt95WchiQ565kjfX2YvMRBE1SfQ2ezVo0A=; b=SsTIAjZ4o5icijZNmbL9o0/bR32hZUb0AFQKlLGFSnacOhNnot8RoRgL9AM6G9pRbtlaI+QtgzKjIsiGmOCtUp1bI2NcwF/Pn1IY7irQYkzAlJLU4ovS90IAoYY3YMJLhnBxpOFNN5sDEycgOsHRoLD4qSM3ecOGFx+SPLbAYIc= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass header.from= (p=none dis=none) header.from= Return-Path: Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mx.zohomail.com with SMTP id 1612336018262230.782475696142; Tue, 2 Feb 2021 23:06:58 -0800 (PST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231667AbhBCHGr (ORCPT ); Wed, 3 Feb 2021 02:06:47 -0500 Received: from mail.kernel.org ([198.145.29.99]:35512 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231842AbhBCHDZ (ORCPT ); Wed, 3 Feb 2021 02:03:25 -0500 Received: by mail.kernel.org (Postfix) with ESMTPSA id 7BBB864F6A; Wed, 3 Feb 2021 07:02:00 +0000 (UTC) Received: by mail.kernel.org with local (Exim 4.94) (envelope-from ) id 1l7CAo-001CAZ-5N; Wed, 03 Feb 2021 08:01:58 +0100 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1612335720; bh=s0fWuTBl8hAo4L2qfW2NnlYEUKCtkScUbYwEi6kp9HY=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=ZxV86Y/IpuUgUWybL8qRFzEO9IkHmm64Wrv19xHBVtFs8mK6n447N6yieivwpbHCl j23AP1npMuCjNkXvG1qixmLxsGnnESp3x52YUx1t1kadUOELiqUt66GgBX6XfAyGBk eFeHLa4fDaEKFLYBwUKA9ANI0GDZHDSTGTofxcwyczAdLtrpTAZiT7Xq+y0DeU6Jxk 8fKSOuvE3IXQof3zaBmzEMAzXpEeI1uryPpimdaON47fDJXniZxJuiDBgNzw1mgbcm UVq12srYXOGTT1zCK7OZP1isYgIsBhB5+W9qqbuQ4FMubzDPknp5DhbibmdALJMUmY +m8K2dNxEGpVg== From: Mauro Carvalho Chehab Cc: Mauro Carvalho Chehab , Binghui Wang , Bjorn Helgaas , Lorenzo Pieralisi , Rob Herring , Xiaowei Song , linux-kernel@vger.kernel.org, linux-pci@vger.kernel.org Subject: [PATCH v2 05/11] PCI: dwc: pcie-kirin: simplify error handling logic Date: Wed, 3 Feb 2021 08:01:49 +0100 Message-Id: <6298de2ed4384c2a3f012e9dedc101ae5f184ab2.1612335031.git.mchehab+huawei@kernel.org> X-Mailer: git-send-email 2.29.2 In-Reply-To: References: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Sender: Mauro Carvalho Chehab To: unlisted-recipients:; (no To-header on input) Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-ZohoMail-DKIM: fail (Header signature does not verify) Content-Type: text/plain; charset="utf-8" Instead of returning -ENODEV when of_get_named_gpio() fails, make it return the actual error code. With that, there's no need anymore to check for -EPROBE_DEFER at kirin_pcie_probe(). Signed-off-by: Mauro Carvalho Chehab --- drivers/pci/controller/dwc/pcie-kirin.c | 16 ++++++---------- 1 file changed, 6 insertions(+), 10 deletions(-) diff --git a/drivers/pci/controller/dwc/pcie-kirin.c b/drivers/pci/controll= er/dwc/pcie-kirin.c index 5925d2b345a8..f46a51ffd2c8 100644 --- a/drivers/pci/controller/dwc/pcie-kirin.c +++ b/drivers/pci/controller/dwc/pcie-kirin.c @@ -320,7 +320,7 @@ static long kirin960_pcie_get_resource(struct kirin_pci= e *kirin_pcie, kirin_pcie->gpio_id_reset[0] =3D of_get_named_gpio(dev->of_node, "reset-gpios", 0); if (kirin_pcie->gpio_id_reset[0] < 0) - return -ENODEV; + return kirin_pcie->gpio_id_reset[0]; =20 return 0; } @@ -354,22 +354,22 @@ static long kirin970_pcie_get_resource(struct kirin_p= cie *kirin_pcie, kirin_pcie->gpio_id_reset[0] =3D of_get_named_gpio(dev->of_node, "switch,reset-gpios", 0); if (kirin_pcie->gpio_id_reset[0] < 0) - return -ENODEV; + return kirin_pcie->gpio_id_reset[0]; =20 kirin_pcie->gpio_id_reset[1] =3D of_get_named_gpio(dev->of_node, "eth,reset-gpios", 0); if (kirin_pcie->gpio_id_reset[1] < 0) - return -ENODEV; + return kirin_pcie->gpio_id_reset[1]; =20 kirin_pcie->gpio_id_reset[2] =3D of_get_named_gpio(dev->of_node, "m_2,reset-gpios", 0); if (kirin_pcie->gpio_id_reset[2] < 0) - return -ENODEV; + return kirin_pcie->gpio_id_reset[2]; =20 kirin_pcie->gpio_id_reset[3] =3D of_get_named_gpio(dev->of_node, "mini1,reset-gpios", 0); if (kirin_pcie->gpio_id_reset[3] < 0) - return -ENODEV; + return kirin_pcie->gpio_id_reset[3]; =20 ret =3D devm_gpio_request(dev, kirin_pcie->gpio_id_reset[0], "pcie_switch_reset"); @@ -1148,11 +1148,7 @@ static int kirin_pcie_probe(struct platform_device *= pdev) if (ret) return ret; =20 - kirin_pcie->gpio_id_reset[0] =3D of_get_named_gpio(dev->of_node, - "reset-gpios", 0); - if (kirin_pcie->gpio_id_reset[0] =3D=3D -EPROBE_DEFER) { - return -EPROBE_DEFER; - } else if (!gpio_is_valid(kirin_pcie->gpio_id_reset[0])) { + if (!gpio_is_valid(kirin_pcie->gpio_id_reset[0])) { dev_err(dev, "unable to get a valid gpio pin\n"); return -ENODEV; } --=20 2.29.2