[PATCH] mfd: tps65217: Drop call to irq_set_parent()

From	Guenter Roeck <linux@roeck-us.net>
Newsgroups	linux.kernel
Subject	[PATCH] mfd: tps65217: Drop call to irq_set_parent()
Date	2016-10-14 23:30 +0200
Message-ID	<ssidb-6Lt-5@gated-at.bofh.it> (permalink)
Organization	linux.* mail to news gateway

Show all headers | View raw

The call to irq_set_parent() causes the following build error if tps65217
is built as module.

ERROR: ".irq_set_parent" [drivers/mfd/tps65217.ko] undefined!

The problem was introduced with commit 6556bdacf646f ("mfd: tps65217: Add
support for IRQs").

The author states: "I have added irq_set_parent() similarly as in
drivers/base/regmap/regmap-irq.c. But to be honest I am not sure what it
really does in case of tps65217."

So let's drop it.

Fixes: 6556bdacf646f ("mfd: tps65217: Add support for IRQs")
Cc: Marcin Niestroj <m.niestroj@grinn-global.com>
Cc: Arnd Bergmann <arnd@arndb.de>
Cc: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
---
 drivers/mfd/tps65217.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/mfd/tps65217.c b/drivers/mfd/tps65217.c
index 9a4d8684dd32..2a57b444859c 100644
--- a/drivers/mfd/tps65217.c
+++ b/drivers/mfd/tps65217.c
@@ -170,7 +170,6 @@ static int tps65217_irq_map(struct irq_domain *h, unsigned int virq,
 	irq_set_chip_data(virq, tps);
 	irq_set_chip_and_handler(virq, &tps65217_irq_chip, handle_edge_irq);
 	irq_set_nested_thread(virq, 1);
-	irq_set_parent(virq, tps->irq);
 	irq_set_noprobe(virq);
 
 	return 0;
-- 
2.5.0

Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread

Thread

[PATCH] mfd: tps65217: Drop call to irq_set_parent() Guenter Roeck <linux@roeck-us.net> - 2016-10-14 23:30 +0200
  Re: [PATCH] mfd: tps65217: Drop call to irq_set_parent() Thomas Gleixner <tglx@linutronix.de> - 2016-10-26 14:50 +0200
    Re: [PATCH] mfd: tps65217: Drop call to irq_set_parent() Lee Jones <lee.jones@linaro.org> - 2016-10-26 15:00 +0200
      Re: [PATCH] mfd: tps65217: Drop call to irq_set_parent() Guenter Roeck <linux@roeck-us.net> - 2016-10-26 15:30 +0200
  Re: [PATCH] mfd: tps65217: Drop call to irq_set_parent() Lee Jones <lee.jones@linaro.org> - 2016-10-26 15:10 +0200

csiph-web