Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754205AbbGPHi5 (ORCPT ); Thu, 16 Jul 2015 03:38:57 -0400 Received: from foss.arm.com ([217.140.101.70]:42234 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752133AbbGPHiz (ORCPT ); Thu, 16 Jul 2015 03:38:55 -0400 Message-ID: <55A75F8B.3080009@arm.com> Date: Thu, 16 Jul 2015 08:38:51 +0100 From: Marc Zyngier Organization: ARM Ltd User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Icedove/31.7.0 MIME-Version: 1.0 To: David Daney CC: David Daney , "linux-arm-kernel@lists.infradead.org" , Catalin Marinas , Will Deacon , Bjorn Helgaas , "linux-pci@vger.kernel.org" , Thomas Gleixner , Jason Cooper , Robert Richter , "linux-kernel@vger.kernel.org" , David Daney Subject: Re: [PATCH 4/5] irqchip: gic-v3: Add gic_get_irq_domain() to get the irqdomain of the GIC. References: <1436979285-8177-1-git-send-email-ddaney.cavm@gmail.com> <1436979285-8177-5-git-send-email-ddaney.cavm@gmail.com> <55A6947F.8070903@arm.com> <55A6AD1F.902@caviumnetworks.com> In-Reply-To: <55A6AD1F.902@caviumnetworks.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2704 Lines: 75 On 15/07/15 19:57, David Daney wrote: > On 07/15/2015 10:12 AM, Marc Zyngier wrote: >> On 15/07/15 17:54, David Daney wrote: >>> From: David Daney >>> >>> Needed to map SPI interrupt sources. >>> >>> Signed-off-by: David Daney >>> --- >>> drivers/irqchip/irq-gic-v3.c | 5 +++++ >>> include/linux/irqchip/arm-gic-v3.h | 1 + >>> 2 files changed, 6 insertions(+) >>> >>> diff --git a/drivers/irqchip/irq-gic-v3.c b/drivers/irqchip/irq-gic-v3.c >>> index c52f7ba..0019fed 100644 >>> --- a/drivers/irqchip/irq-gic-v3.c >>> +++ b/drivers/irqchip/irq-gic-v3.c >>> @@ -58,6 +58,11 @@ static struct gic_chip_data gic_data __read_mostly; >>> /* Our default, arbitrary priority value. Linux only uses one anyway. */ >>> #define DEFAULT_PMR_VALUE 0xf0 >>> >>> +struct irq_domain *gic_get_irq_domain(void) >>> +{ >>> + return gic_data.domain; >>> +} >>> + >>> static inline unsigned int gic_irq(struct irq_data *d) >>> { >>> return d->hwirq; >>> diff --git a/include/linux/irqchip/arm-gic-v3.h b/include/linux/irqchip/arm-gic-v3.h >>> index 18e3757..5992224 100644 >>> --- a/include/linux/irqchip/arm-gic-v3.h >>> +++ b/include/linux/irqchip/arm-gic-v3.h >>> @@ -391,6 +391,7 @@ int its_init(struct device_node *node, struct rdists *rdists, >>> >>> typedef u32 (*its_pci_requester_id_t)(struct pci_dev *, u16); >>> void set_its_pci_requester_id(its_pci_requester_id_t fn); >>> +struct irq_domain *gic_get_irq_domain(void); >>> #endif >>> >>> #endif >>> >> >> Hmmmffff... You need the domain for SPIs?? >> >> What is wrong with putting these interrupts in your device tree? >> > > There is no device tree node for ECAM based "PCIe" devices, they are > discovered in the PCI bus scan, yet they still need to use SPI interrupts. What is different between these and the normal legacy interrupts that are normally expressed in the RC node using an interrupt map? Sorry if I'm only showing my ignorance here, but I'd like to understand the exact nature of the problem. > We need a way to be able to map these. However you're going to map them, it will not be by just blindly exporting random irqdomains from an unsuspecting interrupt controller. Patch 5 has established that you're using "virtual wire" SPIs, so we need to work on exposing that with the normal kernel abstraction, and not by messing with the internals of the GIC. Thanks, M. -- Jazz is not dead. It just smells funny... -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/