Re: [PATCH] PCI: artpec6: Remove surplus break statement after return

[Bjorn Helgaas <helgaas@kernel.org>]

On Thu, Jul 01, 2021 at 07:16:40PM +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 artpec6_add_pcie_ep() has been removed and the call
> to the dw_pcie_ep_init() has been moved into artpec6_pcie_probe().
> 
> This change left a break statement behind that is not needed any more as
> as the function artpec6_pcie_probe() return immediately after making
> a call to dw_pcie_ep_init().
> 
> Thus remove this surplus break statement that became a dead code.
> 
> Signed-off-by: Krzysztof Wilczyński <kw@linux.com>

According to

  $ git grep -n -A1 "return.*;" drivers/pci

there's at least one more instance in
drivers/pci/controller/dwc/pcie-designware-plat.c.

> ---
>  drivers/pci/controller/dwc/pcie-artpec6.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/pci/controller/dwc/pcie-artpec6.c b/drivers/pci/controller/dwc/pcie-artpec6.c
> index 597c282f586c..739871bece75 100644
> --- a/drivers/pci/controller/dwc/pcie-artpec6.c
> +++ b/drivers/pci/controller/dwc/pcie-artpec6.c
> @@ -445,7 +445,6 @@ static int artpec6_pcie_probe(struct platform_device *pdev)
>  		pci->ep.ops = &pcie_ep_ops;
>  
>  		return dw_pcie_ep_init(&pci->ep);
> -		break;
>  	}

Not related to your patch, but I'm not really a fan of the block here
(needed because of the local "u32 val" declaration) because we end up
with two close braces at the same indent level.  I'd rather declare
the variable at the top with the other local variables and dispense
with the braces.

>  	default:
>  		dev_err(dev, "INVALID device type %d\n", artpec6_pcie->mode);
> -- 
> 2.32.0
>