From: Joao Pinto <Joao.Pinto@synopsys.com>
To: Vineet Gupta <Vineet.Gupta1@synopsys.com>,
"bhelgaas@google.com" <bhelgaas@google.com>
Cc: "linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"linux-snps-arc@lists.infradead.org"
<linux-snps-arc@lists.infradead.org>
Subject: Re: [PATCH] pci: adding support for PCI_MSI_IRQ_DOMAIN in ARC arch
Date: Tue, 6 Sep 2016 12:05:20 +0100 [thread overview]
Message-ID: <05c5bab3-77bf-9993-8661-49ca9f41a7ca@synopsys.com> (raw)
In-Reply-To: <64d6513b-13f1-953d-410b-a4aeea6908c3@synopsys.com>
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
>>
>
>
next prev parent reply other threads:[~2016-09-06 11:13 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-08-09 15:49 Joao Pinto
2016-08-10 17:55 ` Vineet Gupta
2016-09-06 11:05 ` Joao Pinto [this message]
2016-09-06 15:57 ` Bjorn Helgaas
2016-09-06 16:14 ` Joao Pinto
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=05c5bab3-77bf-9993-8661-49ca9f41a7ca@synopsys.com \
--to=joao.pinto@synopsys.com \
--cc=Vineet.Gupta1@synopsys.com \
--cc=bhelgaas@google.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=linux-snps-arc@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®