Commit 68ebb7ce authored Aug 20, 2015 by Fabio Estevam Committed by Bjorn Helgaas Aug 20, 2015

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>

parent bc0195aa

drivers/pci/host/pcie-spear13xx.c

+1 −2

Original line number	Diff line number	Diff line
		@@ -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);
		}

Admin message