Re: [PATCH] irq: fix a Kconfig dependency

From: David Rientjes
Date: Fri Jan 31 2014 - 15:46:10 EST


On Fri, 31 Jan 2014, Kamble, Nitin A wrote:

> > So, if this dependency is reasonable, it begs the question of why we have
> > CONFIG_GENERIC_IRQ_CHIP at all and shouldn't just make generic-chip.c
> > depend on CONFIG_IRQ_DOMAIN.
> Looking at the code there are cases where IRQ_DOMAIN functionality is used
> without using the GENERIC_IRQ_CHIP. So making generic-chip.c depend
> on CONFIG_IRQ_DOMAIN in the makefile will reduce the flexibility of using
> these config options separately.
>

Acked-by: David Rientjes <rientjes@xxxxxxxxxx>

Although the changelog should be updated since I get a couple problems
when trying to enable CONFIG_GENERIC_IRQ_CHIP without CONFIG_IRQ_DOMAIN on
Linus's latest tree:

kernel/irq/generic-chip.c:400:11: error: âirq_domain_xlate_onetwocellâ undeclared here (not in a function)
kernel/irq/generic-chip.c: In function âirq_gc_get_irq_dataâ:
kernel/irq/generic-chip.c:522:2: error: implicit declaration of function âirq_find_mappingâ [-Werror=implicit-function-declaration]