| Message ID | 20210701210252.1638709-1-kw@linux.com |
|---|---|
| State | New |
| Headers | show |
| Series | PCI: dwc: Remove surplus break statement after return | expand |
On Thu, 1 Jul 2021 21:02:52 +0000, Krzysztof Wilczyński wrote: > As part of code refactoring completed in the commit a0fd361db8e5 ("PCI: > dwc: Move "dbi", "dbi2", and "addr_space" resource setup into common > code") the function dw_plat_add_pcie_ep() has been removed and the call > to the dw_pcie_ep_init() has been moved into dw_plat_pcie_probe(). > > This change left a break statement behind that is not needed any more as > as the function dw_plat_pcie_probe() returns immediately after making > a call to dw_pcie_ep_init(). > > [...] Applied to pci/dwc, thanks! [1/1] PCI: dwc: Remove surplus break statement after return https://git.kernel.org/lpieralisi/pci/c/2999568def Thanks, Lorenzo
diff --git a/drivers/pci/controller/dwc/pcie-designware-plat.c b/drivers/pci/controller/dwc/pcie-designware-plat.c index 9b397c807261..8851eb161a0e 100644 --- a/drivers/pci/controller/dwc/pcie-designware-plat.c +++ b/drivers/pci/controller/dwc/pcie-designware-plat.c @@ -164,7 +164,6 @@ static int dw_plat_pcie_probe(struct platform_device *pdev) pci->ep.ops = &pcie_ep_ops; return dw_pcie_ep_init(&pci->ep); - break; default: dev_err(dev, "INVALID device type %d\n", dw_plat_pcie->mode); }
As part of code refactoring completed in the commit a0fd361db8e5 ("PCI: dwc: Move "dbi", "dbi2", and "addr_space" resource setup into common code") the function dw_plat_add_pcie_ep() has been removed and the call to the dw_pcie_ep_init() has been moved into dw_plat_pcie_probe(). This change left a break statement behind that is not needed any more as as the function dw_plat_pcie_probe() returns immediately after making a call to dw_pcie_ep_init(). Thus remove this surplus break statement that became a dead code. Suggested-by: Bjorn Helgaas <bhelgaas@google.com> Signed-off-by: Krzysztof Wilczyński <kw@linux.com> --- drivers/pci/controller/dwc/pcie-designware-plat.c | 1 - 1 file changed, 1 deletion(-)