diff mbox

pci: adding support for PCI_MSI_IRQ_DOMAIN in ARC arch

Message ID fdd37b0b995d178fddb34fdbdb5daf5981822f52.1470757628.git.jpinto@synopsys.com
State Accepted
Headers show

Commit Message

Joao Pinto Aug. 9, 2016, 3:49 p.m. UTC
To fix this, this patch adds ARC as a PCI_MSI_IRQ_DOMAIN supportive
platform and adds the generation of msi.h in the ARC arch.

Signed-off-by: Joao Pinto <jpinto@synopsys.com>
---
 arch/arc/include/asm/Kbuild | 1 +
 drivers/pci/Kconfig         | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

Comments

Vineet Gupta Aug. 10, 2016, 5:55 p.m. UTC | #1
On 08/09/2016 08:51 AM, Joao Pinto wrote:
> Due to the added dependency on PCI_MSI_IRQ_DOMAIN for all PCIe RC 
> drivers, we were unable to build a RC solution for an ARC platform.
>
> To fix this, this patch adds ARC as a PCI_MSI_IRQ_DOMAIN supportive
> platform and adds the generation of msi.h in the ARC arch.
>
> Signed-off-by: Joao Pinto <jpinto@synopsys.com>

LGTM - Bjorn I presume you will pick this up ?

-Vineet
> ---
>  arch/arc/include/asm/Kbuild | 1 +
>  drivers/pci/Kconfig         | 2 +-
>  2 files changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arc/include/asm/Kbuild b/arch/arc/include/asm/Kbuild
> index 0b10ef2..c332604 100644
> --- a/arch/arc/include/asm/Kbuild
> +++ b/arch/arc/include/asm/Kbuild
> @@ -25,6 +25,7 @@ generic-y += mcs_spinlock.h
>  generic-y += mm-arch-hooks.h
>  generic-y += mman.h
>  generic-y += msgbuf.h
> +generic-y += msi.h
>  generic-y += param.h
>  generic-y += parport.h
>  generic-y += pci.h
> diff --git a/drivers/pci/Kconfig b/drivers/pci/Kconfig
> index 67f9916..6555eb7 100644
> --- a/drivers/pci/Kconfig
> +++ b/drivers/pci/Kconfig
> @@ -25,7 +25,7 @@ config PCI_MSI
>  	   If you don't know what to do here, say Y.
>  
>  config PCI_MSI_IRQ_DOMAIN
> -	def_bool ARM || ARM64 || X86
> +	def_bool ARC || ARM || ARM64 || X86
>  	depends on PCI_MSI
>  	select GENERIC_MSI_IRQ_DOMAIN
>  


--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Joao Pinto Sept. 6, 2016, 11:05 a.m. UTC | #2
Hi Bjorn,
Did you have the chance to check this patch?

Thanks.

On 8/10/2016 6:55 PM, Vineet Gupta wrote:
> On 08/09/2016 08:51 AM, Joao Pinto wrote:
>> Due to the added dependency on PCI_MSI_IRQ_DOMAIN for all PCIe RC 
>> drivers, we were unable to build a RC solution for an ARC platform.
>>
>> To fix this, this patch adds ARC as a PCI_MSI_IRQ_DOMAIN supportive
>> platform and adds the generation of msi.h in the ARC arch.
>>
>> Signed-off-by: Joao Pinto <jpinto@synopsys.com>
> 
> LGTM - Bjorn I presume you will pick this up ?
> 
> -Vineet
>> ---
>>  arch/arc/include/asm/Kbuild | 1 +
>>  drivers/pci/Kconfig         | 2 +-
>>  2 files changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/arch/arc/include/asm/Kbuild b/arch/arc/include/asm/Kbuild
>> index 0b10ef2..c332604 100644
>> --- a/arch/arc/include/asm/Kbuild
>> +++ b/arch/arc/include/asm/Kbuild
>> @@ -25,6 +25,7 @@ generic-y += mcs_spinlock.h
>>  generic-y += mm-arch-hooks.h
>>  generic-y += mman.h
>>  generic-y += msgbuf.h
>> +generic-y += msi.h
>>  generic-y += param.h
>>  generic-y += parport.h
>>  generic-y += pci.h
>> diff --git a/drivers/pci/Kconfig b/drivers/pci/Kconfig
>> index 67f9916..6555eb7 100644
>> --- a/drivers/pci/Kconfig
>> +++ b/drivers/pci/Kconfig
>> @@ -25,7 +25,7 @@ config PCI_MSI
>>  	   If you don't know what to do here, say Y.
>>  
>>  config PCI_MSI_IRQ_DOMAIN
>> -	def_bool ARM || ARM64 || X86
>> +	def_bool ARC || ARM || ARM64 || X86
>>  	depends on PCI_MSI
>>  	select GENERIC_MSI_IRQ_DOMAIN
>>  
> 
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Bjorn Helgaas Sept. 6, 2016, 3:57 p.m. UTC | #3
On Tue, Aug 09, 2016 at 04:49:37PM +0100, Joao Pinto wrote:
> To fix this, this patch adds ARC as a PCI_MSI_IRQ_DOMAIN supportive
> platform and adds the generation of msi.h in the ARC arch.
> 
> Signed-off-by: Joao Pinto <jpinto@synopsys.com>

I added Vineet's ack and put this on pci/msi for v4.9, thanks!

> ---
>  arch/arc/include/asm/Kbuild | 1 +
>  drivers/pci/Kconfig         | 2 +-
>  2 files changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/arch/arc/include/asm/Kbuild b/arch/arc/include/asm/Kbuild
> index 0b10ef2..c332604 100644
> --- a/arch/arc/include/asm/Kbuild
> +++ b/arch/arc/include/asm/Kbuild
> @@ -25,6 +25,7 @@ generic-y += mcs_spinlock.h
>  generic-y += mm-arch-hooks.h
>  generic-y += mman.h
>  generic-y += msgbuf.h
> +generic-y += msi.h
>  generic-y += param.h
>  generic-y += parport.h
>  generic-y += pci.h
> diff --git a/drivers/pci/Kconfig b/drivers/pci/Kconfig
> index 67f9916..6555eb7 100644
> --- a/drivers/pci/Kconfig
> +++ b/drivers/pci/Kconfig
> @@ -25,7 +25,7 @@ config PCI_MSI
>  	   If you don't know what to do here, say Y.
>  
>  config PCI_MSI_IRQ_DOMAIN
> -	def_bool ARM || ARM64 || X86
> +	def_bool ARC || ARM || ARM64 || X86
>  	depends on PCI_MSI
>  	select GENERIC_MSI_IRQ_DOMAIN
>  
> -- 
> 1.8.1.5
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-pci" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Joao Pinto Sept. 6, 2016, 4:14 p.m. UTC | #4
On 9/6/2016 4:57 PM, Bjorn Helgaas wrote:
> On Tue, Aug 09, 2016 at 04:49:37PM +0100, Joao Pinto wrote:
>> To fix this, this patch adds ARC as a PCI_MSI_IRQ_DOMAIN supportive
>> platform and adds the generation of msi.h in the ARC arch.
>>
>> Signed-off-by: Joao Pinto <jpinto@synopsys.com>
> 
> I added Vineet's ack and put this on pci/msi for v4.9, thanks!

Great! Thanks!

> 
>> ---
>>  arch/arc/include/asm/Kbuild | 1 +
>>  drivers/pci/Kconfig         | 2 +-
>>  2 files changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/arch/arc/include/asm/Kbuild b/arch/arc/include/asm/Kbuild
>> index 0b10ef2..c332604 100644
>> --- a/arch/arc/include/asm/Kbuild
>> +++ b/arch/arc/include/asm/Kbuild
>> @@ -25,6 +25,7 @@ generic-y += mcs_spinlock.h
>>  generic-y += mm-arch-hooks.h
>>  generic-y += mman.h
>>  generic-y += msgbuf.h
>> +generic-y += msi.h
>>  generic-y += param.h
>>  generic-y += parport.h
>>  generic-y += pci.h
>> diff --git a/drivers/pci/Kconfig b/drivers/pci/Kconfig
>> index 67f9916..6555eb7 100644
>> --- a/drivers/pci/Kconfig
>> +++ b/drivers/pci/Kconfig
>> @@ -25,7 +25,7 @@ config PCI_MSI
>>  	   If you don't know what to do here, say Y.
>>  
>>  config PCI_MSI_IRQ_DOMAIN
>> -	def_bool ARM || ARM64 || X86
>> +	def_bool ARC || ARM || ARM64 || X86
>>  	depends on PCI_MSI
>>  	select GENERIC_MSI_IRQ_DOMAIN
>>  
>> -- 
>> 1.8.1.5
>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-pci" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html

--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/arch/arc/include/asm/Kbuild b/arch/arc/include/asm/Kbuild
index 0b10ef2..c332604 100644
--- a/arch/arc/include/asm/Kbuild
+++ b/arch/arc/include/asm/Kbuild
@@ -25,6 +25,7 @@  generic-y += mcs_spinlock.h
 generic-y += mm-arch-hooks.h
 generic-y += mman.h
 generic-y += msgbuf.h
+generic-y += msi.h
 generic-y += param.h
 generic-y += parport.h
 generic-y += pci.h
diff --git a/drivers/pci/Kconfig b/drivers/pci/Kconfig
index 67f9916..6555eb7 100644
--- a/drivers/pci/Kconfig
+++ b/drivers/pci/Kconfig
@@ -25,7 +25,7 @@  config PCI_MSI
 	   If you don't know what to do here, say Y.
 
 config PCI_MSI_IRQ_DOMAIN
-	def_bool ARM || ARM64 || X86
+	def_bool ARC || ARM || ARM64 || X86
 	depends on PCI_MSI
 	select GENERIC_MSI_IRQ_DOMAIN