From mboxrd@z Thu Jan 1 00:00:00 1970 From: gregory.clement@free-electrons.com (Gregory CLEMENT) Date: Wed, 21 Jun 2017 17:50:20 +0200 Subject: [PATCH v5 0/6] Add support for the ICU unit in Marvell Armada 7K/8K In-Reply-To: <245df82c-60b1-bc4b-e343-5435713c453c@arm.com> (Marc Zyngier's message of "Wed, 21 Jun 2017 16:47:20 +0100") References: <20170621132917.14187-1-thomas.petazzoni@free-electrons.com> <20170621171809.34bcce1e@windsurf.home> <87r2yd1h5x.fsf@free-electrons.com> <245df82c-60b1-bc4b-e343-5435713c453c@arm.com> Message-ID: <87k2451gsj.fsf@free-electrons.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Marc, On mer., juin 21 2017, Marc Zyngier wrote: > On 21/06/17 16:42, Gregory CLEMENT wrote: >> Hi Marc and Thomas, >> >> On mer., juin 21 2017, Thomas Petazzoni wrote: >> >>> Hello, >>> >>> On Wed, 21 Jun 2017 16:14:21 +0100, Marc Zyngier wrote: >>> >>>>> Thomas Petazzoni (6): >>>>> dt-bindings: interrupt-controller: add DT binding for the Marvell GICP >>>>> dt-bindings: interrupt-controller: add DT binding for the Marvell ICU >>>>> irqchip: irq-mvebu-gicp: new driver for Marvell GICP >>>>> irqchip: irq-mvebu-icu: new driver for Marvell ICU >>>>> arm64: marvell: enable ICU and GICP drivers >>>>> arm64: dts: marvell: enable GICP and ICU on Armada 7K/8K >>>>> >>>>> .../bindings/interrupt-controller/marvell,gicp.txt | 27 ++ >>>>> .../bindings/interrupt-controller/marvell,icu.txt | 51 ++++ >>>>> arch/arm64/Kconfig.platforms | 2 + >>>>> arch/arm64/boot/dts/marvell/armada-ap806.dtsi | 7 + >>>>> .../boot/dts/marvell/armada-cp110-master.dtsi | 59 +++-- >>>>> .../arm64/boot/dts/marvell/armada-cp110-slave.dtsi | 55 ++-- >>>>> drivers/irqchip/Kconfig | 6 + >>>>> drivers/irqchip/Makefile | 2 + >>>>> drivers/irqchip/irq-mvebu-gicp.c | 279 ++++++++++++++++++++ >>>>> drivers/irqchip/irq-mvebu-gicp.h | 12 + >>>>> drivers/irqchip/irq-mvebu-icu.c | 289 +++++++++++++++++++++ >>>>> .../dt-bindings/interrupt-controller/mvebu-icu.h | 15 ++ >>>>> 12 files changed, 756 insertions(+), 48 deletions(-) >>>>> create mode 100644 Documentation/devicetree/bindings/interrupt-controller/marvell,gicp.txt >>>>> create mode 100644 Documentation/devicetree/bindings/interrupt-controller/marvell,icu.txt >>>>> create mode 100644 drivers/irqchip/irq-mvebu-gicp.c >>>>> create mode 100644 drivers/irqchip/irq-mvebu-gicp.h >>>>> create mode 100644 drivers/irqchip/irq-mvebu-icu.c >>>>> create mode 100644 include/dt-bindings/interrupt-controller/mvebu-icu.h >>>>> >>>> >>>> It all looks good to me. How do we merge this? I take the first five >>>> patches and Gregory takes the last one? >>> >>> I think you should take the first four patches, and Gr?gory takes the >>> last two ones. But I'll let Gr?gory confirm. >> >> Indeed it makes more sens to merge the patch 4 through the arm-soc >> subsystem. > > Hmmm. You mean patches 5 and 6, right? And I queue 1 to 4? Yes that was what I meant! (it's too hot here and my brain is melting...) Thanks, Gregory > > Thanks, > > M. > -- > Jazz is not dead. It just smells funny... -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com From mboxrd@z Thu Jan 1 00:00:00 1970 From: Gregory CLEMENT Subject: Re: [PATCH v5 0/6] Add support for the ICU unit in Marvell Armada 7K/8K Date: Wed, 21 Jun 2017 17:50:20 +0200 Message-ID: <87k2451gsj.fsf@free-electrons.com> References: <20170621132917.14187-1-thomas.petazzoni@free-electrons.com> <20170621171809.34bcce1e@windsurf.home> <87r2yd1h5x.fsf@free-electrons.com> <245df82c-60b1-bc4b-e343-5435713c453c@arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <245df82c-60b1-bc4b-e343-5435713c453c@arm.com> (Marc Zyngier's message of "Wed, 21 Jun 2017 16:47:20 +0100") List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Marc Zyngier Cc: Thomas Petazzoni , devicetree@vger.kernel.org, Yehuda Yitschak , Jason Cooper , Pawel Moll , Ian Campbell , Hanna Hawa , Mark Rutland , linux-kernel@vger.kernel.org, Nadav Haklai , Rob Herring , Andrew Lunn , =?utf-8?Q?Miqu?= =?utf-8?Q?=C3=A8l?= Raynal , Kumar Gala , Thomas Gleixner , Antoine Tenart , linux-arm-kernel@lists.infradead.org, Sebastian Hesselbarth List-Id: devicetree@vger.kernel.org SGkgTWFyYywKIAogT24gbWVyLiwganVpbiAyMSAyMDE3LCBNYXJjIFp5bmdpZXIgPG1hcmMuenlu Z2llckBhcm0uY29tPiB3cm90ZToKCj4gT24gMjEvMDYvMTcgMTY6NDIsIEdyZWdvcnkgQ0xFTUVO VCB3cm90ZToKPj4gSGkgTWFyYyBhbmQgVGhvbWFzLAo+PiAgCj4+ICBPbiBtZXIuLCBqdWluIDIx IDIwMTcsIFRob21hcyBQZXRhenpvbmkgPHRob21hcy5wZXRhenpvbmlAZnJlZS1lbGVjdHJvbnMu Y29tPiB3cm90ZToKPj4gCj4+PiBIZWxsbywKPj4+Cj4+PiBPbiBXZWQsIDIxIEp1biAyMDE3IDE2 OjE0OjIxICswMTAwLCBNYXJjIFp5bmdpZXIgd3JvdGU6Cj4+Pgo+Pj4+PiBUaG9tYXMgUGV0YXp6 b25pICg2KToKPj4+Pj4gICBkdC1iaW5kaW5nczogaW50ZXJydXB0LWNvbnRyb2xsZXI6IGFkZCBE VCBiaW5kaW5nIGZvciB0aGUgTWFydmVsbCBHSUNQCj4+Pj4+ICAgZHQtYmluZGluZ3M6IGludGVy cnVwdC1jb250cm9sbGVyOiBhZGQgRFQgYmluZGluZyBmb3IgdGhlIE1hcnZlbGwgSUNVCj4+Pj4+ ICAgaXJxY2hpcDogaXJxLW12ZWJ1LWdpY3A6IG5ldyBkcml2ZXIgZm9yIE1hcnZlbGwgR0lDUAo+ Pj4+PiAgIGlycWNoaXA6IGlycS1tdmVidS1pY3U6IG5ldyBkcml2ZXIgZm9yIE1hcnZlbGwgSUNV Cj4+Pj4+ICAgYXJtNjQ6IG1hcnZlbGw6IGVuYWJsZSBJQ1UgYW5kIEdJQ1AgZHJpdmVycwo+Pj4+ PiAgIGFybTY0OiBkdHM6IG1hcnZlbGw6IGVuYWJsZSBHSUNQIGFuZCBJQ1Ugb24gQXJtYWRhIDdL LzhLCj4+Pj4+Cj4+Pj4+ICAuLi4vYmluZGluZ3MvaW50ZXJydXB0LWNvbnRyb2xsZXIvbWFydmVs bCxnaWNwLnR4dCB8ICAyNyArKwo+Pj4+PiAgLi4uL2JpbmRpbmdzL2ludGVycnVwdC1jb250cm9s bGVyL21hcnZlbGwsaWN1LnR4dCAgfCAgNTEgKysrKwo+Pj4+PiAgYXJjaC9hcm02NC9LY29uZmln LnBsYXRmb3JtcyAgICAgICAgICAgICAgICAgICAgICAgfCAgIDIgKwo+Pj4+PiAgYXJjaC9hcm02 NC9ib290L2R0cy9tYXJ2ZWxsL2FybWFkYS1hcDgwNi5kdHNpICAgICAgfCAgIDcgKwo+Pj4+PiAg Li4uL2Jvb3QvZHRzL21hcnZlbGwvYXJtYWRhLWNwMTEwLW1hc3Rlci5kdHNpICAgICAgfCAgNTkg KysrLS0KPj4+Pj4gIC4uLi9hcm02NC9ib290L2R0cy9tYXJ2ZWxsL2FybWFkYS1jcDExMC1zbGF2 ZS5kdHNpIHwgIDU1ICsrLS0KPj4+Pj4gIGRyaXZlcnMvaXJxY2hpcC9LY29uZmlnICAgICAgICAg ICAgICAgICAgICAgICAgICAgIHwgICA2ICsKPj4+Pj4gIGRyaXZlcnMvaXJxY2hpcC9NYWtlZmls ZSAgICAgICAgICAgICAgICAgICAgICAgICAgIHwgICAyICsKPj4+Pj4gIGRyaXZlcnMvaXJxY2hp cC9pcnEtbXZlYnUtZ2ljcC5jICAgICAgICAgICAgICAgICAgIHwgMjc5ICsrKysrKysrKysrKysr KysrKysrCj4+Pj4+ICBkcml2ZXJzL2lycWNoaXAvaXJxLW12ZWJ1LWdpY3AuaCAgICAgICAgICAg ICAgICAgICB8ICAxMiArCj4+Pj4+ICBkcml2ZXJzL2lycWNoaXAvaXJxLW12ZWJ1LWljdS5jICAg ICAgICAgICAgICAgICAgICB8IDI4OSArKysrKysrKysrKysrKysrKysrKysKPj4+Pj4gIC4uLi9k dC1iaW5kaW5ncy9pbnRlcnJ1cHQtY29udHJvbGxlci9tdmVidS1pY3UuaCAgIHwgIDE1ICsrCj4+ Pj4+ICAxMiBmaWxlcyBjaGFuZ2VkLCA3NTYgaW5zZXJ0aW9ucygrKSwgNDggZGVsZXRpb25zKC0p Cj4+Pj4+ICBjcmVhdGUgbW9kZSAxMDA2NDQgRG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRp bmdzL2ludGVycnVwdC1jb250cm9sbGVyL21hcnZlbGwsZ2ljcC50eHQKPj4+Pj4gIGNyZWF0ZSBt b2RlIDEwMDY0NCBEb2N1bWVudGF0aW9uL2RldmljZXRyZWUvYmluZGluZ3MvaW50ZXJydXB0LWNv bnRyb2xsZXIvbWFydmVsbCxpY3UudHh0Cj4+Pj4+ICBjcmVhdGUgbW9kZSAxMDA2NDQgZHJpdmVy cy9pcnFjaGlwL2lycS1tdmVidS1naWNwLmMKPj4+Pj4gIGNyZWF0ZSBtb2RlIDEwMDY0NCBkcml2 ZXJzL2lycWNoaXAvaXJxLW12ZWJ1LWdpY3AuaAo+Pj4+PiAgY3JlYXRlIG1vZGUgMTAwNjQ0IGRy aXZlcnMvaXJxY2hpcC9pcnEtbXZlYnUtaWN1LmMKPj4+Pj4gIGNyZWF0ZSBtb2RlIDEwMDY0NCBp bmNsdWRlL2R0LWJpbmRpbmdzL2ludGVycnVwdC1jb250cm9sbGVyL212ZWJ1LWljdS5oCj4+Pj4+ ICAgCj4+Pj4KPj4+PiBJdCBhbGwgbG9va3MgZ29vZCB0byBtZS4gSG93IGRvIHdlIG1lcmdlIHRo aXM/IEkgdGFrZSB0aGUgZmlyc3QgZml2ZQo+Pj4+IHBhdGNoZXMgYW5kIEdyZWdvcnkgdGFrZXMg dGhlIGxhc3Qgb25lPwo+Pj4KPj4+IEkgdGhpbmsgeW91IHNob3VsZCB0YWtlIHRoZSBmaXJzdCBm b3VyIHBhdGNoZXMsIGFuZCBHcsOpZ29yeSB0YWtlcyB0aGUKPj4+IGxhc3QgdHdvIG9uZXMuIEJ1 dCBJJ2xsIGxldCBHcsOpZ29yeSBjb25maXJtLgo+PiAKPj4gSW5kZWVkIGl0IG1ha2VzIG1vcmUg c2VucyB0byBtZXJnZSB0aGUgcGF0Y2ggNCB0aHJvdWdoIHRoZSBhcm0tc29jCj4+IHN1YnN5c3Rl bS4KPgo+IEhtbW0uIFlvdSBtZWFuIHBhdGNoZXMgNSBhbmQgNiwgcmlnaHQ/IEFuZCBJIHF1ZXVl IDEgdG8gND8KClllcyB0aGF0IHdhcyB3aGF0IEkgbWVhbnQhCgooaXQncyB0b28gaG90IGhlcmUg YW5kIG15IGJyYWluIGlzIG1lbHRpbmcuLi4pCgpUaGFua3MsCgpHcmVnb3J5Cgo+Cj4gVGhhbmtz LAo+Cj4gCU0uCj4gLS0gCj4gSmF6eiBpcyBub3QgZGVhZC4gSXQganVzdCBzbWVsbHMgZnVubnku Li4KCi0tIApHcmVnb3J5IENsZW1lbnQsIEZyZWUgRWxlY3Ryb25zCktlcm5lbCwgZHJpdmVycywg cmVhbC10aW1lIGFuZCBlbWJlZGRlZCBMaW51eApkZXZlbG9wbWVudCwgY29uc3VsdGluZywgdHJh aW5pbmcgYW5kIHN1cHBvcnQuCmh0dHA6Ly9mcmVlLWVsZWN0cm9ucy5jb20KCl9fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmxpbnV4LWFybS1rZXJuZWwgbWFp bGluZyBsaXN0CmxpbnV4LWFybS1rZXJuZWxAbGlzdHMuaW5mcmFkZWFkLm9yZwpodHRwOi8vbGlz dHMuaW5mcmFkZWFkLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xpbnV4LWFybS1rZXJuZWwK From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752056AbdFUQ1Q convert rfc822-to-8bit (ORCPT ); Wed, 21 Jun 2017 12:27:16 -0400 Received: from mail.free-electrons.com ([62.4.15.54]:39808 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751640AbdFUQ1O (ORCPT ); Wed, 21 Jun 2017 12:27:14 -0400 From: Gregory CLEMENT To: Marc Zyngier Cc: Thomas Petazzoni , Thomas Gleixner , Jason Cooper , linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, Rob Herring , Ian Campbell , Pawel Moll , Mark Rutland , Kumar Gala , Andrew Lunn , Sebastian Hesselbarth , linux-arm-kernel@lists.infradead.org, Nadav Haklai , Hanna Hawa , Yehuda Yitschak , Antoine Tenart , =?utf-8?Q?Miqu?= =?utf-8?Q?=C3=A8l?= Raynal Subject: Re: [PATCH v5 0/6] Add support for the ICU unit in Marvell Armada 7K/8K References: <20170621132917.14187-1-thomas.petazzoni@free-electrons.com> <20170621171809.34bcce1e@windsurf.home> <87r2yd1h5x.fsf@free-electrons.com> <245df82c-60b1-bc4b-e343-5435713c453c@arm.com> Date: Wed, 21 Jun 2017 17:50:20 +0200 In-Reply-To: <245df82c-60b1-bc4b-e343-5435713c453c@arm.com> (Marc Zyngier's message of "Wed, 21 Jun 2017 16:47:20 +0100") Message-ID: <87k2451gsj.fsf@free-electrons.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Marc, On mer., juin 21 2017, Marc Zyngier wrote: > On 21/06/17 16:42, Gregory CLEMENT wrote: >> Hi Marc and Thomas, >> >> On mer., juin 21 2017, Thomas Petazzoni wrote: >> >>> Hello, >>> >>> On Wed, 21 Jun 2017 16:14:21 +0100, Marc Zyngier wrote: >>> >>>>> Thomas Petazzoni (6): >>>>> dt-bindings: interrupt-controller: add DT binding for the Marvell GICP >>>>> dt-bindings: interrupt-controller: add DT binding for the Marvell ICU >>>>> irqchip: irq-mvebu-gicp: new driver for Marvell GICP >>>>> irqchip: irq-mvebu-icu: new driver for Marvell ICU >>>>> arm64: marvell: enable ICU and GICP drivers >>>>> arm64: dts: marvell: enable GICP and ICU on Armada 7K/8K >>>>> >>>>> .../bindings/interrupt-controller/marvell,gicp.txt | 27 ++ >>>>> .../bindings/interrupt-controller/marvell,icu.txt | 51 ++++ >>>>> arch/arm64/Kconfig.platforms | 2 + >>>>> arch/arm64/boot/dts/marvell/armada-ap806.dtsi | 7 + >>>>> .../boot/dts/marvell/armada-cp110-master.dtsi | 59 +++-- >>>>> .../arm64/boot/dts/marvell/armada-cp110-slave.dtsi | 55 ++-- >>>>> drivers/irqchip/Kconfig | 6 + >>>>> drivers/irqchip/Makefile | 2 + >>>>> drivers/irqchip/irq-mvebu-gicp.c | 279 ++++++++++++++++++++ >>>>> drivers/irqchip/irq-mvebu-gicp.h | 12 + >>>>> drivers/irqchip/irq-mvebu-icu.c | 289 +++++++++++++++++++++ >>>>> .../dt-bindings/interrupt-controller/mvebu-icu.h | 15 ++ >>>>> 12 files changed, 756 insertions(+), 48 deletions(-) >>>>> create mode 100644 Documentation/devicetree/bindings/interrupt-controller/marvell,gicp.txt >>>>> create mode 100644 Documentation/devicetree/bindings/interrupt-controller/marvell,icu.txt >>>>> create mode 100644 drivers/irqchip/irq-mvebu-gicp.c >>>>> create mode 100644 drivers/irqchip/irq-mvebu-gicp.h >>>>> create mode 100644 drivers/irqchip/irq-mvebu-icu.c >>>>> create mode 100644 include/dt-bindings/interrupt-controller/mvebu-icu.h >>>>> >>>> >>>> It all looks good to me. How do we merge this? I take the first five >>>> patches and Gregory takes the last one? >>> >>> I think you should take the first four patches, and Grégory takes the >>> last two ones. But I'll let Grégory confirm. >> >> Indeed it makes more sens to merge the patch 4 through the arm-soc >> subsystem. > > Hmmm. You mean patches 5 and 6, right? And I queue 1 to 4? Yes that was what I meant! (it's too hot here and my brain is melting...) Thanks, Gregory > > Thanks, > > M. > -- > Jazz is not dead. It just smells funny... -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com