* [PATCH] pci: controller: dwc: Remove dev_err use after platform_get_irq
@ 2019-10-28 20:21 Saurav Girepunje
2019-10-30 21:46 ` Bjorn Helgaas
0 siblings, 1 reply; 2+ messages in thread
From: Saurav Girepunje @ 2019-10-28 20:21 UTC (permalink / raw)
To: jingoohan1, lorenzo.pieralisi, andrew.murray, bhelgaas, kgene,
krzk, linux-pci, linux-arm-kernel, linux-samsung-soc,
linux-kernel
Cc: saurav.girepunje
Don't need dev_err() messages when platform_get_irq() fails now that
platform_get_irq() prints an error message itself when something goes
wrong.
Signed-off-by: Saurav Girepunje <saurav.girepunje@gmail.com>
---
drivers/pci/controller/dwc/pci-exynos.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/pci/controller/dwc/pci-exynos.c b/drivers/pci/controller/dwc/pci-exynos.c
index 14a6ba4067fb..2293b346d96a 100644
--- a/drivers/pci/controller/dwc/pci-exynos.c
+++ b/drivers/pci/controller/dwc/pci-exynos.c
@@ -403,7 +403,6 @@ static int __init exynos_add_pcie_port(struct exynos_pcie *ep,
pp->irq = platform_get_irq(pdev, 1);
if (pp->irq < 0) {
- dev_err(dev, "failed to get irq\n");
return pp->irq;
}
ret = devm_request_irq(dev, pp->irq, exynos_pcie_irq_handler,
@@ -416,7 +415,6 @@ static int __init exynos_add_pcie_port(struct exynos_pcie *ep,
if (IS_ENABLED(CONFIG_PCI_MSI)) {
pp->msi_irq = platform_get_irq(pdev, 0);
if (pp->msi_irq < 0) {
- dev_err(dev, "failed to get msi irq\n");
return pp->msi_irq;
}
}
--
2.20.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] pci: controller: dwc: Remove dev_err use after platform_get_irq
2019-10-28 20:21 [PATCH] pci: controller: dwc: Remove dev_err use after platform_get_irq Saurav Girepunje
@ 2019-10-30 21:46 ` Bjorn Helgaas
0 siblings, 0 replies; 2+ messages in thread
From: Bjorn Helgaas @ 2019-10-30 21:46 UTC (permalink / raw)
To: Saurav Girepunje
Cc: jingoohan1, lorenzo.pieralisi, andrew.murray, kgene, krzk,
linux-pci, linux-arm-kernel, linux-samsung-soc, linux-kernel,
saurav.girepunje
Please run "git log --oneline drivers/pci/controller/dwc/pci-exynos.c"
and make your subject line match, e.g.,
PCI: exynos: Remove dev_err() usage after platform_get_irq()
The body of that was copied from similar commits like:
fb5a35dbee8d pwm: Remove dev_err() usage after platform_get_irq()
9a7957d0c955 mmc: Remove dev_err() usage after platform_get_irq()
1df217868178 tty: Remove dev_err() usage after platform_get_irq()
It's nice when similar commits have similar subject lines.
In fact, this whole thing has been approached before:
https://lore.kernel.org/lkml/20190730181557.90391-32-swboyd@chromium.org/
That patch would have fixed many similar issues in PCI, but I don't
know what happened to it. Would you mind resurrecting that and fixing
the minor issues so we can do everything in PCI at once?
On Tue, Oct 29, 2019 at 01:51:44AM +0530, Saurav Girepunje wrote:
> Don't need dev_err() messages when platform_get_irq() fails now that
> platform_get_irq() prints an error message itself when something goes
> wrong.
>
> Signed-off-by: Saurav Girepunje <saurav.girepunje@gmail.com>
> ---
> drivers/pci/controller/dwc/pci-exynos.c | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/drivers/pci/controller/dwc/pci-exynos.c b/drivers/pci/controller/dwc/pci-exynos.c
> index 14a6ba4067fb..2293b346d96a 100644
> --- a/drivers/pci/controller/dwc/pci-exynos.c
> +++ b/drivers/pci/controller/dwc/pci-exynos.c
> @@ -403,7 +403,6 @@ static int __init exynos_add_pcie_port(struct exynos_pcie *ep,
>
> pp->irq = platform_get_irq(pdev, 1);
> if (pp->irq < 0) {
> - dev_err(dev, "failed to get irq\n");
> return pp->irq;
> }
> ret = devm_request_irq(dev, pp->irq, exynos_pcie_irq_handler,
> @@ -416,7 +415,6 @@ static int __init exynos_add_pcie_port(struct exynos_pcie *ep,
> if (IS_ENABLED(CONFIG_PCI_MSI)) {
> pp->msi_irq = platform_get_irq(pdev, 0);
> if (pp->msi_irq < 0) {
> - dev_err(dev, "failed to get msi irq\n");
> return pp->msi_irq;
> }
> }
> --
> 2.20.1
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-10-30 21:46 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-28 20:21 [PATCH] pci: controller: dwc: Remove dev_err use after platform_get_irq Saurav Girepunje
2019-10-30 21:46 ` Bjorn Helgaas
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).