* [PATCH] PCI: mediatek: Remove surplus return from a void function
@ 2019-08-25 22:10 Krzysztof Wilczynski
2019-08-27 23:11 ` Bjorn Helgaas
0 siblings, 1 reply; 2+ messages in thread
From: Krzysztof Wilczynski @ 2019-08-25 22:10 UTC (permalink / raw)
To: Matthias Brugger
Cc: Bjorn Helgaas, Ryder Lee, Lorenzo Pieralisi, linux-pci,
linux-kernel, linux-arm-kernel, linux-mediatek
Remove unnecessary empty return statement at the
end of a void function mtk_pcie_intr_handler() in
the drivers/pci/controller/pcie-mediatek.c.
The surplus return statement was added as part of
the work in commit 42fe2f91b4eb ("PCI: mediatek:
Implement chained IRQ handling setup").
Signed-off-by: Krzysztof Wilczynski <kw@linux.com>
---
drivers/pci/controller/pcie-mediatek.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/pci/controller/pcie-mediatek.c b/drivers/pci/controller/pcie-mediatek.c
index 3eaa7081ab2a..626a7c352dfd 100644
--- a/drivers/pci/controller/pcie-mediatek.c
+++ b/drivers/pci/controller/pcie-mediatek.c
@@ -635,8 +635,6 @@ static void mtk_pcie_intr_handler(struct irq_desc *desc)
}
chained_irq_exit(irqchip, desc);
-
- return;
}
static int mtk_pcie_setup_irq(struct mtk_pcie_port *port,
--
2.22.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] PCI: mediatek: Remove surplus return from a void function
2019-08-25 22:10 [PATCH] PCI: mediatek: Remove surplus return from a void function Krzysztof Wilczynski
@ 2019-08-27 23:11 ` Bjorn Helgaas
0 siblings, 0 replies; 2+ messages in thread
From: Bjorn Helgaas @ 2019-08-27 23:11 UTC (permalink / raw)
To: Krzysztof Wilczynski
Cc: Matthias Brugger, Ryder Lee, Lorenzo Pieralisi, linux-pci,
linux-kernel, linux-arm-kernel, linux-mediatek
On Mon, Aug 26, 2019 at 12:10:39AM +0200, Krzysztof Wilczynski wrote:
> Remove unnecessary empty return statement at the
> end of a void function mtk_pcie_intr_handler() in
> the drivers/pci/controller/pcie-mediatek.c.
>
> The surplus return statement was added as part of
> the work in commit 42fe2f91b4eb ("PCI: mediatek:
> Implement chained IRQ handling setup").
>
> Signed-off-by: Krzysztof Wilczynski <kw@linux.com>
I squashed this together with the other patch doing the same thing.
If it causes any conflict with Lorenzo's branches, I'll resolve it
when merging.
> ---
> drivers/pci/controller/pcie-mediatek.c | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/drivers/pci/controller/pcie-mediatek.c b/drivers/pci/controller/pcie-mediatek.c
> index 3eaa7081ab2a..626a7c352dfd 100644
> --- a/drivers/pci/controller/pcie-mediatek.c
> +++ b/drivers/pci/controller/pcie-mediatek.c
> @@ -635,8 +635,6 @@ static void mtk_pcie_intr_handler(struct irq_desc *desc)
> }
>
> chained_irq_exit(irqchip, desc);
> -
> - return;
> }
>
> static int mtk_pcie_setup_irq(struct mtk_pcie_port *port,
> --
> 2.22.1
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-08-27 23:11 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-08-25 22:10 [PATCH] PCI: mediatek: Remove surplus return from a void function Krzysztof Wilczynski
2019-08-27 23:11 ` 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).