diff options
| author | Fabio Estevam <fabio.estevam@freescale.com> | 2015-08-20 01:31:24 -0500 |
|---|---|---|
| committer | Bjorn Helgaas <bhelgaas@google.com> | 2015-08-20 01:31:24 -0500 |
| commit | 68ebb7ce395c6d8ca99163911ec384853d4f88ad (patch) | |
| tree | 63586d7b843ccfc4e906b8398784ed3ef1a0e9bc | |
| parent | bc0195aad0daa2ad5b0d76cce22b167bc3435590 (diff) | |
PCI: spear: Use BUG_ON() instead of condition followed by BUG()
Use BUG_ON() instead of an if condition followed by BUG().
The semantic patch that makes this change is available in
scripts/coccinelle/misc/bugon.cocci.
Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
CC: Pratyush Anand <pratyush.anand@gmail.com>
| -rw-r--r-- | drivers/pci/host/pcie-spear13xx.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/pci/host/pcie-spear13xx.c b/drivers/pci/host/pcie-spear13xx.c index c49fbdc0f6e4..98d2683181bc 100644 --- a/drivers/pci/host/pcie-spear13xx.c +++ b/drivers/pci/host/pcie-spear13xx.c @@ -223,8 +223,7 @@ static irqreturn_t spear13xx_pcie_irq_handler(int irq, void *arg) status = readl(&app_reg->int_sts); if (status & MSI_CTRL_INT) { - if (!IS_ENABLED(CONFIG_PCI_MSI)) - BUG(); + BUG_ON(!IS_ENABLED(CONFIG_PCI_MSI)); dw_handle_msi_irq(pp); } |