* [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
* [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
* [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 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
* 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
* 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
* 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
* 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
* 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
* 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).