diff mbox series

[v1,1/2] bcm47xx: Get rid of redundant 'else'

Message ID 20211013134940.83295-1-andriy.shevchenko@linux.intel.com (mailing list archive)
State Accepted
Commit a274bdbdfcf7d0631004b20c03d90bbfed70c9d2
Headers show
Series [v1,1/2] bcm47xx: Get rid of redundant 'else' | expand

Commit Message

Andy Shevchenko Oct. 13, 2021, 1:49 p.m. UTC
In the snipped like

	if (...)
		return ...;
	else

the 'else' is redundant. Get rid of it.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 arch/mips/pci/pci-bcm47xx.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

Comments

Thomas Bogendoerfer Oct. 18, 2021, 4:17 p.m. UTC | #1
On Wed, Oct 13, 2021 at 04:49:39PM +0300, Andy Shevchenko wrote:
> In the snipped like
> 
> 	if (...)
> 		return ...;
> 	else
> 
> the 'else' is redundant. Get rid of it.
> 
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> ---
>  arch/mips/pci/pci-bcm47xx.c | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)

applied to mips-next.

Thomas.
diff mbox series

Patch

diff --git a/arch/mips/pci/pci-bcm47xx.c b/arch/mips/pci/pci-bcm47xx.c
index 230d7dd273e2..e8ff8e32a729 100644
--- a/arch/mips/pci/pci-bcm47xx.c
+++ b/arch/mips/pci/pci-bcm47xx.c
@@ -93,12 +93,10 @@  int pcibios_plat_dev_init(struct pci_dev *dev)
 #ifdef CONFIG_BCM47XX_SSB
 	if (bcm47xx_bus_type ==	 BCM47XX_BUS_TYPE_SSB)
 		return bcm47xx_pcibios_plat_dev_init_ssb(dev);
-	else
 #endif
 #ifdef CONFIG_BCM47XX_BCMA
 	if  (bcm47xx_bus_type ==  BCM47XX_BUS_TYPE_BCMA)
 		return bcm47xx_pcibios_plat_dev_init_bcma(dev);
-	else
 #endif
-		return 0;
+	return 0;
 }