summaryrefslogtreecommitdiffstats
path: root/drivers/pci
diff options
context:
space:
mode:
authorFabio Estevam <fabio.estevam@freescale.com>2015-08-20 01:31:24 -0500
committerBjorn Helgaas <bhelgaas@google.com>2015-08-20 01:31:24 -0500
commit68ebb7ce395c6d8ca99163911ec384853d4f88ad (patch)
tree63586d7b843ccfc4e906b8398784ed3ef1a0e9bc /drivers/pci
parentbc0195aad0daa2ad5b0d76cce22b167bc3435590 (diff)
downloadlinux-68ebb7ce395c6d8ca99163911ec384853d4f88ad.tar.bz2
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>
Diffstat (limited to 'drivers/pci')
-rw-r--r--drivers/pci/host/pcie-spear13xx.c3
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);
}