PCI: ibmphp: Remove unneeded break
authorBjorn Helgaas <bhelgaas@google.com>
Thu, 5 Nov 2020 14:52:30 +0000 (08:52 -0600)
committerBjorn Helgaas <bhelgaas@google.com>
Fri, 20 Nov 2020 17:17:55 +0000 (11:17 -0600)
A break is not needed if it is preceded by a return.

Based on Tom Rix's treewide patch; this instance extracted from Joe
Perches' list.

Link: https://lore.kernel.org/r/20201017160928.12698-1-trix@redhat.com
Link: https://lore.kernel.org/r/f530b7aeecbbf9654b4540cfa20023a4c2a11889.camel@perches
.com
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Cc: Tom Rix <trix@redhat.com>
Cc: Joe Perches <joe@perches.com>
drivers/pci/hotplug/ibmphp_pci.c

index e22d023..754c3f2 100644 (file)
@@ -294,7 +294,6 @@ int ibmphp_configure_card(struct pci_func *func, u8 slotno)
                                default:
                                        err("MAJOR PROBLEM!!!!, header type not supported? %x\n", hdr_type);
                                        return -ENXIO;
-                                       break;
                        }       /* end of switch */
                }       /* end of valid device */
        }       /* end of for */
@@ -1509,7 +1508,6 @@ static int unconfigure_boot_card(struct slot *slot_cur)
                                default:
                                        err("MAJOR PROBLEM!!!! Cannot read device's header\n");
                                        return -1;
-                                       break;
                        }       /* end of switch */
                }       /* end of valid device */
        }       /* end of for */