[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20181226022236.71682-1-kjlu@umn.edu>
Date: Tue, 25 Dec 2018 20:22:36 -0600
From: Kangjie Lu <kjlu@....edu>
To: kjlu@....edu
Cc: pakki001@....edu, Jingoo Han <jingoohan1@...il.com>,
Lorenzo Pieralisi <lorenzo.pieralisi@....com>,
Bjorn Helgaas <bhelgaas@...gle.com>,
Kukjin Kim <kgene@...nel.org>,
Krzysztof Kozlowski <krzk@...nel.org>,
linux-pci@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
linux-samsung-soc@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: [PATCH] pci: dwc: add a check for resetting gpio
devm_gpio_request_one() could fail. The fix checks its status and issues
an error if it fails.
Signed-off-by: Kangjie Lu <kjlu@....edu>
---
drivers/pci/controller/dwc/pci-exynos.c | 9 ++++++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/drivers/pci/controller/dwc/pci-exynos.c b/drivers/pci/controller/dwc/pci-exynos.c
index cee5f2f590e2..e3a045e215d2 100644
--- a/drivers/pci/controller/dwc/pci-exynos.c
+++ b/drivers/pci/controller/dwc/pci-exynos.c
@@ -226,9 +226,12 @@ static void exynos_pcie_assert_reset(struct exynos_pcie *ep)
struct dw_pcie *pci = ep->pci;
struct device *dev = pci->dev;
- if (ep->reset_gpio >= 0)
- devm_gpio_request_one(dev, ep->reset_gpio,
- GPIOF_OUT_INIT_HIGH, "RESET");
+ if (ep->reset_gpio >= 0) {
+ if (devm_gpio_request_one(dev, ep->reset_gpio,
+ GPIOF_OUT_INIT_HIGH, "RESET"))
+ dev_err(dev, "Failed requesting reset gpio %d\n",
+ ep->reset_gpio);
+ }
}
static int exynos_pcie_establish_link(struct exynos_pcie *ep)
--
2.17.2 (Apple Git-113)
Powered by blists - more mailing lists