* [PATCH 0/3] irqchip: irq-bcm2836: add support for DT interrupt polarity @ 2017-12-11 20:39 Stefan Wahren [not found] ` <1513024752-11246-1-git-send-email-stefan.wahren-eS4NqCHxEME@public.gmane.org> 0 siblings, 1 reply; 13+ messages in thread From: Stefan Wahren @ 2017-12-11 20:39 UTC (permalink / raw) To: Thomas Gleixner, Jason Cooper, Marc Zyngier, Eric Anholt, Florian Fainelli, Scott Branden, Rob Herring, Mark Rutland Cc: Russell King, phil-FnsA7b+Nu9XbIbC87yuRow, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, linux-rpi-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Stefan Wahren This patch series implements DT polarity support for the 1st level interrupt controller. Stefan Wahren (3): dt-bindings: bcm2836-l1-intc: add interrupt polarity support irqchip: irq-bcm2836: add support for DT interrupt polarity ARM: dts: bcm283x: Define polarity of per-cpu interrupts .../interrupt-controller/brcm,bcm2836-l1-intc.txt | 4 +- arch/arm/boot/dts/bcm2836.dtsi | 14 +++---- arch/arm/boot/dts/bcm2837.dtsi | 12 +++--- arch/arm/boot/dts/bcm283x.dtsi | 1 + drivers/irqchip/irq-bcm2836.c | 46 +++++++++++++--------- 5 files changed, 44 insertions(+), 33 deletions(-) -- 2.7.4 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 13+ messages in thread
[parent not found: <1513024752-11246-1-git-send-email-stefan.wahren-eS4NqCHxEME@public.gmane.org>]
* [PATCH 1/3] dt-bindings: bcm2836-l1-intc: add interrupt polarity support [not found] ` <1513024752-11246-1-git-send-email-stefan.wahren-eS4NqCHxEME@public.gmane.org> @ 2017-12-11 20:39 ` Stefan Wahren [not found] ` <1513024752-11246-2-git-send-email-stefan.wahren-eS4NqCHxEME@public.gmane.org> 2017-12-11 20:39 ` [PATCH 2/3] irqchip: irq-bcm2836: add support for DT interrupt polarity Stefan Wahren ` (2 subsequent siblings) 3 siblings, 1 reply; 13+ messages in thread From: Stefan Wahren @ 2017-12-11 20:39 UTC (permalink / raw) To: Thomas Gleixner, Jason Cooper, Marc Zyngier, Eric Anholt, Florian Fainelli, Scott Branden, Rob Herring, Mark Rutland Cc: Russell King, phil-FnsA7b+Nu9XbIbC87yuRow, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, linux-rpi-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Stefan Wahren This increases the interrupt cells for the 1st level interrupt controller binding in order to describe the polarity like on the other ARM platforms. Signed-off-by: Stefan Wahren <stefan.wahren-eS4NqCHxEME@public.gmane.org> --- .../devicetree/bindings/interrupt-controller/brcm,bcm2836-l1-intc.txt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Documentation/devicetree/bindings/interrupt-controller/brcm,bcm2836-l1-intc.txt b/Documentation/devicetree/bindings/interrupt-controller/brcm,bcm2836-l1-intc.txt index f320dcd..8ced169 100644 --- a/Documentation/devicetree/bindings/interrupt-controller/brcm,bcm2836-l1-intc.txt +++ b/Documentation/devicetree/bindings/interrupt-controller/brcm,bcm2836-l1-intc.txt @@ -12,7 +12,7 @@ Required properties: registers - interrupt-controller: Identifies the node as an interrupt controller - #interrupt-cells: Specifies the number of cells needed to encode an - interrupt source. The value shall be 1 + interrupt source. The value shall be 2 Please refer to interrupts.txt in this directory for details of the common Interrupt Controllers bindings used by client devices. @@ -32,6 +32,6 @@ local_intc: local_intc { compatible = "brcm,bcm2836-l1-intc"; reg = <0x40000000 0x100>; interrupt-controller; - #interrupt-cells = <1>; + #interrupt-cells = <2>; interrupt-parent = <&local_intc>; }; -- 2.7.4 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply related [flat|nested] 13+ messages in thread
[parent not found: <1513024752-11246-2-git-send-email-stefan.wahren-eS4NqCHxEME@public.gmane.org>]
* Re: [PATCH 1/3] dt-bindings: bcm2836-l1-intc: add interrupt polarity support [not found] ` <1513024752-11246-2-git-send-email-stefan.wahren-eS4NqCHxEME@public.gmane.org> @ 2017-12-15 21:53 ` Eric Anholt 2017-12-15 22:23 ` Rob Herring 1 sibling, 0 replies; 13+ messages in thread From: Eric Anholt @ 2017-12-15 21:53 UTC (permalink / raw) To: Thomas Gleixner, Jason Cooper, Marc Zyngier, Florian Fainelli, Scott Branden, Rob Herring, Mark Rutland Cc: Russell King, phil-FnsA7b+Nu9XbIbC87yuRow, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, linux-rpi-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Stefan Wahren [-- Attachment #1: Type: text/plain, Size: 350 bytes --] Stefan Wahren <stefan.wahren-eS4NqCHxEME@public.gmane.org> writes: > This increases the interrupt cells for the 1st level interrupt controller > binding in order to describe the polarity like on the other ARM platforms. > > Signed-off-by: Stefan Wahren <stefan.wahren-eS4NqCHxEME@public.gmane.org> I'm happy with this. Any DT maintainer concerns? [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 832 bytes --] ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 1/3] dt-bindings: bcm2836-l1-intc: add interrupt polarity support [not found] ` <1513024752-11246-2-git-send-email-stefan.wahren-eS4NqCHxEME@public.gmane.org> 2017-12-15 21:53 ` Eric Anholt @ 2017-12-15 22:23 ` Rob Herring 1 sibling, 0 replies; 13+ messages in thread From: Rob Herring @ 2017-12-15 22:23 UTC (permalink / raw) To: Stefan Wahren Cc: Thomas Gleixner, Jason Cooper, Marc Zyngier, Eric Anholt, Florian Fainelli, Scott Branden, Mark Rutland, Russell King, phil-FnsA7b+Nu9XbIbC87yuRow, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, linux-rpi-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r On Mon, Dec 11, 2017 at 09:39:10PM +0100, Stefan Wahren wrote: > This increases the interrupt cells for the 1st level interrupt controller > binding in order to describe the polarity like on the other ARM platforms. > > Signed-off-by: Stefan Wahren <stefan.wahren-eS4NqCHxEME@public.gmane.org> > --- > .../devicetree/bindings/interrupt-controller/brcm,bcm2836-l1-intc.txt | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Reviewed-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH 2/3] irqchip: irq-bcm2836: add support for DT interrupt polarity [not found] ` <1513024752-11246-1-git-send-email-stefan.wahren-eS4NqCHxEME@public.gmane.org> 2017-12-11 20:39 ` [PATCH 1/3] dt-bindings: bcm2836-l1-intc: add interrupt polarity support Stefan Wahren @ 2017-12-11 20:39 ` Stefan Wahren 2017-12-11 20:39 ` [PATCH 3/3] ARM: dts: bcm283x: Define polarity of per-cpu interrupts Stefan Wahren 2017-12-19 7:02 ` [PATCH 0/3] irqchip: irq-bcm2836: add support for DT interrupt polarity Stefan Wahren 3 siblings, 0 replies; 13+ messages in thread From: Stefan Wahren @ 2017-12-11 20:39 UTC (permalink / raw) To: Thomas Gleixner, Jason Cooper, Marc Zyngier, Eric Anholt, Florian Fainelli, Scott Branden, Rob Herring, Mark Rutland Cc: Russell King, phil-FnsA7b+Nu9XbIbC87yuRow, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, linux-rpi-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Stefan Wahren In order to properly define the polarity of the per-cpu interrupts, we need to support for a second property cell. But this must be optional to keep backward compatibility with old DT blobs. Suggested-by: Marc Zyngier <marc.zyngier-5wv7dgnIgG8@public.gmane.org> Signed-off-by: Stefan Wahren <stefan.wahren-eS4NqCHxEME@public.gmane.org> --- drivers/irqchip/irq-bcm2836.c | 46 ++++++++++++++++++++++++++----------------- 1 file changed, 28 insertions(+), 18 deletions(-) diff --git a/drivers/irqchip/irq-bcm2836.c b/drivers/irqchip/irq-bcm2836.c index 667b9e1..dfe4a46 100644 --- a/drivers/irqchip/irq-bcm2836.c +++ b/drivers/irqchip/irq-bcm2836.c @@ -98,13 +98,35 @@ static struct irq_chip bcm2836_arm_irqchip_gpu = { .irq_unmask = bcm2836_arm_irqchip_unmask_gpu_irq, }; -static void bcm2836_arm_irqchip_register_irq(int hwirq, struct irq_chip *chip) -{ - int irq = irq_create_mapping(intc.domain, hwirq); +static int bcm2836_map(struct irq_domain *d, unsigned int irq, + irq_hw_number_t hw) +{ + struct irq_chip *chip; + + switch (hw) { + case LOCAL_IRQ_CNTPSIRQ: + case LOCAL_IRQ_CNTPNSIRQ: + case LOCAL_IRQ_CNTHPIRQ: + case LOCAL_IRQ_CNTVIRQ: + chip = &bcm2836_arm_irqchip_timer; + break; + case LOCAL_IRQ_GPU_FAST: + chip = &bcm2836_arm_irqchip_gpu; + break; + case LOCAL_IRQ_PMU_FAST: + chip = &bcm2836_arm_irqchip_pmu; + break; + default: + pr_warn_once("Unexpected hw irq: %lu\n", hw); + return -EINVAL; + } irq_set_percpu_devid(irq); - irq_set_chip_and_handler(irq, chip, handle_percpu_devid_irq); + irq_domain_set_info(d, irq, hw, chip, d->host_data, + handle_percpu_devid_irq, NULL, NULL); irq_set_status_flags(irq, IRQ_NOAUTOEN); + + return 0; } static void @@ -165,7 +187,8 @@ static int bcm2836_cpu_dying(unsigned int cpu) #endif static const struct irq_domain_ops bcm2836_arm_irqchip_intc_ops = { - .xlate = irq_domain_xlate_onecell + .xlate = irq_domain_xlate_onetwocell, + .map = bcm2836_map, }; static void @@ -218,19 +241,6 @@ static int __init bcm2836_arm_irqchip_l1_intc_of_init(struct device_node *node, if (!intc.domain) panic("%pOF: unable to create IRQ domain\n", node); - bcm2836_arm_irqchip_register_irq(LOCAL_IRQ_CNTPSIRQ, - &bcm2836_arm_irqchip_timer); - bcm2836_arm_irqchip_register_irq(LOCAL_IRQ_CNTPNSIRQ, - &bcm2836_arm_irqchip_timer); - bcm2836_arm_irqchip_register_irq(LOCAL_IRQ_CNTHPIRQ, - &bcm2836_arm_irqchip_timer); - bcm2836_arm_irqchip_register_irq(LOCAL_IRQ_CNTVIRQ, - &bcm2836_arm_irqchip_timer); - bcm2836_arm_irqchip_register_irq(LOCAL_IRQ_GPU_FAST, - &bcm2836_arm_irqchip_gpu); - bcm2836_arm_irqchip_register_irq(LOCAL_IRQ_PMU_FAST, - &bcm2836_arm_irqchip_pmu); - bcm2836_arm_irqchip_smp_init(); set_handle_irq(bcm2836_arm_irqchip_handle_irq); -- 2.7.4 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 3/3] ARM: dts: bcm283x: Define polarity of per-cpu interrupts [not found] ` <1513024752-11246-1-git-send-email-stefan.wahren-eS4NqCHxEME@public.gmane.org> 2017-12-11 20:39 ` [PATCH 1/3] dt-bindings: bcm2836-l1-intc: add interrupt polarity support Stefan Wahren 2017-12-11 20:39 ` [PATCH 2/3] irqchip: irq-bcm2836: add support for DT interrupt polarity Stefan Wahren @ 2017-12-11 20:39 ` Stefan Wahren 2017-12-19 7:02 ` [PATCH 0/3] irqchip: irq-bcm2836: add support for DT interrupt polarity Stefan Wahren 3 siblings, 0 replies; 13+ messages in thread From: Stefan Wahren @ 2017-12-11 20:39 UTC (permalink / raw) To: Thomas Gleixner, Jason Cooper, Marc Zyngier, Eric Anholt, Florian Fainelli, Scott Branden, Rob Herring, Mark Rutland Cc: Russell King, phil-FnsA7b+Nu9XbIbC87yuRow, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, linux-rpi-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Stefan Wahren This patch define the polarity of the per-cpu interrupts on BCM2836 and BCM2837 in order to avoid the warnings from ARM arch timer code: arch_timer: WARNING: Invalid trigger for IRQ19, assuming level low arch_timer: WARNING: Please fix your firmware arch_timer: cp15 timer(s) running at 19.20MHz (virt). Signed-off-by: Stefan Wahren <stefan.wahren-eS4NqCHxEME@public.gmane.org> --- arch/arm/boot/dts/bcm2836.dtsi | 14 +++++++------- arch/arm/boot/dts/bcm2837.dtsi | 12 ++++++------ arch/arm/boot/dts/bcm283x.dtsi | 1 + 3 files changed, 14 insertions(+), 13 deletions(-) diff --git a/arch/arm/boot/dts/bcm2836.dtsi b/arch/arm/boot/dts/bcm2836.dtsi index 61e1580..1dfd764 100644 --- a/arch/arm/boot/dts/bcm2836.dtsi +++ b/arch/arm/boot/dts/bcm2836.dtsi @@ -13,24 +13,24 @@ compatible = "brcm,bcm2836-l1-intc"; reg = <0x40000000 0x100>; interrupt-controller; - #interrupt-cells = <1>; + #interrupt-cells = <2>; interrupt-parent = <&local_intc>; }; arm-pmu { compatible = "arm,cortex-a7-pmu"; interrupt-parent = <&local_intc>; - interrupts = <9>; + interrupts = <9 IRQ_TYPE_LEVEL_HIGH>; }; }; timer { compatible = "arm,armv7-timer"; interrupt-parent = <&local_intc>; - interrupts = <0>, // PHYS_SECURE_PPI - <1>, // PHYS_NONSECURE_PPI - <3>, // VIRT_PPI - <2>; // HYP_PPI + interrupts = <0 IRQ_TYPE_LEVEL_HIGH>, // PHYS_SECURE_PPI + <1 IRQ_TYPE_LEVEL_HIGH>, // PHYS_NONSECURE_PPI + <3 IRQ_TYPE_LEVEL_HIGH>, // VIRT_PPI + <2 IRQ_TYPE_LEVEL_HIGH>; // HYP_PPI always-on; }; @@ -76,7 +76,7 @@ compatible = "brcm,bcm2836-armctrl-ic"; reg = <0x7e00b200 0x200>; interrupt-parent = <&local_intc>; - interrupts = <8>; + interrupts = <8 IRQ_TYPE_LEVEL_HIGH>; }; &cpu_thermal { diff --git a/arch/arm/boot/dts/bcm2837.dtsi b/arch/arm/boot/dts/bcm2837.dtsi index bc1cca5..efa7d33 100644 --- a/arch/arm/boot/dts/bcm2837.dtsi +++ b/arch/arm/boot/dts/bcm2837.dtsi @@ -12,7 +12,7 @@ compatible = "brcm,bcm2836-l1-intc"; reg = <0x40000000 0x100>; interrupt-controller; - #interrupt-cells = <1>; + #interrupt-cells = <2>; interrupt-parent = <&local_intc>; }; }; @@ -20,10 +20,10 @@ timer { compatible = "arm,armv7-timer"; interrupt-parent = <&local_intc>; - interrupts = <0>, // PHYS_SECURE_PPI - <1>, // PHYS_NONSECURE_PPI - <3>, // VIRT_PPI - <2>; // HYP_PPI + interrupts = <0 IRQ_TYPE_LEVEL_HIGH>, // PHYS_SECURE_PPI + <1 IRQ_TYPE_LEVEL_HIGH>, // PHYS_NONSECURE_PPI + <3 IRQ_TYPE_LEVEL_HIGH>, // VIRT_PPI + <2 IRQ_TYPE_LEVEL_HIGH>; // HYP_PPI always-on; }; @@ -73,7 +73,7 @@ compatible = "brcm,bcm2836-armctrl-ic"; reg = <0x7e00b200 0x200>; interrupt-parent = <&local_intc>; - interrupts = <8>; + interrupts = <8 IRQ_TYPE_LEVEL_HIGH>; }; &cpu_thermal { diff --git a/arch/arm/boot/dts/bcm283x.dtsi b/arch/arm/boot/dts/bcm283x.dtsi index e08203c..0d43bd4 100644 --- a/arch/arm/boot/dts/bcm283x.dtsi +++ b/arch/arm/boot/dts/bcm283x.dtsi @@ -2,6 +2,7 @@ #include <dt-bindings/clock/bcm2835.h> #include <dt-bindings/clock/bcm2835-aux.h> #include <dt-bindings/gpio/gpio.h> +#include <dt-bindings/interrupt-controller/irq.h> /* firmware-provided startup stubs live here, where the secondary CPUs are * spinning. -- 2.7.4 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH 0/3] irqchip: irq-bcm2836: add support for DT interrupt polarity [not found] ` <1513024752-11246-1-git-send-email-stefan.wahren-eS4NqCHxEME@public.gmane.org> ` (2 preceding siblings ...) 2017-12-11 20:39 ` [PATCH 3/3] ARM: dts: bcm283x: Define polarity of per-cpu interrupts Stefan Wahren @ 2017-12-19 7:02 ` Stefan Wahren [not found] ` <d0c81267-8379-d4a1-ec56-fec3492c1df6-eS4NqCHxEME@public.gmane.org> 3 siblings, 1 reply; 13+ messages in thread From: Stefan Wahren @ 2017-12-19 7:02 UTC (permalink / raw) To: Thomas Gleixner, Jason Cooper, Marc Zyngier, Eric Anholt, Florian Fainelli, Scott Branden, Rob Herring, Mark Rutland Cc: devicetree-u79uwXL29TY76Z2rM5mHXA, phil-FnsA7b+Nu9XbIbC87yuRow, Russell King, linux-rpi-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r Hi Marc, Am 11.12.2017 um 21:39 schrieb Stefan Wahren: > This patch series implements DT polarity support for the 1st level interrupt > controller. > > Stefan Wahren (3): > dt-bindings: bcm2836-l1-intc: add interrupt polarity support > irqchip: irq-bcm2836: add support for DT interrupt polarity > ARM: dts: bcm283x: Define polarity of per-cpu interrupts > > .../interrupt-controller/brcm,bcm2836-l1-intc.txt | 4 +- > arch/arm/boot/dts/bcm2836.dtsi | 14 +++---- > arch/arm/boot/dts/bcm2837.dtsi | 12 +++--- > arch/arm/boot/dts/bcm283x.dtsi | 1 + > drivers/irqchip/irq-bcm2836.c | 46 +++++++++++++--------- > 5 files changed, 44 insertions(+), 33 deletions(-) > is this series okay? -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 13+ messages in thread
[parent not found: <d0c81267-8379-d4a1-ec56-fec3492c1df6-eS4NqCHxEME@public.gmane.org>]
* Re: [PATCH 0/3] irqchip: irq-bcm2836: add support for DT interrupt polarity [not found] ` <d0c81267-8379-d4a1-ec56-fec3492c1df6-eS4NqCHxEME@public.gmane.org> @ 2017-12-19 16:36 ` Marc Zyngier [not found] ` <a7939fae-31d1-c743-5957-d88be98a0c63-5wv7dgnIgG8@public.gmane.org> 0 siblings, 1 reply; 13+ messages in thread From: Marc Zyngier @ 2017-12-19 16:36 UTC (permalink / raw) To: Stefan Wahren, Thomas Gleixner, Jason Cooper, Eric Anholt, Florian Fainelli, Scott Branden, Rob Herring, Mark Rutland Cc: devicetree-u79uwXL29TY76Z2rM5mHXA, phil-FnsA7b+Nu9XbIbC87yuRow, Russell King, linux-rpi-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r On 19/12/17 07:02, Stefan Wahren wrote: > Hi Marc, > > Am 11.12.2017 um 21:39 schrieb Stefan Wahren: >> This patch series implements DT polarity support for the 1st level interrupt >> controller. >> >> Stefan Wahren (3): >> dt-bindings: bcm2836-l1-intc: add interrupt polarity support >> irqchip: irq-bcm2836: add support for DT interrupt polarity >> ARM: dts: bcm283x: Define polarity of per-cpu interrupts >> >> .../interrupt-controller/brcm,bcm2836-l1-intc.txt | 4 +- >> arch/arm/boot/dts/bcm2836.dtsi | 14 +++---- >> arch/arm/boot/dts/bcm2837.dtsi | 12 +++--- >> arch/arm/boot/dts/bcm283x.dtsi | 1 + >> drivers/irqchip/irq-bcm2836.c | 46 +++++++++++++--------- >> 5 files changed, 44 insertions(+), 33 deletions(-) >> > > is this series okay? Yes, it does look good. I'll queue that for 4.16. Thanks, M. -- Jazz is not dead. It just smells funny... -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 13+ messages in thread
[parent not found: <a7939fae-31d1-c743-5957-d88be98a0c63-5wv7dgnIgG8@public.gmane.org>]
* Re: [PATCH 0/3] irqchip: irq-bcm2836: add support for DT interrupt polarity [not found] ` <a7939fae-31d1-c743-5957-d88be98a0c63-5wv7dgnIgG8@public.gmane.org> @ 2017-12-19 18:41 ` Eric Anholt [not found] ` <8737461ry0.fsf-WhKQ6XTQaPysTnJN9+BGXg@public.gmane.org> 2018-01-03 17:10 ` Stefan Wahren 1 sibling, 1 reply; 13+ messages in thread From: Eric Anholt @ 2017-12-19 18:41 UTC (permalink / raw) To: Marc Zyngier, Stefan Wahren, Thomas Gleixner, Jason Cooper, Florian Fainelli, Scott Branden, Rob Herring, Mark Rutland Cc: devicetree-u79uwXL29TY76Z2rM5mHXA, phil-FnsA7b+Nu9XbIbC87yuRow, Russell King, linux-rpi-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r [-- Attachment #1: Type: text/plain, Size: 1084 bytes --] Marc Zyngier <marc.zyngier-5wv7dgnIgG8@public.gmane.org> writes: > On 19/12/17 07:02, Stefan Wahren wrote: >> Hi Marc, >> >> Am 11.12.2017 um 21:39 schrieb Stefan Wahren: >>> This patch series implements DT polarity support for the 1st level interrupt >>> controller. >>> >>> Stefan Wahren (3): >>> dt-bindings: bcm2836-l1-intc: add interrupt polarity support >>> irqchip: irq-bcm2836: add support for DT interrupt polarity >>> ARM: dts: bcm283x: Define polarity of per-cpu interrupts >>> >>> .../interrupt-controller/brcm,bcm2836-l1-intc.txt | 4 +- >>> arch/arm/boot/dts/bcm2836.dtsi | 14 +++---- >>> arch/arm/boot/dts/bcm2837.dtsi | 12 +++--- >>> arch/arm/boot/dts/bcm283x.dtsi | 1 + >>> drivers/irqchip/irq-bcm2836.c | 46 +++++++++++++--------- >>> 5 files changed, 44 insertions(+), 33 deletions(-) >>> >> >> is this series okay? > > Yes, it does look good. I'll queue that for 4.16. Are you grabbing all 3, or should I be grabbing the DT one? [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 832 bytes --] ^ permalink raw reply [flat|nested] 13+ messages in thread
[parent not found: <8737461ry0.fsf-WhKQ6XTQaPysTnJN9+BGXg@public.gmane.org>]
* Re: [PATCH 0/3] irqchip: irq-bcm2836: add support for DT interrupt polarity [not found] ` <8737461ry0.fsf-WhKQ6XTQaPysTnJN9+BGXg@public.gmane.org> @ 2017-12-19 18:50 ` Marc Zyngier [not found] ` <97a68b71-2e62-13ba-72ac-575c3b68617a-5wv7dgnIgG8@public.gmane.org> 0 siblings, 1 reply; 13+ messages in thread From: Marc Zyngier @ 2017-12-19 18:50 UTC (permalink / raw) To: Eric Anholt, Stefan Wahren, Thomas Gleixner, Jason Cooper, Florian Fainelli, Scott Branden, Rob Herring, Mark Rutland Cc: devicetree-u79uwXL29TY76Z2rM5mHXA, phil-FnsA7b+Nu9XbIbC87yuRow, Russell King, linux-rpi-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r On 19/12/17 18:41, Eric Anholt wrote: > Marc Zyngier <marc.zyngier-5wv7dgnIgG8@public.gmane.org> writes: > >> On 19/12/17 07:02, Stefan Wahren wrote: >>> Hi Marc, >>> >>> Am 11.12.2017 um 21:39 schrieb Stefan Wahren: >>>> This patch series implements DT polarity support for the 1st level interrupt >>>> controller. >>>> >>>> Stefan Wahren (3): >>>> dt-bindings: bcm2836-l1-intc: add interrupt polarity support >>>> irqchip: irq-bcm2836: add support for DT interrupt polarity >>>> ARM: dts: bcm283x: Define polarity of per-cpu interrupts >>>> >>>> .../interrupt-controller/brcm,bcm2836-l1-intc.txt | 4 +- >>>> arch/arm/boot/dts/bcm2836.dtsi | 14 +++---- >>>> arch/arm/boot/dts/bcm2837.dtsi | 12 +++--- >>>> arch/arm/boot/dts/bcm283x.dtsi | 1 + >>>> drivers/irqchip/irq-bcm2836.c | 46 +++++++++++++--------- >>>> 5 files changed, 44 insertions(+), 33 deletions(-) >>>> >>> >>> is this series okay? >> >> Yes, it does look good. I'll queue that for 4.16. > > Are you grabbing all 3, or should I be grabbing the DT one? All 3, if that's OK with you. M. -- Jazz is not dead. It just smells funny... -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 13+ messages in thread
[parent not found: <97a68b71-2e62-13ba-72ac-575c3b68617a-5wv7dgnIgG8@public.gmane.org>]
* Re: [PATCH 0/3] irqchip: irq-bcm2836: add support for DT interrupt polarity [not found] ` <97a68b71-2e62-13ba-72ac-575c3b68617a-5wv7dgnIgG8@public.gmane.org> @ 2017-12-20 0:12 ` Eric Anholt 0 siblings, 0 replies; 13+ messages in thread From: Eric Anholt @ 2017-12-20 0:12 UTC (permalink / raw) To: Marc Zyngier, Stefan Wahren, Thomas Gleixner, Jason Cooper, Florian Fainelli, Scott Branden, Rob Herring, Mark Rutland Cc: devicetree-u79uwXL29TY76Z2rM5mHXA, phil-FnsA7b+Nu9XbIbC87yuRow, Russell King, linux-rpi-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r [-- Attachment #1: Type: text/plain, Size: 1299 bytes --] Marc Zyngier <marc.zyngier-5wv7dgnIgG8@public.gmane.org> writes: > On 19/12/17 18:41, Eric Anholt wrote: >> Marc Zyngier <marc.zyngier-5wv7dgnIgG8@public.gmane.org> writes: >> >>> On 19/12/17 07:02, Stefan Wahren wrote: >>>> Hi Marc, >>>> >>>> Am 11.12.2017 um 21:39 schrieb Stefan Wahren: >>>>> This patch series implements DT polarity support for the 1st level interrupt >>>>> controller. >>>>> >>>>> Stefan Wahren (3): >>>>> dt-bindings: bcm2836-l1-intc: add interrupt polarity support >>>>> irqchip: irq-bcm2836: add support for DT interrupt polarity >>>>> ARM: dts: bcm283x: Define polarity of per-cpu interrupts >>>>> >>>>> .../interrupt-controller/brcm,bcm2836-l1-intc.txt | 4 +- >>>>> arch/arm/boot/dts/bcm2836.dtsi | 14 +++---- >>>>> arch/arm/boot/dts/bcm2837.dtsi | 12 +++--- >>>>> arch/arm/boot/dts/bcm283x.dtsi | 1 + >>>>> drivers/irqchip/irq-bcm2836.c | 46 +++++++++++++--------- >>>>> 5 files changed, 44 insertions(+), 33 deletions(-) >>>>> >>>> >>>> is this series okay? >>> >>> Yes, it does look good. I'll queue that for 4.16. >> >> Are you grabbing all 3, or should I be grabbing the DT one? > > All 3, if that's OK with you. Go for it. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 832 bytes --] ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 0/3] irqchip: irq-bcm2836: add support for DT interrupt polarity [not found] ` <a7939fae-31d1-c743-5957-d88be98a0c63-5wv7dgnIgG8@public.gmane.org> 2017-12-19 18:41 ` Eric Anholt @ 2018-01-03 17:10 ` Stefan Wahren [not found] ` <60587e4d-d285-34e7-3d27-ea1daa1f97d1-eS4NqCHxEME@public.gmane.org> 1 sibling, 1 reply; 13+ messages in thread From: Stefan Wahren @ 2018-01-03 17:10 UTC (permalink / raw) To: Marc Zyngier Cc: Thomas Gleixner, Jason Cooper, Eric Anholt, Florian Fainelli, Scott Branden, Rob Herring, Mark Rutland, devicetree-u79uwXL29TY76Z2rM5mHXA, phil-FnsA7b+Nu9XbIbC87yuRow, Russell King, linux-rpi-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r Hi Marc, Am 19.12.2017 um 17:36 schrieb Marc Zyngier: > On 19/12/17 07:02, Stefan Wahren wrote: >> Hi Marc, >> >> Am 11.12.2017 um 21:39 schrieb Stefan Wahren: >>> This patch series implements DT polarity support for the 1st level interrupt >>> controller. >>> >>> Stefan Wahren (3): >>> dt-bindings: bcm2836-l1-intc: add interrupt polarity support >>> irqchip: irq-bcm2836: add support for DT interrupt polarity >>> ARM: dts: bcm283x: Define polarity of per-cpu interrupts >>> >>> .../interrupt-controller/brcm,bcm2836-l1-intc.txt | 4 +- >>> arch/arm/boot/dts/bcm2836.dtsi | 14 +++---- >>> arch/arm/boot/dts/bcm2837.dtsi | 12 +++--- >>> arch/arm/boot/dts/bcm283x.dtsi | 1 + >>> drivers/irqchip/irq-bcm2836.c | 46 +++++++++++++--------- >>> 5 files changed, 44 insertions(+), 33 deletions(-) >>> >> is this series okay? > Yes, it does look good. I'll queue that for 4.16. > > Thanks, > > M. since i didn't found this in linux-next, please take this as a gentle reminder. Thanks Stefan -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 13+ messages in thread
[parent not found: <60587e4d-d285-34e7-3d27-ea1daa1f97d1-eS4NqCHxEME@public.gmane.org>]
* Re: [PATCH 0/3] irqchip: irq-bcm2836: add support for DT interrupt polarity [not found] ` <60587e4d-d285-34e7-3d27-ea1daa1f97d1-eS4NqCHxEME@public.gmane.org> @ 2018-01-03 17:14 ` Marc Zyngier 0 siblings, 0 replies; 13+ messages in thread From: Marc Zyngier @ 2018-01-03 17:14 UTC (permalink / raw) To: Stefan Wahren Cc: Thomas Gleixner, Jason Cooper, Eric Anholt, Florian Fainelli, Scott Branden, Rob Herring, Mark Rutland, devicetree-u79uwXL29TY76Z2rM5mHXA, phil-FnsA7b+Nu9XbIbC87yuRow, Russell King, linux-rpi-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r On 03/01/18 17:10, Stefan Wahren wrote: > Hi Marc, > > > Am 19.12.2017 um 17:36 schrieb Marc Zyngier: >> On 19/12/17 07:02, Stefan Wahren wrote: >>> Hi Marc, >>> >>> Am 11.12.2017 um 21:39 schrieb Stefan Wahren: >>>> This patch series implements DT polarity support for the 1st level interrupt >>>> controller. >>>> >>>> Stefan Wahren (3): >>>> dt-bindings: bcm2836-l1-intc: add interrupt polarity support >>>> irqchip: irq-bcm2836: add support for DT interrupt polarity >>>> ARM: dts: bcm283x: Define polarity of per-cpu interrupts >>>> >>>> .../interrupt-controller/brcm,bcm2836-l1-intc.txt | 4 +- >>>> arch/arm/boot/dts/bcm2836.dtsi | 14 +++---- >>>> arch/arm/boot/dts/bcm2837.dtsi | 12 +++--- >>>> arch/arm/boot/dts/bcm283x.dtsi | 1 + >>>> drivers/irqchip/irq-bcm2836.c | 46 +++++++++++++--------- >>>> 5 files changed, 44 insertions(+), 33 deletions(-) >>>> >>> is this series okay? >> Yes, it does look good. I'll queue that for 4.16. >> >> Thanks, >> >> M. > > since i didn't found this in linux-next, please take this as a gentle > reminder. It will take some time before I push this to tglx, as I'm otherwise engaged. Patches are not lost. Thanks, M. -- Jazz is not dead. It just smells funny... -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2018-01-03 17:14 UTC | newest] Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2017-12-11 20:39 [PATCH 0/3] irqchip: irq-bcm2836: add support for DT interrupt polarity Stefan Wahren [not found] ` <1513024752-11246-1-git-send-email-stefan.wahren-eS4NqCHxEME@public.gmane.org> 2017-12-11 20:39 ` [PATCH 1/3] dt-bindings: bcm2836-l1-intc: add interrupt polarity support Stefan Wahren [not found] ` <1513024752-11246-2-git-send-email-stefan.wahren-eS4NqCHxEME@public.gmane.org> 2017-12-15 21:53 ` Eric Anholt 2017-12-15 22:23 ` Rob Herring 2017-12-11 20:39 ` [PATCH 2/3] irqchip: irq-bcm2836: add support for DT interrupt polarity Stefan Wahren 2017-12-11 20:39 ` [PATCH 3/3] ARM: dts: bcm283x: Define polarity of per-cpu interrupts Stefan Wahren 2017-12-19 7:02 ` [PATCH 0/3] irqchip: irq-bcm2836: add support for DT interrupt polarity Stefan Wahren [not found] ` <d0c81267-8379-d4a1-ec56-fec3492c1df6-eS4NqCHxEME@public.gmane.org> 2017-12-19 16:36 ` Marc Zyngier [not found] ` <a7939fae-31d1-c743-5957-d88be98a0c63-5wv7dgnIgG8@public.gmane.org> 2017-12-19 18:41 ` Eric Anholt [not found] ` <8737461ry0.fsf-WhKQ6XTQaPysTnJN9+BGXg@public.gmane.org> 2017-12-19 18:50 ` Marc Zyngier [not found] ` <97a68b71-2e62-13ba-72ac-575c3b68617a-5wv7dgnIgG8@public.gmane.org> 2017-12-20 0:12 ` Eric Anholt 2018-01-03 17:10 ` Stefan Wahren [not found] ` <60587e4d-d285-34e7-3d27-ea1daa1f97d1-eS4NqCHxEME@public.gmane.org> 2018-01-03 17:14 ` Marc Zyngier
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).