diff mbox

[2/2] arm64: Kconfig: select HISILICON_IRQ_MBIGEN only if PCI is selected

Message ID 1470234574-19027-2-git-send-email-sudeep.holla@arm.com (mailing list archive)
State New, archived
Headers show

Commit Message

Sudeep Holla Aug. 3, 2016, 2:29 p.m. UTC
Even when PCI is disabled, ARCH_HISI selects HISILICON_IRQ_MBIGEN
triggerring the following config warning:

warning: (ARM64 && HISILICON_IRQ_MBIGEN) selects ARM_GIC_V3_ITS which
	has unmet direct dependencies (PCI && PCI_MSI)

This patch makes selection of HISILICON_IRQ_MBIGEN conditional on PCI.

Cc: Ma Jun <majun258@huawei.com>
Cc: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
---
 arch/arm64/Kconfig.platforms | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--
2.7.4

Comments

Arnd Bergmann Aug. 10, 2016, 8:51 p.m. UTC | #1
On Wednesday, August 3, 2016 3:29:34 PM CEST Sudeep Holla wrote:
> Even when PCI is disabled, ARCH_HISI selects HISILICON_IRQ_MBIGEN
> triggerring the following config warning:
> 
> warning: (ARM64 && HISILICON_IRQ_MBIGEN) selects ARM_GIC_V3_ITS which
>         has unmet direct dependencies (PCI && PCI_MSI)
> 
> This patch makes selection of HISILICON_IRQ_MBIGEN conditional on PCI.

I think strictly speaking HISILICON_IRQ_MBIGEN should work without PCI,
but that is only a theoretical setup, and the patch seems to be the
simplest way to avoid the warning.

Applied both patches to fixes, sorry for the delay.

	Arnd
majun (F) Aug. 11, 2016, 1:26 a.m. UTC | #2
在 2016/8/11 4:51, Arnd Bergmann 写道:
> On Wednesday, August 3, 2016 3:29:34 PM CEST Sudeep Holla wrote:
>> Even when PCI is disabled, ARCH_HISI selects HISILICON_IRQ_MBIGEN
>> triggerring the following config warning:
>>
>> warning: (ARM64 && HISILICON_IRQ_MBIGEN) selects ARM_GIC_V3_ITS which
>>         has unmet direct dependencies (PCI && PCI_MSI)
>>
>> This patch makes selection of HISILICON_IRQ_MBIGEN conditional on PCI.
> 
> I think strictly speaking HISILICON_IRQ_MBIGEN should work without PCI,
> but that is only a theoretical setup, and the patch seems to be the
> simplest way to avoid the warning.
> 
> Applied both patches to fixes, sorry for the delay.
> 
sorry for late response, I just noticed this patch.

You are right, HISILICON_IRQ_MBIGEN uses the platform msi and has
no relation with pci.

> 	Arnd
> 
>
diff mbox

Patch

diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
index b0e6850aa3e4..41d82b327e6a 100644
--- a/arch/arm64/Kconfig.platforms
+++ b/arch/arm64/Kconfig.platforms
@@ -66,7 +66,7 @@  config ARCH_LG1K
 config ARCH_HISI
 	bool "Hisilicon SoC Family"
 	select ARM_TIMER_SP804
-	select HISILICON_IRQ_MBIGEN
+	select HISILICON_IRQ_MBIGEN if PCI
 	help
 	  This enables support for Hisilicon ARMv8 SoC family