diff options
| author | Jeff Garzik <jeff@garzik.org> | 2007-10-20 13:45:57 +0200 |
|---|---|---|
| committer | David S. Miller <davem@sunset.davemloft.net> | 2007-10-22 11:59:44 +0200 |
| commit | ac019360fe311dd6aa11b358a02eb3a61675882e (patch) | |
| tree | dfbfeb253ba43c3142c8a06e22bfea5182d700f3 /drivers/bluetooth/btuart_cs.c | |
| parent | [Bluetooth] Add UART driver for Texas Instruments' BRF63xx chips (diff) | |
| download | linux-ac019360fe311dd6aa11b358a02eb3a61675882e.tar.xz linux-ac019360fe311dd6aa11b358a02eb3a61675882e.zip | |
[Bluetooth] Eliminate checks for impossible conditions in IRQ handler
Our info structure and info->hdev is always passed to the IRQ handler,
so we don't have to worry about these checks in every interrupt.
Leave a BUG_ON() just to help unwary programmers, but these could
probably be removed as well.
Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Diffstat (limited to 'drivers/bluetooth/btuart_cs.c')
| -rw-r--r-- | drivers/bluetooth/btuart_cs.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/bluetooth/btuart_cs.c b/drivers/bluetooth/btuart_cs.c index d7d2ea0d86a1..08f48d577aba 100644 --- a/drivers/bluetooth/btuart_cs.c +++ b/drivers/bluetooth/btuart_cs.c @@ -294,10 +294,7 @@ static irqreturn_t btuart_interrupt(int irq, void *dev_inst) int boguscount = 0; int iir, lsr; - if (!info || !info->hdev) { - BT_ERR("Call of irq %d for unknown device", irq); - return IRQ_NONE; - } + BUG_ON(!info->hdev); iobase = info->p_dev->io.BasePort1; |