diff mbox

[1/1] irqchip: exynos: Remove hardcoded irqbase

Message ID 1382007881-1161-1-git-send-email-sachin.kamat@linaro.org (mailing list archive)
State New, archived
Headers show

Commit Message

Sachin Kamat Oct. 17, 2013, 11:04 a.m. UTC
Now that Exynos platforms are DT only, hard-coded irqbase is not
necessary.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
 drivers/irqchip/exynos-combiner.c |    9 +--------
 1 file changed, 1 insertion(+), 8 deletions(-)

Comments

Tomasz Figa Oct. 17, 2013, 5:55 p.m. UTC | #1
Hi Sachin,

On Thursday 17 of October 2013 16:34:41 Sachin Kamat wrote:
> Now that Exynos platforms are DT only, hard-coded irqbase is not
> necessary.
> 
> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
> ---
>  drivers/irqchip/exynos-combiner.c |    9 +--------
>  1 file changed, 1 insertion(+), 8 deletions(-)

A patch like this has been already submitted by Chander Kashyap and it was 
supposed to be already applied by Kukjin:

http://thread.gmane.org/gmane.linux.kernel.samsung-soc/23312/focus=23442

I'm not sure why it is not yet in Samsung tree, so I pinged original 
thread.

Best regards,
Tomasz
diff mbox

Patch

diff --git a/drivers/irqchip/exynos-combiner.c b/drivers/irqchip/exynos-combiner.c
index 868ed40..08538df 100644
--- a/drivers/irqchip/exynos-combiner.c
+++ b/drivers/irqchip/exynos-combiner.c
@@ -207,7 +207,7 @@  static int __init combiner_of_init(struct device_node *np,
 {
 	void __iomem *combiner_base;
 	unsigned int max_nr = 20;
-	int irq_base = -1;
+	int irq_base = 0;
 
 	combiner_base = of_iomap(np, 0);
 	if (!combiner_base) {
@@ -221,13 +221,6 @@  static int __init combiner_of_init(struct device_node *np,
 			__func__, max_nr);
 	}
 
-	/* 
-	 * FIXME: This is a hardwired COMBINER_IRQ(0,0). Once all devices
-	 * get their IRQ from DT, remove this in order to get dynamic
-	 * allocation.
-	 */
-	irq_base = 160;
-
 	combiner_init(combiner_base, np, max_nr, irq_base);
 
 	return 0;