diff options
| author | RAGHU Halharvi <raghuhack78@gmail.com> | 2018-07-17 12:20:09 +0200 |
|---|---|---|
| committer | Thomas Gleixner <tglx@linutronix.de> | 2018-07-17 13:35:44 +0200 |
| commit | d91cfeb0aa79445fcfa9f523a5b57c5e9f4113ec (patch) | |
| tree | 79a47e8f19747373471a978a1d8dbad330bea707 /kernel/irq | |
| parent | genirq: Synchronize only with single thread on free_irq() (diff) | |
| download | linux-d91cfeb0aa79445fcfa9f523a5b57c5e9f4113ec.tar.xz linux-d91cfeb0aa79445fcfa9f523a5b57c5e9f4113ec.zip | |
genirq: Remove redundant NULL pointer check in __free_irq()
The NULL pointer check in __free_irq() triggers a 'dereference before NULL
pointer check' warning in static code analysis. It turns out that the check
is redundant because all callers have a NULL pointer check already.
Remove it.
Signed-off-by: RAGHU Halharvi <raghuhack78@gmail.com>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Link: https://lkml.kernel.org/r/20180717102009.7708-1-raghuhack78@gmail.com
Diffstat (limited to 'kernel/irq')
| -rw-r--r-- | kernel/irq/manage.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/kernel/irq/manage.c b/kernel/irq/manage.c index 1f8be33572a7..a66c58f91bff 100644 --- a/kernel/irq/manage.c +++ b/kernel/irq/manage.c @@ -1570,9 +1570,6 @@ static struct irqaction *__free_irq(struct irq_desc *desc, void *dev_id) WARN(in_interrupt(), "Trying to free IRQ %d from IRQ context!\n", irq); - if (!desc) - return NULL; - mutex_lock(&desc->request_mutex); chip_bus_lock(desc); raw_spin_lock_irqsave(&desc->lock, flags); |